Skip to content

фикс главной страницы#2

Merged
constklimenko merged 2 commits intoconstklimenko:masterfrom
bogdangill:bugfixes-gill
Jun 25, 2021
Merged

фикс главной страницы#2
constklimenko merged 2 commits intoconstklimenko:masterfrom
bogdangill:bugfixes-gill

Conversation

@bogdangill
Copy link
Copy Markdown
Collaborator

@bogdangill bogdangill commented Jun 24, 2021

header

  1. Сделал шапку более гибкой и резиновой. Пришлось отказаться от гридов, т.к. не получалось с ними добиться нужного эффекта. Флексы все равно были как фоллбэк, так что таким образом даже получилось уменьшить объем кода.
  2. Добавил min-width для блоков в процентах для более предсказуемого поведения на разных разрешениях экрана. Ширина отсчитывается относительно ширины родителя.
  3. Исправил мелкие недочеты по стилям в соответствии с дизайном в макете:
  • лишняя прозрачность при наведении
  • отсутствие внутренних отступов у пунктов меню
  • "прыгающие" пункты меню при наведении из-за нижней рамки

index_first

  1. поправлены svg иконки. Объединены в гексагоном в одно изображение, добавляются через БЭМ-миксины в качестве фонового изображения. Создание карточек с иконками генерируется в паге через миксин, который я чуть подправил для удобства. Предыдущий способ добавления иконок имел недостатки:
  • сами иконки внутри блоков, отображавших гексагон, уползали от центра при различных разрешениях экрана (скрин в багрепорте). Теперь, когда они объединены в одно изображение, то нет нужды вкладывать одно фоновое в другое и прописывать стили для позиционирования.
  1. добавлен ховер для кнопки «Арендовать офис»

p.s. старые свойства я закомментировал, в будущем можно будет удалить и не тащить мусор на проду.

@bogdangill bogdangill changed the title фикс шапки (adaptive -> responsive) фикс главной страницы Jun 24, 2021
@constklimenko constklimenko merged commit 33dc92f into constklimenko:master Jun 25, 2021
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.

2 participants