Ilanevazno / UI-KIT

Second lesson
0 stars 0 forks source link

Фиксированные ширина и высота #33

Closed imin314 closed 4 years ago

imin314 commented 4 years ago

Пока что пример ненужного задания высоты: Выделение_019

Ты используешь height для управления высотой флекс-элементов в вертикальном флексбоксе, хотя в данном случае применение флексбокса кажется излишним: заголовок и контейнер для кнопок и так будут располагаться друг под другом, а выравнивания по центру можно добиться одним свойством text-align. Выделение_020

И лучше между кнопками добавить немного пространства. В стандартах даже есть пункт 10 про "лепку".

Ilanevazno commented 4 years ago

Убрал фиксированную ширину в панеле аккаунта, и прошёл по всем компонентам, заметил ещё одну в user-profile (тоже поправил)

imin314 commented 4 years ago

Правка была о том, что здесь можно обойтись без вертикального флексбокса и высоты (height, min-height).

Ilanevazno commented 4 years ago

Можно, но всё таки мне удобнее через флексбокс.

imin314 commented 4 years ago

А можешь объяснить, в чем проявляется удобство?:) Вертикальный порядок у p и div был бы по умолчанию, выравнивание по центру делается с помощью text-align: center. Свойство height: max-content совершенно ни на что не влияет. Выделение_130

Управление отступом между заголовком и кнопками происходит через высоту. Вместо этого нужно использовать margin. Выделение_133

Чтобы установить горизонтальный отступ между кнопками, ты переопределяешь box-sizing на content-box, чтобы ширина оставалась шириной контента, а padding не входил в это значение. Это очень неочевидное поведение, которого легко избежать, если использовать контейнеры-обертки и управлять отступами между кнопками в них. Выделение_132

Ilanevazno commented 4 years ago

сменил вёрстку.