Здравствуйте. Оцените пожалуйста верстку. Хотелось бы получить какие то замечания и наставления. Занимаюсь этим 3й месяц. Скажу сразу макет содержал много странных не логических вещей и не было адаптива так что доделывал сам как мог. Использовал в верстке бэм,pug,scss,js. и собирал gulp + немного webpack. Главная Каталог Корзина Страница товара
david94ez, круть! Либо быстро либо первый раз недогрузилась страница.
Следующим ходом проверяйте сайт на всех размерах и во всех актуальных браузерах.
Например,
Наверное, переключалки должны были быть кружочками, а не овальчиками.
А текст видимым.
Но в общем, хорошо.
Можно поправить семантику, заменить бессмысленные картинки на фоны и/или псевдо, перестать закрывать одиночные теги и еще какие-нибудь мелочи.
Ankhena, Благодарю за замечания. Без мобильного макета трудно было некоторые вещи куда то девать которые не влезали. Опыта в создании адаптивов без макета у меня не много) По поводу закрытия одиночных тегов. Я верстал с pug(где вообще ничего не закрывал по сути)) и gulp собирал из него уже html и я не знаю можно ли там задать что бы он конкретно их не закрывал при сборке.
david94ez, Да, такая ситуация, когда макет только один, распространена. И тем выше ценятся верстальщики, которые умеют сделать хорошо и без макета. На мой взгляд, у вас получилось достаточно гармонично. Можно где-то работать с отступами и шрифтами, но это во многом вопросы вкуса.
Где-то есть недочеты типа как указала постом выше.
Дальше работа над семантикой, доступностью, читаемостью классов, js и подобные усовершенствования.
Средний
