Неплохо для начинающего.
По мелочи
Ниже все мое ИМХО (что то родилось и прижилось на основе требований при работе в различных командах) и не претендует на истину
По HTML:
1. У тегов <a> всегда стаить атрибут title. Заполнять (верстальщику) не надо. Я всегда пишу на подобии <a href="#" title="">....</a>
2. У тегов <img> всегда стаить атрибуты alt и title. Заполнять (верстальщику) не надо. Я всегда пишу на подобии <img src="...." title="" alt="">
3. все стили в таблицу стилей. Здесь этому не место
<div style="width: 900px; height: 1px; clear: both;"></div>
Кроме того width:900px здесь (если я не проглядел чего) лишнее, у вас родительский блок имеет фиксированную ширину. (кстати уже чем 900px).
Во многих командах (и я для себя перенял) такую конструкцию обзывают классом clear.
.clear{
height:1px;
clear:both;
overflow:hidden;
}
<div class="clear"> </div>
- не помню точно для какого, но старого браузера (скорее всего из линейки осла) - он подобную конструкцию без пробела игнорировал. Вообще и в пустых боках и в пустых ячейках таблицах всегда ставлю если ситуация, что оно мешает убираю, но как правило на стадии построения страницы вообще этот элемент убрать можно в этом случае
4. Параметры id оставляю тем кто будет пришивать скрипты. для верстки только class
5. Как то читал статью по сео. Там рекомендовалось, чтоб на одной странице был только один тег <h1>...</h1>. вопрос, конечно, холиварный, но у меня прижилось
6. Тут больше вопрос, конечно, к дизайнеру. Но кнопки не живые. Лучше, чтоб при наведении они как то видоизменялись. ВОзможно вы знаете этот метод, на сякий случай расскажу:
Картинку кнопки или ее фона делаете двойной (а можно и тройной высоты)
По высоте делаете. В нормальном состоянии и при наведеном. Третье состояние можно например для текущего выбора. А затем:
.button{
background:#CCC url("bt.png") center top no-repeat;
}
.button:hover{
background:#CCC url("bt.png") center bottom no-repeat;
}
.active{ /* стиль добавляется скриптом при формировании страницы */
background:#CCC url("bt.png") center center no-repeat;
}
Кстати, в одном зарубежном проекте видел когда в один файл png засунули вообще все кнопки (и их состояния) с сайта. И опирировали в таблице стилей уже смещениями этого фона.
То что не убивает нас, делает нас сильнее! © Ницше.
When life puts you in tough situations, don’t say "why me". Just say "try me".
GNU/Linux Debian