оцените вёрстку

[DivMan]

http://allnet.16mb.com/smart/

потом адаптации сделаю 

Edited June 2, 2014 by DivMan

[swandev]

1.  .menu li

border-left:1px solid #cccccc;border-right:1px solid #f7f7f7;border-bottom:1px solid #cccccc;border-top:1px solid #cccccc;

Сократить до

border: 1px solid #cccccc;border-right-color: #f7f7f7;

2. Лишняя вложенность

http://take.ms/0du7g

 

И почему 'border'? Впрочем помимо этого, много названий классов не соответствуют действительности.

3. Вот это http://take.ms/ADrZD  можно заменить на h2 

4. Вот эти http://take.ms/wBIer  заголовки оформлены тегом h1, лучше заменить например на h3

5. Разобраться со флоатами, например тут http://take.ms/VllF9 если добавить еще пунктов во вторую колонку, то они перейдут в первую колонку

5. Нужно поставить какой нибудь ховер эффект

     тут http://take.ms/6VHEx тут http://take.ms/x4M1F тут http://take.ms/N7S6g

Edited June 2, 2014 by ALEEX SWN

[MichaelD]

1. Что за "karkas". Лучше уже wrap, wrapper или container.

2. Я так думаю, что логотип - это картинка вместе с текстом, целесообразно это все засунуть в ссылку. Или сделать логотип полностью картинкой.

3. Чем ссылка в menu с классом active отличается от обычной ссылки?

4. border-top-left-radius:10px; border-bottom-left-radius:10px; -> border-radius:10px 0 0 10px; Такое-же замечание и к остальным свойствам.

5. Градиент картинкой - это нормально, но достаточно 1px вместо 11px;

6. *{margin:0; padding:0} в принципе - это нормально для тестирования, но не для готовой верстки, надо конкретно указывать чему вы хотите сбросить отступы. Селектор "*" нагружает страницу. Тут уже было куча тем по этому поводу

7.<div class="border"><div class="border"> ...  </div></div>  Зачем? Конечно возможно просто невнимательность.

Названия классов вообще не логичны. Как я догадаюсь, что в блоке с классом "border" у меня находится основной контент.

 

И еще всего всего много

[DivMan]

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