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

[@ndrey]

Сайт - www.planetmebel.ru (без контента пока)

Верстка блочная

Хотелось бы услышать мнение форумчан о верстке.

Заранее спасибо...

[strix]

Первое впечатление весьма хорошее. Было бы здорово учесть следующее:

1. Протестируйте в ФФ - несколько мелких правок сразу бросятся вам в глаза

2. Сделайте выделения для активного пункта меню

3. Список продукции - лучше выровнять по центру и убрать верхний регистр

4. Подтяните контент под заголовок (имею в виду уберите расстояние между заголовком "О фирме" и текстом под заголовком)

5. Поисковая оптимизация - тег h1 больше подходит для заголовка, чем h2, подписи к картинкам

6. Оптимизируйте css - используйте краткие формы для объявления свойств (например, свойства фона объедините в background), сделайте переносы строк - через год вы будете по полчаса искать нужное свойство в подобном коде

7. Копирайт слишком броский - одинаковый с преимуществами. И вообще подзаголовки стоит сделать крупнее и не в верхнем регистре (так хуже читаются длинные названия)

А так - молодца.

[@ndrey]

strix по поводу пунктов 3, 4, 7 не могу - таков утвержденный дизайн макет, по поводу оптимизации css учел, спасибо.

А по поводу ФФ там вроде косяков нет...

[Vlad]

Ну если верстку, то ошибки имеются.

так нельзя писать, убирай px.

В некоторых случаях теги закрываются так тег/>, а в других тег />. Это не ошибка, но почему по разному на одном сайте.

[@ndrey]

Ну если верстку, то ошибки имеются.

так нельзя писать, убирай px.

но валидатор ( ) говорит, что все правильно

[strix]

А по поводу ФФ там вроде косяков нет...

Андрей, наведите на линки в меню слева в ФФ - долго разбираться в чем ошибка, посмотрите css. Возможно, в том что класс для линков назван также, как и родительский блок - так не стоит делать.

Для размеров картинки в пикселях само слово px не пишется - http://www.w3.org/TR/html4/sgml/dtd.html#Length. То, что сегодня валидатор проглотил, не значит, что так правильно :-)

сократите ещ? стили (например, padding, font), а также лишние классы - например, для блока рекламы у вас 5 классов - слишком много

То же самое для

- не дублируйте в одно название для класса линков и ид блока. Лучше напишите:

div#mainmenu a{объявление стилей для всех линков в этом блоке}

[Дядя Саша]

Мне понравилось, уже хочу себе такую кухню ))

Единственное - ссылки в меню можно сделать подчеркнутыми при наведении указателя, ну это лично мое предпочтение...

[rash]

Плагин "HTML Validator" для FF выдает по поводу Width и Height варнинги...

[Fu-tai]

Наконец то ресурс который, реально, можно отрецензировать!

Ну для начала! Сайт статичный, хотя дизайн позволяет сверстать его резиновым!

Дизйн кстати замечательный!

Верстка грязноватая! Много лишних дивов, и неучтенных тегов!

Кодировку наверное всетаки выбрать utf-8 Лояльней и лучше.

Непонятно почему, но в разметке встречаются стили положения align="right" align="center"

Из за этих неучтенных стилевых тегов, при отключении стиля страницы весь текст почему то справа, а не слева как должен быть!

Возникает логичный вопрос, а CSS для чего придумали? (Все стили в CSS и вам будет проще и владельцы наладонников и прочих устройств, браузеры которых не поддерживают стили, скажут вам спасибо).

Про заголовки, действительно лучше использовать h1. Хотя поисковики и берут заголовки вплоть до h4, всетаки важность они оценивают начиная с h1!

Вообще название зря обернуто в див! h и так блочный элемент и обладает всеми свойствами для блока!

Почему Вам проще обернуть текст в див, а не написать для него стиль?

Тоже самое относится к картинкам. Не надо плодить дивы если вам надо лишь распорложить картинку и рядом тест. Задайте float прямо для картики и все будет как надо!

Почему меню вправаа написано списком, а меню над контентом просто ссылками?

Используйте списки для меню и прочих групп ссылок! Эт намного проще!

Заголовки "О НАШЕЙ ФИРМЕ" всетаки лучше сделать тегом h, скажем h3, главное чтоб он был ниже, по рангу, названия компании, а не Дивом. Который кстати лишний! Текст лучше не уводить от заголовков за тридевять земель, а распологать поближе к заголовку! Тоже к остальным заголовкам!

CSS захламлен.

Вопервых читать код сложно!

Во вторых много id и практически нет class хотя повторяющихся по стилям элементов у вас достаточно!

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

Очень много обнулений. лучше используйте ластик вначале кода

* {

margin:0;

padding:0;

border:0;

}

Так будет проще и сэкономите себе и нам(пользователям) время.

Поехали дальше!?

Кнопку GO лучше заменить на "Найти" Сайт ддля русскоговорящей публики или как?!

Ссылки всетаки лучше подчеркивать!

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

К чему такой большой копирайт? Он превосходно бы смотрелся под адресом! Который кстати тоже не на месте! Желание сдвинуть его в правый блок, никак не покидает!

Собаку лучше отодвинуть от края на расстояние формы поиска!

Заключение:

Дизайн понравился. Сверстано грязно, но сверстано же! А это уже заслуживает респекта!

Дерзайте!

[PacMan]

Плагин "HTML Validator" для FF выдает по поводу Width и Height варнинги...

Его имя tidy Он ругается на width="100px" ибо указывать еденицы измерения тут нафик не нада. А так все валидно.

[@ndrey]

Fu-tai большое спасибо за замечания, учел

что касается размещений элементов дизайна на странице (собаки, текста и т.п. ) - изменить не могу, таков дизайн , который кстати не мой.

Не понял только про кодировку utf-8? Прикол? и про ластик если можно поподробнее - не сталкивался еще пока...

[Fu-tai]

У тебя кодировка указана charset=windows-1251

Хотя уже всем миром признано что логичнее использовать utf-8

Ластик- это прием убирания маржингов, паддингов и бордеров у всех элементов, в начале написания кода CSS. Вот здесь очень хорошая статья про это http://designformasters.info/posts/70-expe...ter-css-coding/