http://websterjoy.tk/ Впервые сверстал на html5, как скажете? правильно ли все, семантично ли? Какие замечания вы могли бы сказать по поводу верстки? CSS разумеется валидацию не проходит, т.к вендорные префиксы и т.д.
Ответ
Моё мнение, которое не претендует на истину. По поводу семантичности: Вы слишком увлеклись тэгом section, в большинстве случаев можно было спокойно использовать div Для тэга header главного для страницы, рекомендуется использовать класс, например .page-header и уже от него настраивать разметку его содержимого. По поводу самой вёрстки: Не используйте классы типа .gray, .green, .black2 и т.д. Название класса должно быть осмысленным. То же самое касается и классов типа fs12, fs14, fs16. Их можно переименовать например в font-size-small, font-size-medium и т.д. Т.к. вы используете на сайте чаще размер шрифта в 12px, то можно, например, такой размер задать по умолчанию для body и тогда код станет чище. Забудьте про
, используйте .clearfix Не используйте h3 для вывода цены и h1 для количества упаковок, который у вас по счёту вторым получается, что недопустимо. Сеошники могут и побить :) Маленькие картинки объедините в sprite ( спрайты ) Нет смысла использовать float с display:inline-block т.к. float делает элемент блочным (т.е. display:block). Определитесь с поддержкой браузеров, а то для border-radius вы поддерживаете аж FF 3.6 c префиксом -moz, хотя не делаете того же для box-shadow ( класс .show_current ) я надеюсь вы используете @import только при разработке, а потом всё равно стили склеиваете. Если что-то ещё замечу, обновлю пост.
Комментариев нет:
Отправить комментарий