Насколько ужасна HTML-верстка? :)

Приветствую всех пользователей WL.

Вот решил попробовать себя в роли HTML-верстальщика... Нашел в Сети очень простой макет и взялся его верстать. Что из этого вышло можно увидеть в портфолио:

ссылка

Выскажите, пожалуйста, Ваше мнение. Критикуйте - ничего не боюсь!

PS. Кто может дать пару не очень сложных, но реальных макетиков, чтобы потренироваться?

С уважением, Святослав.

class="logo">Golfing.com

Зачем тебе класс? Ты его ещё раз на странице будешь использовать?
Если нет, то используй id.

Картинки весят 282 кило. :o :idea: :!:
Оптимизировать не пытался?

Нет ни одного комментария в CSS. Типа гадайте сами, что к чему.

Футер к низу не прижал.

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

Зачем тебе класс? Ты его ещё раз на странице будешь использовать?
Если нет, то используй id.

А чем по-Вашему id лучше, чем класс?

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

Зачем тебе класс? Ты его ещё раз на странице будешь использовать?
Если нет, то используй id.

Я id использую предпочтительно вместе с js. + глянул на w3.org - там для лого используется h1 (который, понятное дело, больше нигде не используется) и для него все же задается класс "logo".

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

Картинки весят 282 кило.
Оптимизировать не пытался?

На самом деле не пытался - сохранил как есть. Надо будет попробовать, спасибо

Прошу совет насчет футера: всегда ли его нужно прижимать? Мне, например, показалось, что оно и так выглядит хорошо.

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

Прошу совет насчет футера: всегда ли его нужно прижимать?

Как правило, лучше прижимать.

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

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

Зачем тебе класс? Ты его ещё раз на странице будешь использовать?
Если нет, то используй id.

А чем по-Вашему id лучше, чем класс?

Он ничем не лучше, просто семантичней, на мой взгляд, использовать id, тк класс подразумевает использование несколько раз.
Айдишка используется только один раз.

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

Он ничем не лучше, просто семантичней, на мой взгляд, использовать id, тк класс подразумевает использование несколько раз.
Айдишка используется только один раз.

Поэтому лучше использовать класс. Если понадобится скопировать какой-нибудь блок, то это можно будет смело делать.
Если же в блоке использован id, то надо будет переделывать вёрстку.

Если это Ваше - то очень даже хорошо!

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

Пока средне.
Графика, как уже сказали, совсем не оптимизирована. Header и в png весил бы существенно меньше. JPG же для простых макетов Save for web, 60% - более чем достаточно.
Виндовый Thumbs.db (скрытый файл) в архиве с картинками совсем не нужен.
pics.jpg - полагаю, тут подразумевалась не цельная картинка, а небольшая галерея.

Разделители в меню делаются через css, например, так: ссылка

В css нет структуры, свойства прописаны в разнобой - обычно либо расставляют по алфавиту, либо в каком-то задуманном порядке.
Для <link> хорошим тоном будет прописать media.

min-width не помешал бы, иначе при маленьком разрешении правая колонка падает под левую.

Vitaflax, в крупных проектах сейчас принято делать всю разметку на классах. А id остаются для js-программистов, им нужнее. В мелких - все на усмотрение верстальщика, рекомендация же одиночные элементы размечать строго id немного устарела морально.

Всем добрый вечер.

По поводу использования id: лично я практически везде использую классы.
После верстальщика статика зачастую попадает к программисту. Он, в свою очередь, прикручивает скрипты и вставляет в документ теги шаблонизатора
Чтобы облегчить жизнь программисту, лучше использовать классы, так как на id зачастую вешается js обработчик.
Если скрипт написан для конкретного id (а если их еще и несколько вложенных), а в документе стоит ваш и на него повешен сss - то программисту нужно будет копать скрипты, менять Id.
А это трата лишнего времени.

По поводу id Sir_Michael опередил

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

Если это Ваше - то очень даже хорошо!

Спасибо.

Sir_Michae, спасибо за такой развернутый ответ.

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

Графика, как уже сказали, совсем не оптимизирована. Header и в png весил бы существенно меньше. JPG же для простых макетов Save for web, 60% - более чем достаточно.
Виндовый Thumbs.db (скрытый файл) в архиве с картинками совсем не нужен.

В png не удалось в приличном качестве получить небольшой размер - буду пробовать. Зато в jpg добился пока 14кб.

Насчет thumbs.db - действительно он здесь ни к чему - только на неделе установил windows, так archlinux пользуюсь. Удалил.

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

pics.jpg - полагаю, тут подразумевалась не цельная картинка, а небольшая галерея.

Действительно, изменил.

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

Разделители в меню делаются через css, например, так: ссылка

Сделал.
Цитата ("Sir_Michael"):

В css нет структуры, свойства прописаны в разнобой - обычно либо расставляют по алфавиту, либо в каком-то задуманном порядке.

Спасибо, теперь буду ставить в алфавитном порядке, изменю.
Цитата ("Sir_Michael"):

Для <link> хорошим тоном будет прописать media.

Проставил.
Цитата ("Sir_Michael"):

min-width не помешал бы, иначе при маленьком разрешении правая колонка падает под левую.

Сделал, завтра поправлю для IE 6.

Перезалил новый архив. Сервера сейчас под рукой нет, позже залью.

Всем огромное спасибо за комментарии

Упорядочил свойства по алфавиту, перезалил.

Все доделал, перезалил.

Спасибо большое всем ответившим в данной теме