Оцените верстку

[Kisa1993]

Одна из первых моих работ.

Интересует всё - мнения по html-разметке, качество кода, читабельность и тд и тп.

Сам сайт

PSD'шка

Критика приветствуется

[cyklop77]

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

Edited July 12, 2012 by cyklop77

[hedgehog]

Беглый взгляд:

1. При коротком доктайпе html кодировка объявляется просто <meta charset="кодировка" /> . И почему cp1251? Есть же utf-8.
   
2. Что за глупости с айдишниками и собственными бекграундами для каждого пункта меню?
   
3. По каким соображением практически везде используются ID?
   
4. Что за глупости с кнопкой "SIGN UP" в виде img и JS для смены src по ховеру? Можно же просто сделать ссылку с текстом.
   
5. Та же ерунда с лого. Можно менять по ховеру бекграунд, зачем там JS?
   
6. div#header - это не хедер. Хедером у вас является див с непонятным классом roof100. header100 из той же оперы.
   
7. Зачем дублировать фон у header, если у его родителя - такой же фон?
   
8. width:100% для блочных элементов не нужно, за редким исключением.
   
9. <div class="space"/> - что это? ) margin-top/padding-top уже не работают?
   
10. В html5 для группировки заголовков можно использовать hgroup (div.block>h3+h4)
    
11. <div class="space_for_blocks"/> ?!
    
12. Дублируется атрибут class: <img class="four_img" class="four_img" src="img/man.png"
    
13. Неправильно объявлены комментарии. Комментарий закрывается БЕЗ восклицательного знака: <!-- комментарий -->
    
14. В футере почему-то <p> обернут в <strong>
    

[Kisa1993]

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

Получится не очень хорошо . Но изначально в макете планировалось 5 пунктов).

Беглый взгляд:

1 Возьму на заметку, к utf-8 у меня какое-то отторжение, часто с ней мучилась, ну и есть крупные сайты которые на cp-1251.

2 У всех пунктов меню разный размер текста => и бэкграунд должен быть разным (на форуме писали что не такое уж и горе использовать и class и id вместе)

3 Не могу отучиться (хотя на форуме гуру писал что id для java а class для css)

4,5 Точно, признаю глупость)

6 header точно, не там использовала. А вот почему header100 непонятен?(

7 тупанула

8 не понимаю

9 упс

10 html 5 вообще не знаю и боюсь на него переходить (не знаю с чего начать)

11 упс

12 ощепятка

13 приму к сведению

14 а нужно только текст без тега(

Edited July 13, 2012 by Kisa1993

[hedgehog]

1 Возьму на заметку, к utf-8 у меня какое-то отторжение, часто с ней мучилась, ну и есть крупные сайты которые на cp-1251.

По-моему, утф8 уже стала стандартом де-факто. Я не агитирую, но это очень удобная кодировка.

2 У всех пунктов меню разный размер текста => и бэкграунд должен быть разным (на форуме писали что не такое уж и горе использовать и class и id вместе)

В вашем случае можно использовать паддинги и border-radius. Если понадобятся картинки - можно воспользоваться "технологией раздвижных дверей": длинный бекграунд на li, и короткий (с другой стороны) - на <a>. В результате менюшка как бы раздвигается при увеличении текста

14 а нужно только текст без тега(

По поводу очистки флоатов: <div class="clearing"> и тому подобное - это уже история. Сейчас используют overflow:hidden родителю или clear:both в :after, погуглите на эту тему. На css-live.ru вывели новый экзотический способ, но я им не пользовался