fixed 992px header size collapse issue #453 #454
Conversation
|
@bobrov-site задеплойте, пожалуйста, демо |
|
А как и куда мне дето задеплоить? |
|
На vercel? |
|
@bobrov-site на render.com. У нас же еще бекенд. инструкция по деплою должна быть в ридми проека. |
|
Понял. Занимаюсь этим. Как сделаю пришлю. |
|
@bobrov-site, давайте отображение элементов оставим по центру, слева тоже странновато выглядит. Можно завести issue и подумать над новым отображением коллапса |
| @@ -21,7 +21,7 @@ function Header() { | |||
| const logo = isDarkMode ? RunItLogoDark : RunItLogoLight; | |||
| return ( | |||
| <header> | |||
| <Navbar expand="lg"> | |||
| <Navbar expand="xl"> | |||
There was a problem hiding this comment.
Мне кажется что это слишком много. См скрин - поменял класс на -lg, выглядит неплохо
There was a problem hiding this comment.
А ширина какая на скриншоте? Я специально поменял на xl, потому что проблема же на 1168 пикселей идет. Коллапс на lg сворачивает когда разрешение 991, что раньше необходимого. Пришлось сделать на XL, чтобы кнопка поместилась. Вот скрин как у меня на 1138 с lg.
There was a problem hiding this comment.
а, я понял. Да, с русским текстом хедер становится шире. Я проверял на 1024
Я думаю пока ваш вариант ок, в будущем можно убрать лишнее из хедера.
There was a problem hiding this comment.
Я могу предложить другой вариант. Можно уменьшить логотип до 150 пикселей в ширину. Тогда и коллапс останется на lg, как и @fey сделал. Получилось вот так. И все как раз влезает.
|
@bobrov-site давайте уменьшим. |
|
У меня еще предложение по поводу расположение элементов внутри коллапса. Я почему принял изначально решение по левому краю, потому что верстка Лендинга вся опирается на левый край. Центрирование только достигается за счет паддингов контейнера. Поэтому когда элементы по центру, то лично мне кажется, что меню выбивается из дизайна. Но тут уже вам решать. Скриншот прикрепляю |
|
в хекслете и кб, все что находится внутри аккордеона - по левому краю. |
|
Окей, тогда оставляем по левому |
|
Изменения внес, можно посмотреть на https://bobrov-runit.onrender.com/ |
|
По мне выглядит хорошо. Ждем ревью кода от Елены, Если ок, то можно мержить. |
|
Без проблем, могу заняться отступами позже |
|
@HelenOne поревьювишь код? |
|
По коду тоже всё хорошо, мержу |
#453 вношу изменения по этому ишью
Сделал простое решение добавив пару классов бутстрапа на разных разрешениях. Немного отображение элементов в коллапсе. Поставил их слева, чтобы они смотрелись более гармоничнее