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

[stan07]

Добрый день!

Оцените резиновую верстку, укажите на ошибки кода, пожалуйста

svet

[kompolom]

Мне нравится. Валидная верстка. приятно смотреть на код.

<section class="cat clearfix">

здесь бы я поставил <nav>

А вообще, что касается использования семантических html5 тегов, довольно много различный мнений ходит особенно относительно article и section.

обратил внимание, что у вас основной блок лежит в aside. По мнению этого сайта

Тэг aside используется для выделения элементов, которые не являются частью содержимого, но косвенно с ним связаны.

Ну и каталог ,как мне кажется, удобней верстать на inline-block чем на float элементах.

Это позволяет избавляться от костылей типа

.product:nth-child(3n) {
margin-right: 0;
}

Где-то на хабре попадалась статья где подробно расписан метод.

Edited October 2, 2013 by kompolom

[stan07]

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

[klierik]

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

2. научись писать без комментариев. человеко-понятное именование классов заменяет комментарий.

3. не .conteiner а .container

4. это: <p class="tel"> список телефонов а не абзац. тут или список, или див, но никак не Р

5. <aside class="ten"></aside> -- эту тень через CSS делать следует.

ошибок слишком много. посмотри по разделу темы в которых я отвечал. 95% подходит и в твоем случае.

[advokatua]

Неплохо бы видеть и макет изначальный.

Мой монитор 1366х768:

1) Пункт "доставка и оплата" липнет впритык к краям своей кнопки, т.е. отступов вообще нет. На меньшем разрешении перестраивается в 2 строчки, а при большем все ок.

2) Телефоны справа и форма входа забита в угол как узник Гуантанамо в своей камере, которого пришли забирать для очередных пыток.

3) Никакого оформления в html быть не должно.

<img class="head" src="images/header-bg.jpg" alt="картинка">

И еще куча подобных вставок не там где надо.

4) CSS: лишняя строка

.logo a {
118: float: left;
119: display: block; /*Лишняя строка*/

5) Что это за жесть? )

<div class="bottom_header clearfix">
			 <h1 class="logo">
<a href="index.html" class="logo_link" title="На главную"><span>Logo</span></a>
</h1>

6) Никаких руссизмов в названиях классов

class="lustri"

7) CSS - 251: .conteiner

пишется через а - container.

8) Неверная разметка. "<article>" в подобном случае не применяется, здесь скорее подойдет figure и figcaption. Если не уверены, что точно знаете назначение новых элементов html5 лучше использовать все по старому.

<article class="product">
<a href="#"><img src="images/product-img.jpg" alt="картинка"></a>
    <h5>702-L01CH3184 Люстра</h5>
    <h6>Размер ДхШхВ: 930 х 930 х 770</h6>
    <p class="price">49 980 руб</p>
</article>

и т.д.

PS. Тренируйтесь, учитесь, овладевайте. )

Edited October 12, 2013 by advokatua

[webdrug]

1920px съезжает блок авторизации