Можете оценить верстку?

Question

[david94ez]

Здравствуйте. Оцените пожалуйста верстку. Хотелось бы получить какие то замечания и наставления. Занимаюсь этим 3й месяц. Скажу сразу макет содержал много странных не логических вещей и не было адаптива так что доделывал сам как мог. Использовал в верстке бэм,pug,scss,js. и собирал gulp + немного webpack.
Главная
Каталог
Корзина
Страница товара

Comments

[Ankhena]

Начните отсюда https://validator.w3.org/nu/

[godsplan]

сколько времени ушло?

[david94ez]

Ankhena, Там нету ошибок. По крайней мере у меня не показывает.

[david94ez]

godsplane, дня 4

[Ankhena]

david94ez, круть! Либо быстро либо первый раз недогрузилась страница.

Следующим ходом проверяйте сайт на всех размерах и во всех актуальных браузерах.
Например,

Наверное, переключалки должны были быть кружочками, а не овальчиками.
А текст видимым.

Но в общем, хорошо.
Можно поправить семантику, заменить бессмысленные картинки на фоны и/или псевдо, перестать закрывать одиночные теги и еще какие-нибудь мелочи.

[david94ez]

Ankhena, Благодарю за замечания. Без мобильного макета трудно было некоторые вещи куда то девать которые не влезали. Опыта в создании адаптивов без макета у меня не много) По поводу закрытия одиночных тегов. Я верстал с pug(где вообще ничего не закрывал по сути)) и gulp собирал из него уже html и я не знаю можно ли там задать что бы он конкретно их не закрывал при сборке.

[Ankhena]

david94ez, Да, такая ситуация, когда макет только один, распространена. И тем выше ценятся верстальщики, которые умеют сделать хорошо и без макета. На мой взгляд, у вас получилось достаточно гармонично. Можно где-то работать с отступами и шрифтами, но это во многом вопросы вкуса.

Где-то есть недочеты типа как указала постом выше.
Дальше работа над семантикой, доступностью, читаемостью классов, js и подобные усовершенствования.

Удачи в развитии.

[Froggyweb]

и не было адаптива так что доделывал сам как мог

странно вроде Женя верстал с адаптивом. да и классы у тебя так же называются.

Answer 1

[Alex]

Поработайте над доступностью

Answer 2

[Ranwise]

не везде есть ховеры
в выпадающем меню поиска тоже нету ховеров и не понятно что их можно выделять - но это наверно проблема макета)

Answer 3

[ilya granovskyy]

не открывается твоя верстка

Answer 4

[Arsenii]

Также ссылки выдают 404.