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

[Heidel]

http://web-portfolio1.narod.ru/travel/index.html

[gare4ka]

Вместо

margin-left: 52px;
margin-top: 90px;

Лучше писать

margin: 90px 0 0 52px;

Также неправильно

#header  ul{
	padding-top: 9px;
	padding-left: 355px;
}

Лучше сделать через float и margin.

Из-за таких вещей страдает производительность

#menu ul  li  a
#logo > h1 > a > img

Но в маленьких проектах не критично.

Зачем так нагромождать?

<div id="logo">
	<h1><a href="javascript:void(0)" title="ПромCтекло">
		<img src="images/logo.png" title="Beach Travel" alt="" />
	</a></h1>
</div>

Тут можно обойтись 1-2 элементами

Такие ошибки постоянные

<div id="menu">
        <ul>
	<li><a href="javascript:void(0)">\\ home</a></li>
	<li><a href="javascript:void(0)">\\ new products</a></li>
	<li><a href="javascript:void(0)">\\ specials</a></li>
	<li><a href="javascript:void(0)">\\ products all</a></li>
	<li><a href="javascript:void(0)">\\ reviews</a></li>
	<li><a href="javascript:void(0)">\\ contact us</a></li>
	<li><a href="javascript:void(0)">\\ faq</a></li>
	</ul> 
</div>

ul и так блочный, не имеет смысла его оборачивать в div.

id для JavaScript, лучше использовать классы

Edited November 27, 2012 by gare4ka

[FEL]

Из-за таких вещей страдает производительность

#menu ul  li  a
#logo > h1 > a > img

Но в маленьких проектах не критично.

Я так часто делаю привык...

А почему производительность страдает? Где можно об этом почитать подробнее?

[Viper]

Я так часто делаю привык...

А почему производительность страдает? Где можно об этом почитать подробнее?

_http://www.xiper.net/learn/css/efficient-css/efficient-css-selectors.html

_http://markup.by/2012/01/30/css-selectors-performance/

П.С. Но основная проблема такой стилизации, не в производительности, а в дальнейшей его поддержке, редактировании/ре-дизайне и тд. По этому, советую ознакомиться с методикой верстки независимыми блоками и делать все через классы.

[Николя223]

<h1><a href="javascript:void(0)" title="ПромCтекло">
                <img src="images/logo.png" title="Beach Travel" alt="" />
        </a></h1>

Если не ошибаюсь H1 должен быть один на странице и в нем должен быть заголовок, а не картинка.... это уже для поисковых роботов