hexlet-rus / runit

Online Editor Form Embedding
https://runit.hexlet.ru/
Other
33 stars 118 forks source link

fixed 992px header size collapse issue #453 #454

Closed bobrov-site closed 5 months ago

bobrov-site commented 5 months ago

https://github.com/hexlet-rus/runit/issues/453 вношу изменения по этому ишью

Сделал простое решение добавив пару классов бутстрапа на разных разрешениях. Немного отображение элементов в коллапсе. Поставил их слева, чтобы они смотрелись более гармоничнее

fey commented 5 months ago

@bobrov-site задеплойте, пожалуйста, демо

bobrov-site commented 5 months ago

А как и куда мне дето задеплоить?

bobrov-site commented 5 months ago

На vercel?

fey commented 5 months ago

@bobrov-site на render.com. У нас же еще бекенд. инструкция по деплою должна быть в ридми проека.

bobrov-site commented 5 months ago

Понял. Занимаюсь этим. Как сделаю пришлю.

bobrov-site commented 5 months ago

@fey https://bobrov-runit.onrender.com/

HelenOne commented 5 months ago

@bobrov-site, давайте отображение элементов оставим по центру, слева тоже странновато выглядит. Можно завести issue и подумать над новым отображением коллапса

bobrov-site commented 5 months ago

@fey и @HelenOne дайте пожалуйста отмашку по моему предложению с уменьшением логотипа и если нужно будет, я верну выравнивание элементов по центру

fey commented 5 months ago

@bobrov-site давайте уменьшим.

bobrov-site commented 5 months ago

У меня еще предложение по поводу расположение элементов внутри коллапса. Я почему принял изначально решение по левому краю, потому что верстка Лендинга вся опирается на левый край. Центрирование только достигается за счет паддингов контейнера. Поэтому когда элементы по центру, то лично мне кажется, что меню выбивается из дизайна. Но тут уже вам решать. Скриншот прикрепляю

Снимок экрана 2024-04-09 в 13 09 27

@HelenOne

fey commented 5 months ago

image image

в хекслете и кб, все что находится внутри аккордеона - по левому краю.

HelenOne commented 5 months ago

Окей, тогда оставляем по левому

bobrov-site commented 5 months ago

Изменения внес, можно посмотреть на https://bobrov-runit.onrender.com/

fey commented 5 months ago

По мне выглядит хорошо. Ждем ревью кода от Елены, Если ок, то можно мержить. Можно отдельным ПРом поправить отступы: image

bobrov-site commented 5 months ago

Без проблем, могу заняться отступами позже

fey commented 5 months ago

@HelenOne поревьювишь код?

HelenOne commented 5 months ago

По коду тоже всё хорошо, мержу