solution to lection 2 histogram #13
Conversation
|
Добавляю преподавателя (@GSemikozov) для код-ревью. |
| border: 1px solid black; | ||
| display: flex; | ||
| flex-direction: column; | ||
| gap: 6px; |
There was a problem hiding this comment.
Да, между title и counter 6, значит 6))
There was a problem hiding this comment.
да, но ведь это добавляет отступ и между histogram__counter и histogram__preview, а там в свою очередь должно быть 39px
| display: flex; | ||
| justify-content: space-between; | ||
| align-items: flex-end; | ||
| gap: 5px; |
| font-weight: 400; | ||
| font-size: 20px; | ||
| line-height: 26px; | ||
| letter-spacing: 0%; |
There was a problem hiding this comment.
Почему 39px?
39 до первого столбика
до второго уже 54px
это же переменное расстояние
There was a problem hiding this comment.
нет, марджин снизу это не переменное расстяние, это конкрентное расстояние от .histogram__counter до histogram__preview
| } | ||
|
|
||
| .histogram__preview { | ||
| flex-grow: 1; |
There was a problem hiding this comment.
непонятно почему 50, не нашёл в макете такого значения
There was a problem hiding this comment.
самая высокая колонка в макете 50px
| display: flex; | ||
| justify-content: space-between; | ||
| align-items: flex-end; | ||
| gap: 5px; |
| } | ||
|
|
||
| .histogram__preview { | ||
| flex-grow: 1; |
There was a problem hiding this comment.
самая высокая колонка в макете 50px
| font-weight: 400; | ||
| font-size: 20px; | ||
| line-height: 26px; | ||
| letter-spacing: 0%; |
There was a problem hiding this comment.
нет, марджин снизу это не переменное расстяние, это конкрентное расстояние от .histogram__counter до histogram__preview
| border: 1px solid black; | ||
| display: flex; | ||
| flex-direction: column; | ||
| gap: 6px; |
There was a problem hiding this comment.
да, но ведь это добавляет отступ и между histogram__counter и histogram__preview, а там в свою очередь должно быть 39px
|
Добавляю преподавателя (@GSemikozov) для код-ревью. |
|
Решение было обновлено, посмотрим что скажет @GSemikozov |
5 similar comments
|
Решение было обновлено, посмотрим что скажет @GSemikozov |
|
Решение было обновлено, посмотрим что скажет @GSemikozov |
|
Решение было обновлено, посмотрим что скажет @GSemikozov |
|
Решение было обновлено, посмотрим что скажет @GSemikozov |
|
Решение было обновлено, посмотрим что скажет @GSemikozov |
|
Решение было обновлено, посмотрим что скажет @GSemikozov |
4 participants
No description provided.