1)"Никогда особо не понимал зачем первый экран пихают в header, ведь..."
Смотрел видео по верстке и так далее... И во многих делали в header. А как первый экран делать? Нет, ну можно было конечно и div с классом header. Как первый экран можно назвать? Это ведь шапка сайта. По поводу того,что логотип в навигации - об этом я тоже задумывался. Просто смотрел много видосов, где так делали и думал, что так правильно.
2) "нет постоянства в коде, то href = "#", то href="#"" - не понял, о чем речь... Я ведь сейчас делаю дизайн. И абсолютно во все ссылки пихаю #. Потому что больше туда нечего пихать, да и не надо сейчас.
3) "не особо по бэму". Спасибо, тут не заметил,что в wrap-nav (навигацию) закинул еще и блоки .block-cards__card
Почему . block-cards и .wrap-block-cards не по бэму?
.block-cards - это блок, в котором лежат эти три блока с карточками, а .wrap-block-cards я сделал, чтобы задать ему отступы от навигации и так далее, так как блок . block-cards не может знать свое расположение (по БЭМ), нельзя задавать размеры и так далее. (естественно, эту всю инфу я тоже нарыл в гугле). Типа, говорят что блоку . block-cards нельзя задать размеры, отступы, тогда создадим вспомогательный блок (обертка) ".wrap-block-cards", которой зададим отступы. Тут я не совсем тогда понял, так как это ведь тоже блок... Получается, что мы создаем блоку .( block-cards), которому не можем задать отступы, обёртку-блок и ему задаем отступы (хотя говорится, что блоку нельзя отступы задавать). Короче, запутано все для меня пока-что.
4) alt = "icon" - опять же. Я делаю сейчас не seo, а дизайн. Но, кстати, я все равно не знаю, как можно описать alt иконки... Это иконка, просто иконка. Сделать описание типа "iconLogo"? ...
5) нет fallback-шрифта. Имеете ввиду, что нет шрифтов, которые сработаю, если не найдется основного? Это понял, ок, буду дописывать.
6) "фон у .logo__link не картинкой". Почему? Здесь я сделал фон для лого картинкой, как и и у .block-cards__card.
По поводу анимации я тоже заметил, не удобно при наведении. Но я стараюсь делать все как в макете, а в макете этот блок вообще на 60px поднят. Я не знаю, нужна ли там анимация вообще, но я добавил.
Верстка без пре-процессоров. На сколько я знаю,то препроцессоры не дают ни каких преимуществ кроме скорости (возможно что-то не доучил, не отрицаю). А скорость мне сейчас не важна.
По поводу "зачем тут .menu nav__menu margin-left: 30em;?"...
В макете блоки logo и menu стоят именно так. То есть, логотип прижат к левому краю, а пункты меню находятся именно там, как на скрине, который я прикрепил выше в комментариях.
Если бы я задал justify-content: space-aroud, то блоки logo и menu разделили расстояние вокруг себя и получилось бы вот это: (скрин). Я делаю строго по макету.
И даже если я тут не прав, то в любом случае, я и попросил оценить верстку, чтобы понять ошибки)
А про фиксированные размеры блоков я не понял... Типа то, что блоку logo задан фиксированный размер в 270px?
VicTHOR, это не вся верстка, а только шапка от макета... header, потом пойдет секция и так далее. Зачем мне изображение body задавать? Для всей страницы...?
header - это шапка, в которой лежит навигация (логотип и меню) и в этом же блоке (header) лежит блок с блоками меню (меню рестарана)
Здесь получается,что вы Логотип к левому краю прижимаете,а меню к правому) А у меня задача сделать, чтобы лого было прижато к левому, а само меню, оно не прижато, а где-то по середине, чуть дальше середины.
В том то и проблема) Можно было бы, конечно, space-beetwen задать и все
Ankhena, то есть, не было бы ошибкой просто задать так, как я сказал? Просто я уже давно практикуюсь, учу верстку и не понял, почему именно так... Может я чего-то не знаю. Ведь чистоту кода это никак не испортит, если сделать так, как я предполагал
