Жду жёсткой (но обоснованной) критики вёрстки!

[ya_verstaka]

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

ссылка на psd (страница где я скачал макет). 

ссылка на готовую верстку: http://aditii.bl.ee

ссылка на файлы: http://yadi.sk/d/Q8SKGConCpGEK

 

Заранее Спасибо тем ребятам, которые не прошли мимо и высказали свое мнение по поводу данной работы. Удачи Всем!!!

[Galaxy]

особо не вдавался , но что не понравилось это - css код в htmle, дальше - href в div'e , можно было юзнуть меньше div class="center" аля wrapper, если ставите коментарии то ставьте их и в начале и в конце блока чтобы было понятнее

[ya_verstaka]

href в div'e

Согласен, недосмотрел, когда делал редизайн карусели. По поводу всего остального -  учту . Спасибо за замечания!

[cyklop77]

вот это очень плохо

*{    padding:0;    margin:0;}

[volkod]

посоветовал бы использовать HTML 5 теги если верстаете в нем

[ya_verstaka]

 

вот это очень плохо

*{    padding:0;    margin:0;}

А как делать правильнее? 

[Galaxy]

Правильнее использовать сброс стилей. информации про это в интернете много.

[Struggle]

Пустые дивы, начиная от header,  которым не назначан css, спрашивается зачем

<div class="center">				<ul>					<li><a href="#">Home</a></li>					<li><a href="#">Sale</a></li>					<li><a href="#">Handbags</a></li>					<li><a href="#">Wallets</a></li>					<li><a href="#">Accessories</a></li>					<li><a href="#">Ments store</a></li>					<li><a href="#">Shoes</a></li>					<li><a href="#">Vitage</a></li>					<li><a href="#">Services</a></li>					<li><a href="#">Contact us</a></li>				</ul>			</div>

Можно сразу ul class="center"

 

http://validator.w3.org/check?uri=http%3A%2F%2Faditii.bl.ee%2F&charset=%28detect+automatically%29&doctype=Inline&group=0

 

p.s понравилось решение с подвалом и меню, буду юзать, спасибо

Edited November 18, 2013 by Struggle

[ya_verstaka]

Можно сразу ul class="center"

Действительно, я как-то сразу не подумал. Спасибо!!! В будущем буду стараться следить за целесообразностью использования тех или иных блоков 

Edited November 18, 2013 by ya_verstaka

[swandev]

Правильнее использовать сброс стилей. информации про это в интернете много.

Есть те, кто с этим не согласится)

[Ludwig Voltman]

 

Правильнее использовать сброс стилей. информации про это в интернете много.

Есть те, кто с этим не согласится)

 

Можно подробнее?

Просто четкого мнения я не встречал. Влад Мержевич пишет, что сброс стилей порождает безответственную верстку. Макфарланд и Майер, наоборот, призывают использовать глобальный сброс. Я стараюсь сбрасывать стили только для тех тегов, которые используются в макете. Есть другие варианты?

[advokatua]

Правильнее использовать сброс стилей. информации про это в интернете много.

Есть те, кто с этим не согласится)

 

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

 

Автор: Сожми окно браузера и крутани скролл вправо.

 

 

Макфарланд и Майер, наоборот, призывают использовать глобальный сброс.

Не совсем так. Это предложение вырванное из общего контекста, которое значительно меняет смысл написанного.

Edited November 18, 2013 by advokatua

[ya_verstaka]

Автор: Сожми окно браузера и крутани скролл вправо.

.  Уже исправил 

[Struggle]

Валидатор то до сих пор ругается..)

[Temik2704]

По дизайну увидел мелочь - в слайдере при активном слайде - кружочек который отвечает за слайд не совсем пустой,слева белый фон.Может это так и нужно,не знаю