Прокомментируйте вёрстку

[hristoforus]

Обычный макет

http://web.zone.ee/rezolution/

[Grifit]

Первое что кидается в глаза - большое количество файлов с CSS. Представь, что у тебя 30 страниц и каждая из них раз в 5 больше той, что у тебя. И представь сколько файлов будет в хадере. Нужно писать все в одном файле. Или выделить отдельно один файл под плагины.

"media="screen"" - это не обязательно.

"<br /><br />" - про это забудь. Когда программист вставляет текст, он вставляет это без "<br />". Все должно выравниваться без этого. Это очень крайний случай.

"<b><div id="data">22.12.12</div></b>" - зачем </b> ? У тебя есть id="data" туда просто добавить font-weight. Если делаешь жирным в тексте, то лучше применяй strong.

.bottom - оно не должно в том месте лететь через всю страницу.

По возможности не задавай ширину и высоту.

Не задавай названия классов "

shadow2

", "

shadow1

" - когда "

shadow

" будет 100 штук ты забудешь где что было.

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

Не пиши классы и id с большой буквы.

Почему в

Esileht два спана в одной ссылке?

Последняя ссылка потерялась, раз оно в спанах, пусть будет все в спанах.

Блоки с текстам у тебя в спанах. Там должен быть "p", тогда и не нужно br.

kartinki - находится слева, но при этом в html внизу? - зачем?

"

<span>Tahe Wood OU     <b>Mobil: </b>+372  77  69235  <b>Email:

" - так не делается. html должен быть чистым.

Почему скрипты в футтере? Они все должны быть в хадаре.

Подключать плагин нужно в главном файле js. Никогда не пиши скипты в html.

"

#footer

" - тут не должно быть высоты.

Избегай отрицательных марджинов.

У картинок должен быть всегда прописан alt . Высота и ширина тоже, но тут можно схалтурить.

"

html, body, div, span, applet, object, iframe,

h1, h2, h3, h4, h5, h6, p, blockquote, pre,

a, abbr, acronym, address, big, cite, code,

del, dfn, em, font, img, ins, kbd, q, s, samp,

small, strike, strong, sub, sup, tt, var,

dl, dt, dd, ol, ul, li,

fieldset, form, label, legend,

table, caption, tbody, tfoot, thead, tr, th, td

" - зачем это?

"<link rel="stylesheet" href="css/style.css" type="text/css" media="screen, projection">" - зачем он там пустой?

[Struggle]

1. Логотип должен быть кликабельным.

2. Зачем пилить лишний див который оборачивает меню?

3. Разберись с подвалом, там какой-то бардак, копирайты в форме картинки, контакты липнут к низу, ну наверное так и должно быть..)

Лень копаться дальше в верстке, дополните

Edited July 31, 2013 by Struggle