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

Здравствуйте, оцените, пожалуйста верстку
ссылка

7 Errors, 4 warning(s) - w3 validator

1. Лого - ссылкой, а не фоном
2. Разделители в меню не <li>|</li>, а через бордеры
3. Отступы у параграфов не через <br />, а маргинами
4. Я бы вместо
<div id="title1">
<p>Title</p>
</div>
использовала тег h1, а следующий заголовок через h2

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

я тоже собрался выложить верстку на оценку но смотрю желающих наставлять новичков на путь истинный тут немного, картинки критиковать интересней чем разбираться в коде =\

Слабо. Текст в первой и второй колонке вылезает под footer, поэтому без правок код оценивать смысла не имеет.

Спасибо, всем за критику, буду разбирать.

Спасибо, эти ошибки выходят из-за рекламы на narod.ru

Цитата ("Jokena"):

1. Лого - ссылкой, а не фоном
2. Разделители в меню не <li>|</li>, а через бордеры
3. Отступы у параграфов не через <br />, а маргинами
4. Я бы вместо
<div id="title1">
<p>Title</p>
</div>
использовала тег h1, а следующий заголовок через h2

Спасибо, учла ваши рекомендации.

Цитата ("SvinDevelop"):

Не стоит выкладывать на народе =\

подскажите, пожалуйста, где лучше выложить?

Цитата:

в ie текст не получается увеличить

а как вы увеличиваете?

заранее спасибо

Цитата:

а как вы увеличиваете?

Самый быстрый способ изменить масштаб страницы -- зажать Ctrl и покрутить скролл на мыше
всегда указывайте размер шрифта в %%, если фиксированный шрифт не указан явно заказчиком

Цитата ("418689322"):

Цитата:

в ie текст не получается увеличить

а как вы увеличиваете?

заранее спасибо

Не получается, потому что размер шрифта выставлен в пикселях. Попробуйте сделать верстку на емах и все будет увеличиваться.

Цитата ("hija"):

Kolynia, так вы увеличиваете масштаб, а просят шрифт.

В ие6 шрифт увеличивается через контрол.

Цитата:

Текст не гибкий.
Навигация не масштабируемая.
Фиксированная высота.

Можно по подробнее, я просто еще совсем новичок: 1. гибкий - это какой должен быть; 2. я тоже не совсем поняла; 3. Фиксированная высота - это плохо?

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

418689322
При верстке нужно прежде всего предусматривать, что содержимое будет меняться. Будущий сайт не останется в таком виде, как вы его сверстаете.
Заказчик/владелец сайта может решить изменить порядок или текст пунктов меню, заполнить страницу разнородным контентом любого объема. И если с этим будут проблемы - что-то где-то не влезает или вылезает - это плохо.

Цитата ("418689322"):

Цитата ("SvinDevelop"):

Не стоит выкладывать на народе =\

подскажите, пожалуйста, где лучше выложить?

Пробуй вот этот хостинг удобный для тестов ссылка бесплатный !

Спасибо, гибкий текст я уже сделала, а над высотой буду думать

Спасибо, попробую

Цитата ("418689322"):

Цитата ("SvinDevelop"):

Не стоит выкладывать на народе =\

подскажите, пожалуйста, где лучше выложить?

я этим пользуюсь byethost.com

Не удержался переверстал вашу страничку по своему пониманию
ссылка
тоже не идеально, шапка фиксированая, если сильно увеличить шрифт слова вываливаются из меню и заползают на другие блоки
хочется увидеть комментарии более опытных верстальщиков

Скажите, а чем Вы руководствовались, когда вставляли тег <label> и когда всю навигацию разместили внизу, спозиционировав ее абсолютно?

Цитата ("Jokena"):

Скажите, а чем Вы руководствовались, когда вставляли тег <label> и когда всю навигацию разместили внизу, спозиционировав ее абсолютно?

уменьшил зависимость html документа от стилей и наоборот
лейбел да, фанатизмом отдает , убрать надо