Оцените работу пожалуйста!

[li4e]

Все добрый вечер. Наконец-то сверстал свой первый сайт из psd, раньше баловался просто ковыряясь в чужом коде.

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

Прошу пожалуйста критики! Заранее спасибо! Именно по части верстки, как можно было сделать все проще!

Вот сам макет

Вот результат

Вот Файлы

Edited April 18, 2013 by li4e

[deadrash]

А если в реальности текст заголовка #text3 будет чуть длинней? С заголовком #text1 тоже самое.

ul может содержать только li и никаких a

Вы бы хоть через валидатор прогнали сначала.

Edited April 17, 2013 by deadrash

[li4e]

Так такой же, или отображение?

Сейчас прогоню, извиняюсь!

Edited April 17, 2013 by li4e

[deadrash]

Зачем использовать div вместо картинки, если можно использовать картинку.

[li4e]

Так как она часть дизайна, а когда я делал картинкой, то мышкой она выделялась, поэтому использовал div!

А можно использовать такую конструкцию?

<ul>
<li><a> </a></li>
</ul>

- Проверил, можно!)

Edited April 17, 2013 by li4e

[deadrash]

Так как она часть дизайна, а когда я делал картинкой, то мышкой она выделялась, поэтому использовал div!

Да вообще не желательно использовать элементы просто для визуального оформления. Собачку можно было пустить фоном к элементу обёртке в конце концов.

Когда верстаете страницу, необходимо в первую очередь понимать страницу как документ, а всё визуальное уже подстраивать к этому документу.

У вас совершенно неосмысленная разметка, например логотип - это просто пустой див в ссылке, а его желательно сделать заголовком "Dog Kennels", в котором будет картинка с описанием (alt) и заголовком (title)

Вот прочтите эту статью http://webstandards....tic-markup.html она хоть и не очень свежая, но очень полезная.

Edited April 17, 2013 by deadrash

[li4e]

Хорошо, спасибо!

[deadrash]

Так как она часть дизайна, а когда я делал картинкой, то мышкой она выделялась, поэтому использовал div!

А можно использовать такую конструкцию?

<ul>
<li><a> </a></li>
</ul>

Да, конечно.

На счёт заголовков strong внутри #text3 например, поясню на всякий случай. Когда вы верстаете макет, вы обычно используете рыбу (лорем ипсум или одинаковые картинки), Но когда будет использован реальный текст или картинки, всё может поломаться, это нужно учитывать на момент вёрстки.

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

[li4e]

Насчет этого да, Я как раз это сейчас обсуждал со знакомым, я когда верстал, Я четко подгонял под этот текст, наверно надо было использовать не фиксированную высоту блока с содержимым!

[li4e]

Огромное спасибо за ссылку на эту статью, прочитав ее я понял что такое семантическая верстка, и что мое понимание правильной верстки было неверным, и понял сколько я допустил ошибок при верстке собачьего сайта!