Оцените, пожалуйста, первую работу!

[SuperJerk]

Ребят, оцените саму верстку и код на наличие рациональности.

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

Что с этим делать - без малейшего понятия, прошу помощи у вас

http://www.superjerkstuff.p.ht/

[pashkje]

DEAD SPACER, зачем li:hover {color: white;} ? Не читабельно, оставьте текст черным.

ul.menu_items сделайте шире, хотя бы до 500px. или menu_items li {float: left;

margin: 0 20px 0 0; т.к. последний пункт меню съезжает вниз. И я бы .menu_items a.activeitem как то бы по другому выделил бы,т.к., когда срабатывает hover непонятно какая активная ссылка, а какая просто загоревшаяся. А так дизайн мне нравитсо

Edited December 7, 2012 by pashkje

[Radiocity]

Что то мне подсказывает, что подписи на картинках должны быть сверстаны

<!--[if lt IE 8]>

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

<script src="html5.js"></script>

<![endif]-->

А в горизонтальном меню в макете должен быть пункт под Dead Spacer?

Edited December 7, 2012 by Radiocity

[SuperJerk]

Что то мне подсказывает, что подписи на картинках должны быть сверстаны

Совсем забыл про подписи на картинках исправлю.

А в горизонтальном меню в макете должен быть пункт под Dead Spacer?

Не совсем понимаю, о чем вы, но верстал строго по макету)

ul.menu_items сделайте шире, хотя бы до 500px. или menu_items li {float: left;

margin: 0 20px 0 0; т.к. последний пункт меню съезжает вниз.

Спасибо! Странно, что в FF17 и в опере12 ничего не съезжает...Исправлю.

Есть ещё замечания в плане кода?

Edited December 7, 2012 by SuperJerk

[Bobikys]

а сколько времени ушло на это творение ?

[SuperJerk]

Нарезка шаблона + верстка: 2-3 часа. Дизайн не мой, я ответственен только за верстку

Edited December 7, 2012 by SuperJerk

[npofopr]

В центре это же слайдер, у которого картинки листают кнопки справа, а не просто картинка на фоне.

А что будет, если в любой части верстки будет текста в 2 строки?

[SuperJerk]

В центре это же слайдер, у которого картинки листают кнопки справа, а не просто картинка на фоне.

А что будет, если в любой части верстки будет текста в 2 строки?

О слайдере даже ещё не думал, только начал изучать javascript. Но его можно же ещё на CSS3 сделать? Исправлю в ближайшее время.

А что будет, если в любой части верстки будет текста в 2 строки?

То есть? В колонках же идет текста в несколько строк. Я вас не понял, можно поподробней