Вёрстка
1267 сообщений
2195 сообщений
#17 лет назад
Поворчу немного (все что написано сугубо личное, не стоит принимать за аксиому).Не валидируется - это из-за метода вставки flash. Рекомендую использовать swfobject (собственно, этот метод использует и adobe). ссылка
Нет заголовков h1..h6 (О компании, Новости как минимум), "Производители" тоже можно заключить в заголовок.
Изображения. Все что не часть контента, а часть оформления-дизайна надо бы выносить в css в background. К заголовкам и меню это тоже относится, если их действительно стоило делать изображениями (это обычно диктует заказчик или seo'шник) - тут придет на помощь image replacement (бояться этого приема не нужно, его и яндекс использует).
Стили для ie вынес бы в отдельный файл с conditional comments.
Размеры лучше задавать в em'ах.
Новости я бы оформил как dl - dt dd список.
*{margin:0;padding:0} - сейчас ругают подобный подход, как минимум потому, что этот ластик убирает паддинги-маргины у input'ов. Но, судя по оформлению, в этом проекте допустимо.
rt1..rt6 - прикольно. Думаю, можно было сделать иначе.
background-image:url(images/hf_27.jpg);
background-position:left top;
background-repeat:repeat-x;
background-color:#19191B;
лучше записывать так:
background:#19191B url(images/hf_27.jpg) repeat-x;
то же относится и ко всему остальному, что поддерживает сокращенную запись
Блок #main можно было отцентрировать без абсолютного позиционирования:
body {text-align:center}
#main {
margin:0 auto;
text-align:left;
}Названия блоков lside, rside - спорно, нужно приближаться к полному отделению оформления от содержания => левых-правых блоков бы не надо.
От .cleaner можно отказаться. От класса fl, пожалуй, тоже. Тот же .snews совсем излишне повторяется.
Комментарии внутри html - похвально, но в современной верстке чаще всего таблицы стилей превышают размер "layout" части html-документа - в css комментариев недостает.
Добавил бы какой-либо highlight при наведении на пункты, и какую-то часть шапки сделал бы кликабельной.
В целом все хорошо.
