Покритикуйте вёрстку пожалуйста

Статус
Закрыто для дальнейших ответов.

prozaik81

Active Member
Регистрация
29.05.2011
Сообщения
33
оцените пожалуйста качество вёрстки страницы и укажите на минусы/косяки.
макет здесь
страница здесь
переполненная версия здесь
psd здесь
 

tigra60

Спасатель
Регистрация
10.12.2009
Сообщения
1 990
Код у вас валидный, и это хорошо.

А вот про семантику вы забыли, и это очень плохо!
Неужто никак нельзя обойтись без таблицы в футере?

Очень сомнительно выглядит список <ul class="articles hfeed">. Элементы списка явно перегружены.

Много у вас избыточности, например:
Код:
<form action="/" method="post" class="searchform">
	<div>
...
	</div>
</form>
Зачем здесь еще блок внутри формы?


Код:
<div class="more"><a href="#55">READ MORE</a></div>
Почему бы не сделать просто
Код:
<a href="#55" class="more">READ MORE</a>
добавив классу more стиль display:block;?

Из той же серии:
Код:
<h2><span class="white_letters">TWEET TWEET</span></h2>
Код:
<a href="#85"><span class="par3">Follow Us On Twitter!</span></a>
Не добавляет красоты и навязчивый javascript:
Код:
<body onload="styles();">
Сомнительны также заголовки в коде заглавными буквами. Это (вид заголовка), все-таки, прерогатива стилей.
 

prozaik81

Active Member
Регистрация
29.05.2011
Сообщения
33
Код у вас валидный, и это хорошо.

А вот про семантику вы забыли, и это очень плохо!
Неужто никак нельзя обойтись без таблицы в футере?

Очень сомнительно выглядит список <ul class="articles hfeed">. Элементы списка явно перегружены.

Много у вас избыточности, например:
Код:
<form action="/" method="post" class="searchform">
	<div>
...
	</div>
</form>
Зачем здесь еще блок внутри формы?


Код:
<div class="more"><a href="#55">READ MORE</a></div>
Почему бы не сделать просто
Код:
<a href="#55" class="more">READ MORE</a>
добавив классу more стиль display:block;?

Из той же серии:
Код:
<h2><span class="white_letters">TWEET TWEET</span></h2>
Код:
<a href="#85"><span class="par3">Follow Us On Twitter!</span></a>
Не добавляет красоты и навязчивый javascript:
Код:
<body onload="styles();">
Сомнительны также заголовки в коде заглавными буквами. Это (вид заголовка), все-таки, прерогатива стилей.
спасибо за подробный ответ.

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

без лишнего div в блоке формы к сожалению не обойтись так как валидатор ругается на его отсутствие(при типе документа strict).

а вот это
Сомнительны также заголовки в коде заглавными буквами. Это (вид заголовка), все-таки, прерогатива стилей.
я не понял. поясните пожалуйста. то есть можно например написать заголовок строчными буквами(прописная только первая), а с помощью какого то стилевого правила изменить регистр букв на заглавный?? правильно я понимаю?
 

BaNru

Пацифизжу
Команда форума
Регистрация
13.11.2010
Сообщения
4 153
CSS
Код:
text-transform: uppercase;
 
Статус
Закрыто для дальнейших ответов.
Верх Низ