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

[cof666]

По задумке должны были быть еще и скрипты, но я успел выучить только html и css, поэтому вместо сладйера стоит просто картинка

lysenko-project1.url.ph

Просьба указать ошибки как грубые так и мелкие, и как можно было бы их испарвить

Уже знаю, что верстка сама по себе кривая, много дивов, которые располагаются один за одним как попало и куча id вместо классов

Остальное хочу от вас услышать

Edited July 7, 2014 by cof666

[Нарек]

class="li1", id="biscuits" нехорошо, лучше хотя бы стандартное current-menu-item.

Пустые блоки clear нехорошо, смотрите в сторону clearfix.

 

  <div id="contentImg2"></div>   <div id="contentImg3"></div>   <div id="contentImg4"></div>   <div class="clear"></div>

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

 

 

<div id="text">   <p id="text1">from $5.99</p>   <p id="text2">from $3.75</p>   <p id="text3">from $11</p>   <p id="text4">from $4.99</p>   </div>   <div id="shop">   <a href="#" id="shop1">Shop Now!</a>   <a href="#" id="shop2">Shop Now!</a>   <a href="#" id="shop3">Shop Now!</a>   <a href="#" id="shop4">Shop Now!</a>   </div>

 

То же самое. Очень плохо. Нужно было писать 1 блок, стилизовать его, а дальше просто дублировать класс и поменять текст в html.

<div id="line1"></div>

очень плохо. Есть для этого border, или ::after ::before

 

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

[cof666]

лучше хотя бы стандартное current-menu-item

 

вот тут просьба пояснить, что это

 

остальное понял, спасибо за совет

[Нарек]

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

current-menu-item в переводе означает текущий пункт меню. То есть если текущий пункт меню выглядит как то иначе чем другие пункты, то лучше ему давать такой или подобный класс. А не li1

[cof666]

уяснил, спасибо