Conversation
xaosaki
left a comment
There was a problem hiding this comment.
В целом очень много ненужных оберток.
- нужно привести в порядок sass
блоки разбить по файлам, переменные вынести в отдельный файл.
модификаторы держать рядом с их базовыми элементами/блоками
На кнопках нет ховер состояний
| <div class="wrapper"> | ||
| <div class="maincontent"> | ||
| <section id="main" class="section main"> | ||
| <div class="header__container"> |
There was a problem hiding this comment.
Почему-то . в header__container лежит и main__container. Так же элемент вне блока в данном случае не нужен. Рекомендую пересмотреть созвон более правильный пример разметки мы там показывали.
| </head> | ||
| <body> | ||
| <div class="wrapper"> | ||
| <div class="maincontent"> |
| <header class="header"> | ||
| <div class="container"> | ||
| <div class="header__logo"> | ||
| <a class="logo" href="#main"> |
There was a problem hiding this comment.
Тут ссылка не нужна, так как лого у нас висит на месте.
| <div class="container"> | ||
| <div class="header__logo"> | ||
| <a class="logo" href="#main"> | ||
| <img src="img/icons/logo.svg" alt="burgers logo"> |
There was a problem hiding this comment.
Картинку лучше сделать элементом logo. Дать ей класс logo__image
| <li class="nav__item"><a class="nav__link" >контакты</a> | ||
| </li> | ||
| </ul> | ||
| </nav><a class="order-link btn">Заказать</a> |
There was a problem hiding this comment.
Кнопку желательно через БЭМ привязать к header (элементом)
| </span> | ||
| </a> | ||
| </section> | ||
| <section id="weAreBest" class="section best"> |
There was a problem hiding this comment.
В html CamelCase обычно не используют
| .nav__list::after, | ||
| .menu-acco::after, | ||
| .order__form::after, | ||
| .form-row::after, |
There was a problem hiding this comment.
Зачем это все, есть же клирфикс
| clear: both; | ||
| } | ||
|
|
||
| .fixed-menu { |
There was a problem hiding this comment.
Лучше вынести в отдельный файл. В БЭМ есть правило отдельный блок -- отдельный файл.
| text-align: center; | ||
| } | ||
|
|
||
| .section__title_black { |
There was a problem hiding this comment.
Модификаторы лучше писать в файле с основным элементом
| @@ -0,0 +1,77 @@ | |||
| $black: #2f3234; | |||
There was a problem hiding this comment.
переменные должны быть в одном файле
No description provided.