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

[artemkos88]

Здравствуйте. Прошу оценить мою первую верстку, в частности код (хотя бы общее впечатление). Возможно, какие моменты в верстке ошибочны или сделаны не верно - просьба указать на них.

 

Сайт - http://web-kos.ru/

PSD - http://freepsd.ws/maket-mobile-apps-v-formate-psd/

 

Буду рад комментариям и критике!

[stan07]

Здравствуйте. Прошу оценить мою первую верстку, в частности код (хотя бы общее впечатление). Возможно, какие моменты в верстке ошибочны или сделаны не верно - просьба указать на них.

 

Сайт - http://web-kos.ru/

PSD - http://freepsd.ws/maket-mobile-apps-v-formate-psd/

 

Буду рад комментариям и критике!

Логотип должен быть ссылкой на главную страницу.

По коду, тут лишний див

<header>	<div class="container">		<div class="top">			<h1>MobileApp</h1>			<p>Your Mobile App Slogan</p>		</div>		<nav>			<ul>				<li class="active"><a href="#">Home</a></li>				<li><a href="#">Features</a></li>				<li><a href="#">Support</a></li>				<li><a href="#">Download</a></li>				<li><a href="#">Contact</a></li>			</ul>		</nav>	</div></header>

Ведь можно написать вот так:

<header class="container">		<div class="top">			<h1>MobileApp</h1>			<p>Your Mobile App Slogan</p>		</div>		<nav>			<ul>				<li class="active"><a href="#">Home</a></li>				<li><a href="#">Features</a></li>				<li><a href="#">Support</a></li>				<li><a href="#">Download</a></li>				<li><a href="#">Contact</a></li>			</ul>		</nav></header>

Edited August 23, 2015 by stan07

[IvanRu08]

По коду, тут лишний див

Нет здесь лишнего дива, контейнер имеет совершенно другую ширину и центрируется, соответственно по другому. 

По верстке:

1. Средняя часть не тянется вместе с контентом. Это не гуд. Вот здесь уберите высоту: 

   .content {    float: left;    padding: 0;    margin: 0 auto;    width: 1200px;    height: 344px;    background: url(images8/highlight.png) no-repeat;}

2. При масштабировании составляющие сайта "гуляют" сами по себе. Не понятно, зачем для всего было прописано float:left. Имитируете таблицу?

Edited August 23, 2015 by IvanRu08

[stan07]

 

По коду, тут лишний див

Нет здесь лишнего дива, контейнером имеет совершенно другую ширину и центрируется, соответственно по другому. 

а ну да, забыл в цсс заглянуть )

[artemkos88]

Средняя часть не тянется вместе с контентом. Это не гуд. Вот здесь уберите высоту:

Высоту сделал auto. Сейчас средняя часть растягивается с контентом, но при этом растягивается и картинка смартфона (при добавлении нового текста - вот как пример http://prntscr.com/8876h3).Я так понимаю что ее (картинку смартфона) нужно было добавить отдельно от всего фона, верно?

При масштабировании составляющие сайта "гуляют" сами по себе. Не понятно, зачем для всего было прописано float:left. Имитируете таблицу?

Float:left убрал - сейчас, вроде бы при масштабировании все в норме.

Edited August 24, 2015 by artemkos88