@denisopko
Стремлюсь стать выше)

Оцените верстку из PSD (не много, один блок сверстаный)?

Может есть отдельные площадки, не знаю, так как я начинающий. Если что, сорь)

Можете оценить верстку - https://layoutwork.github.io/Restaraunte/
Написать, какие у меня ошибки и так далее?
Адаптивность, попрошу, не трогать))) Его пока-что не делал. Только обычную, пк версию

У меня сомнения по поводу высоты блока header. Высоту блока должен задавать контент. И вот не знаю, правильно ли я в верстке сделал или нет.

Спасибо заранее
  • Вопрос задан
  • 265 просмотров
Пригласить эксперта
Ответы на вопрос 2
notiv-nt
@notiv-nt
Никогда особо не понимал зачем первый экран пихают в 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 максимум

если верстка без пре-процессоров/сборщико, то пора
Ответ написан
@66demon666
На мобилке расползлось эх
Ответ написан
Ваш ответ на вопрос

Войдите, чтобы написать ответ

Войти через центр авторизации
Похожие вопросы