Оцените верстку (2Breed)

[Floating]

Добрый день! В последнее время, решил заняться веб разработкой и версткой. Раньше был небольшой опыт верстки, php программирования. Нашел в интернете бесплатный симпатичный шаблон. Прошу оценить работу по всей строгости. Код HTML и CSS валидный, проверял верстку в браузерах последних версий FireFox, Opera, Chrome, Safari, IE 9 (так же в разных режимах IE 8, 7).

Верстка: ссылка

Шаблон (jpg): ссылка

Если кому то понадобится шаблон в PSD, предоставлю ссылку. Жду критики и комментариев.

[Nekromancer]

В общем хорошо

[Comex]

Для старта не плохо) Молодец) Выложи шаблон. Для новичков самое то. Можно было заменить картинки текстом в некоторых случаях. Текст всегда лучше картинок)

[ctpz]

Справа есть отступ небольшой.

[ne4to]

Замечательно, чистенько, красиво. На будущее совет - побольше комментариев и отступы строк.

Edited August 15, 2011 by ne4to

[kidrock]

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

[troll]

Неплохо бы использовать заголовки разных уровней вместо того, чтобы вставлять все подряд в h1.

Edited August 15, 2011 by troll

[Nekromancer]

побольше комментариев

Зачем это?

[Floating]

В общем хорошо

Спасибо, будет еще лучше

Для старта не плохо) Молодец) Выложи шаблон. Для новичков самое то. Можно было заменить картинки текстом в некоторых случаях. Текст всегда лучше картинок)

Вот шаблон: psd. Про текст знаю, спасибо.

Справа есть отступ небольшой.

О каком отступе речь?

Замечательно, чистенько, красиво. На будущее совет - побольше комментариев и отступы строк.

Спасибо, учту

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

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

Неплохо бы использовать заголовки разных уровней вместо того, чтобы вставлять все подряд в h1.

В следующих работах так и сделаю