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

[ShootingStar]

Здравствуйте, сверстал тренировочный макет из данного курса.

макет

верстка

Жду критику и рекомендации.

[IIAIIIA]

Не заглядывая в код:

- разные отступы между заголовком Portfolio Design и нижеследующим текстом

- и бордер у верхнего блока отличается от макета

- неправильно вырезали логотип, сравните цвет слогана

- Register-login вышел за сетку

- отступ слева у копирайта на макете есть, у вас нет

Могли бы перед показом работы проверить валидатором, заметили бы <meta charset="utf-8/">

Смысла в использовании html5 тэгов header и footer с объявленными классами не вижу.

Если честно, вообще не вижу причин использования этих новых тегов, div с классом прекрасно справляется со своей задачей.

Но если уж решили поставить новые модные теги, то с помощью них и задавайте стили.

Опять же:

<nav>
	<ul>
		<li>
			<a href="#">Home</a>
		</li>
		<li>
			<a href="#">About</a>
		</li>
		<li>
			<a href="#">Blog</a>
		</li>
		<li>
			<a href="#">Contact</a>
		</li>
	</ul>
</nav>

Усложнили DOM-дерево на один контейнер, а зачем?

Чуть больше любви и внимания к мелочам и деталям - и все у вас будет хорошо.

Edited March 29, 2012 by IIAIIIA

[ShootingStar]

Пофиксил баги.

Бордер там с градиентом, вижу пока 2 способа:

1) это вырезать изображения бордера для всей страницы, для врапера(900px) и использовать border-image.

2) Использовать что-то такое: background-image: -moz-linear-gradient(center bottom, rgb(14,173,173) 33%, rgb(0,255,255) 67% );