Оцените, пожалуйста, верстку макета

[gordey]

Оцените, пожалуйста, верстку макета. Сама верстка. IE ниже 9 не учитывал. По поводу hover на ссылках тоже не замарачивался. Где что сделал неправильно?

Edited October 10, 2012 by gordey

[sammasati]

1. <nav class="shopping">...<nav class="support-head"> - это не навигация, вместо нав лучше использовать списки.

2. <p class="head-tel"> - это не параграф. Тут лучше использовать спан или див.

3. <p class="login">, <p class="login"> - то же самое. Тут должны быть дивы, а не параграфы.

4. <img src="images/phone-img.jpg" width="940" height="74" ...> - размеры лучше задать в стилях, тем более, что у родителя есть класс.

5. Опять же использование nav не там, где нужно. Он применяеться только для основной навигации сайта, во всех остальных случаях лучше использовать ul.

6. Опять же не уместное использование параграфов. Для не семантичных блоков лучше подходит "безликий" div.

Ну вот как-то так. Может кто-нибудь ещё добавит.

[gordey]

1. <nav class="shopping">...<nav class="support-head"> - это не навигация, вместо нав лучше использовать списки.

2. <p class="head-tel"> - это не параграф. Тут лучше использовать спан или див.

3. <p class="login">, <p class="login"> - то же самое. Тут должны быть дивы, а не параграфы.

4. <img src="images/phone-img.jpg" width="940" height="74" ...> - размеры лучше задать в стилях, тем более, что у родителя есть класс.

5. Опять же использование nav не там, где нужно. Он применяеться только для основной навигации сайта, во всех остальных случаях лучше использовать ul.

6. Опять же не уместное использование параграфов. Для не семантичных блоков лучше подходит "безликий" div.

Ну вот как-то так. Может кто-нибудь ещё добавит.

Спасибо за комментарий. С семантикой пока проблема

[talgautb]

у картинки ширина и высота обязательные параметры. при отсутствии могут быть проблемы с ie

[gordey]

у картинки ширина и высота обязательные параметры. при отсутствии могут быть проблемы с ie

Согласен. Этим и руководствовался.