Оцените, пожалуйста, мою верстку [закрыт]

#css3 #html #вёрстка #css

        
             
                
                    
                        
                            Закрыт. На этот вопрос невозможно дать объективный ответ.
Ответы на него в данный момент не принимаются.
                            
                        
                    
                
                            
                                
                
                        
                            
                        
                    
                        
                            Хотите улучшить этот вопрос? Update the question so it
can be answered with facts and citations by editing this post.
                        
                        Закрыт 4 года назад.
                                                                                
           
                
        
http://websterjoy.tk/
Впервые сверстал на html5, как скажете? правильно ли все, семантично ли? Какие замечания
вы могли бы сказать по поводу верстки?
CSS разумеется валидацию не проходит, т.к вендорные префиксы и т.д.    

Ответы

Ответ 1

Моё мнение, которое не претендует на истину.
По поводу семантичности:

Вы слишком увлеклись тэгом 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 только при разработке, а потом всё равно стили склеиваете.

Если что-то ещё замечу, обновлю пост.


Ответ 2

This document was successfully checked as HTML5! Что тут еще скажешь) +1