Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bugfix footer visualisation #173

Merged
merged 2 commits into from Jun 3, 2020
Merged

Conversation

MrFSP
Copy link

@MrFSP MrFSP commented Jun 2, 2020

No description provided.

footer.bg-light.border-top.mt-auto.py-3
.container-xl
.d-md-flex.justify-content-around
.flex-column
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

flex-column срабатывает только вместе с флекс элементом (row, d-flex, etc)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

чем тебя не устраивает структура аля
.container-xl > .row.justify-content-around .col-md-4*3

Copy link
Author

@MrFSP MrFSP Jun 2, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  1. Про row - согласен.
  2. Без flex-column текст в колонках - смещённый. Я начинающий бутстрапщик. Просто так заработало, и лишних движений особо не делал

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

что значит смещенный?
скрин пожалуйста

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Screen Shot 2020-06-02 at 09 24 04

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Что насчет flex-column ?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Запуш новый код, иначе я не могу сказать что не так

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

я не понимаю чем тебе тут помогает .flex-column
отправь свои изменения и будет видно
Предполагаю что в твоем случае, тебе не важно какой класс будет, хоть просто пустой div, таким образом у тебя текст занимает свою ширину и выстраивается ровнее

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Поправка насчёт .row:
слева - .d-md-flex, справа - .row. На десктопе - одинаково.
Screen Shot 2020-06-03 at 07 28 16

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Всё вроде понятно как работает. Как начинаешь что-то делать - всё что-то не так

div.h5.mb-3= t('.help')

.flex-column
h5= t('.help')
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

до этого у тебя тут были <div class="h5">blabal</div>
теперь это стало <h5>blabla</h5>
кажется по пути потеряли семантику
если хочешь использовать это как заголовки какого-то уровня, но с этим же стилем, то стоит сделать что-то типа
h2.h5 === <h2 class="h5">babla</h2>

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Сделаю h2.h5.
Но не понял, что значит "семантика потерялась"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Семантика потерялась - имею ввиду что по логике у нас заголовки должны быть со своей структурой
типа после h1 идет h2, после h2 > h3 и тд
когда у тебя ниоткуда возникает h5 - это выглядит странным и неправильным

с одной стороны секция футер это должна нивелировать (по стандарту каждая секция может иметь свои заголовок первого уровня), но пока это работает не везде, и лучше быть последовательным

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

учитывая что h1 должен быть одним на странице, то в футере это должно быть что-то типа h2 или h3, но уже надо по структуре смотреть

@MrFSP MrFSP mentioned this pull request Jun 2, 2020
footer.bg-light.border-top.mt-auto.py-3
.container-xl
.d-md-flex.justify-content-around
.flex-column
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Запуш новый код, иначе я не могу сказать что не так

footer.bg-light.border-top.mt-auto.py-3
.container-xl
.d-md-flex.justify-content-around
.flex-column
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

я не понимаю чем тебе тут помогает .flex-column
отправь свои изменения и будет видно
Предполагаю что в твоем случае, тебе не важно какой класс будет, хоть просто пустой div, таким образом у тебя текст занимает свою ширину и выстраивается ровнее

@mokevnin mokevnin merged commit ef0c3ea into Hexlet:master Jun 3, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants