Ошибки бегиннера. Оцените верстку?

Question

[msnthrp]

Здравствуйте.

Укажите, пожалуйста, на ошибки.

Два моих первых сайта:
Простой статичный — https://github.com/sashami/staticLP
Адаптив + респонсив — https://github.com/sashami/adaptiveLP

Сейчас уже стараюсь юзать БЭМ и препроцессоры, чтобы не нагромождать код.

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

Заранее спасибо.

Comments

[orbit070]

Верстку не оценю, а вот "Простой статичный" сайт вообще визуальное чудо какое-то, это самый красивый лендос что я видел) понимаю что он простой, но эта графика, минималистичность и цвета просто что-то с чем-то)

[freshik312]

Простой статичный - глянул код, нормально оформил, только я бы кнопку не делал в top-menu.

[Владимир]

Например не в том порядке

<link rel="stylesheet" type="text/css" href="./css/style.css">
    <link rel="stylesheet" type="text/css" href="./css/reset.css">

.wrap-one two thre four ..... достаточно просто wrap везде писать.
а не дублировать одни и теже стили.
.wrap-three {
width: 980px;
margin: 0 auto;
}
.wrap-million {
width: 980px;
margin: 0 auto;
}

[Дмитрий]

orbit070, а можно скрин, с тела просто

[orbit070]

Дмитрий,

spoiler

[msnthrp]

Владимир, да, всегда reset первым. ошибка. спасибо.

По поводу wrap. Будет ли правильным такой синтаксис, если у нескольких врапперов есть дополнительные свойства?

wrap-one, wrap-two, wrap-four, wrap-five, wrap-six {
width: 980px;
margin: 0 auto;
}

.wrap-one {
padding: 40px;
}

.wrap-six {
position: relative;
}

СПАСИБО

[Владимир]

wrtr,

<section class="section name">
<div class"wrap"></div> = просто обертка которая обычно определяет ширину контента.
</section>

отступы секциям задаются, их обычно одинаковыми прорисовывают между собой.
.section {padding: 80px 0;} верх+низ =80 . право-лево =0.

Если в какойто секции отступ отличается просто берешь 2ой класс как ты её назвал
.name{padding: X X}