Оцените мою ужасную вёрстку №2

[DivMan]

Дальше не стал верстать, так как, код стал очень ужасным и уже невозможно продолжать.
http://allnet.16mb.com/
 
макет jpg

 

макет PSD http://yadi.sk/d/vQPNtXErNqtBz

[AlexBes]

1. Ссылки в меню косят. Лучше задавать не margin для li, а padding для a. Чтобы растянуть меню по ширине блока можно использовать этот метод.

2. В меню нет градиента, не указан класс hover.

3. Заголовки материалов должны быть ссылками, текст "добавить комментарий" - тоже.

4. Для maincontent задана высота - так не делают. Wrapper'у тоже высоту не задают. Ее нужно задавать там, где точно знаешь размер блока и он не меняется.

5. Блоки, которые находятся сбоку, оборачивают отдельным блоком. И уже ему ставят обтекание float. Весь правый блок нужно переделывать.

6. Если используешь теги html5, то делай это везде, где они актуальны, например, зачем делать div.article, если есть тег article

Это то, что сразу бросилось в глаза.

[akorshun]

Дальше не стал верстать, так как, код стал очень ужасным и уже невозможно продолжать.

 

не нравится код верстайте дивами

[DivMan]

 

Дальше не стал верстать, так как, код стал очень ужасным и уже невозможно продолжать.

 

не нравится код верстайте дивами

 

а я чем верстаю? и запятые надо ставить.

Edited May 2, 2014 by DivMan

[akorshun]

прошу прощения, не тот макет открыл, почему вы считаете код ужасным? как по мне то он нормальный, форматирование соблюдаете и все понятно

[DivMan]

надо переверстать. Я шапку вырезал на всю ширину, может мне целиком фон сделать этой зелёной картинкой, не вырезая шапки? 

Если присмотреться на шапку, то внизу видны линии 

Edited May 2, 2014 by DivMan

[Shenia132]

Сайт не влазит в 1024px, появляется скролл нижний а это плохо

в wrapper ставь min-width:1000px; и max-width:1280px;

[DivMan]

Сайт не влазит в 1024px, появляется скролл нижний а это плохо

в wrapper ставь min-width:1000px; и max-width:1280px;

это всё из-за шапки 

Edited May 2, 2014 by DivMan

[Shenia132]

тогда шапку уменьшай