[SOK-13] main page #5
Conversation
Общие доработки: - Верстка шапки - Компонент заголовка - Перенос лайоутов в отдельные папки - PublicLayout - Добавил систему гридов - Доработки базовых стилей
…alcon-tanks into SOK-13_main-page
There was a problem hiding this comment.
Выглядит круто!
Анимация дыма на главной странице просто супер!
Грид система тоже выглядит отлична, она как раз мне поможет с ветской лидерборда и форма, благодарю.
Оставил пару комментов, возможно они наведут на новые мысли.
Так же заметил, что проект не отформатирован.
yarn run format возможно забыл. Наша проверка это не отлавила.
Заблочил PR из-за форматирования, все остальные момент дискуссионны.
| .main-nav { | ||
| display: flex; | ||
| gap: $nav_list-gap; | ||
| padding: $nav_list-padding-y $nav_list-padding-x; |
There was a problem hiding this comment.
Возможно было бы удобнее, если у каждого &__link был свой padding, а не общий padding, чтобы клацать на пункт меню было проще (так как размер элемента меню будет больше из-за padding`а).
There was a problem hiding this comment.
Не согласен. Такая большая зона для клика, будет вызывать ложные срабатывания.
There was a problem hiding this comment.
А ложные срабатывания на что? Там же вроде нет ничего рядом.
packages/client/src/pages/Main/components/EnemyTanks/EnemyTank.tsx
Outdated
Show resolved
Hide resolved
Сам лайоут написал внутри страницы.
с подложкой переименовал в СustomPageTitle - Добавил tag main в компонент PublicLayout
…to SOK-13_main-page
### Какую задачу решаем
и стили для publicLayout
Скриншоты/видяшка (если есть)
TBD (если есть)
Боюсь что из-за того что я убрал элемент div.app-layout,
могут возникнуть проблемы со страницей ошибки, если что поправим