Никогда особо не понимал зачем первый экран пихают в header, ведь
The HTML element represents introductory content, typically a group of introductory or navigational aids. It may contain some heading elements but also a logo, a search form, an author name, and other elements.
https://developer.mozilla.org/en-US/docs/Web/HTML/...
Ну это ладно.
Но вот что лого делает в навигации, вот это очень хороший вопрос, на который хотелось бы услышать ответ
нет постоянства в коде, то
href = "#"
, то
href="#"
не особо по бэму
. block-cards и .wrap-block-cards
.header-wrap
.wrap-nav
alt = "icon"
как информативно то
font-family: 'MyriadProRegular';
нет fallback-шрифта
фон у
.logo__link
не картинкой, не помню как, но это 100% можно сделать через css, как и у
.block-cards__card
, UPD: например
https://codepen.io/notiv/pen/VwwxemQ?editors=1100
анимация наведения на
.block-cards__card
полный отстой, 50px слишком много, 15 максимум
если верстка без пре-процессоров/сборщико, то пора