Оцените мою первую верстку

[Kublyakov]

Здравствуйте, оцените пожалуйста качество кода. Наименование классов, правильно или нет я применял различные приемы верстки. Это мой первый макет, сверстал часов за 12 http://kublyakov.ru/ur/

[CroaToa]

Делайте что бы кнопки реагировали хотя бы hover а лучше еще и active http://pressed.ru

.slide-title {position:absolute;} незачем

.contacts {position:absolute;} незачем

.item span зачем спан?

[xzarxzes]

Здравствуйте, оцените пожалуйста качество кода. Наименование классов, правильно или нет я применял различные приемы верстки. Это мой первый макет, сверстал часов за 12 http://kublyakov.ru/ur/

У вас нет заголовка h1, также страница не проходит валидацию. Не рекомендуется использовать html5 shiv из репозитория который указан у вас http://zoompf.com/blog/2012/05/html5shiv-and-serving-content-from-code-repositories. Элемент img следует использовать для картинок которые несут смысловую нагрузку и относятся к контенту страницы, я это к тому что это плохая идея создавать для разделителя, и прочих фоновых картинок отдельный img элемент в html, для этого существует background-image. Для элемента input есть атрибут type="tel", так что пожалуй следует использовать его вместо type="text" который указан у вас для inputa'a с id - phone в хэдере и input'ы следует располагать внутри элементов form.

Edited March 21, 2014 by xzarxzes

[ModMaker]

 -moz-border-radius: 5px; -webkit-border-radius: 5px;-moz-box-shadow: 1px 1px 2px 0px rgba(239, 187, 29, 0.5);

Зачем?

Комментарии тоже мне кажутся лишними.

 

.header-slide:after {display: table;}

Для чего тут таблица?

Инпутам хорошо бы убрать оутлайн. Так как у отлайнов радиусов углов отличается от вашего.

<header class="header">

А для чего тут указан класс? Я понимаю если бы вы указали допусти класс clear, но в рамках этого шаблона разве есть смысл указывать хедеру класс хедер?

 

<div class="callback">                        <input type="text" id="name" name="name" placeholder="Ваше имя">                        <input type="text" id="phone" name="phone" placeholder="Ваш телефон">                        <div class="submit">                            <a href="#">Заказать звонок</a>                        </div>                    </div>

 Тут должен быть не див колбак, а форм колбак, и не див сабмит, а инпут сабмит, <a>вообще не нужен.

“Общество защиты прав потребителей»

 

ство защиты прав потребителей»

“Общестоо

“Общество“Общество защиты прав потребителей»защиты прав потребителей»

Edited March 21, 2014 by ModMaker

[CroaToa]

Для чего тут таблица?

там вроде как clearfix пробовалось сделать) 

[xzarxzes]

 

Для чего тут таблица?

там вроде как clearfix пробовалось сделать) 

 

Это clearfix и есть, единственное только display: table актуален только для псевдоэлемента :before, для предотвращения вертикальных коллапсов http://nicolasgallagher.com/micro-clearfix-hack/

<header class="header">

А для чего тут указан класс? Я понимаю если бы вы указали допусти класс clear, но в рамках этого шаблона разве есть смысл указывать хедеру класс хедер?

»

 

Класс хэдеру нужен, так как может добавиться еще один хэдер в дальнейшем (только лучше  указать "main-header" или "site-header"). Здесь полно других классов которые совершенно не нужны и без которых можно было бы и обойтись.