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

[undina]

Очень хочется конструктивной критики работы. На что обратить внимание?

вот ссылка http://undina.aiq.ru/Page-3.html

Макет брала отсюда http://forum.htmlbook.ru/index.php?showtopic=16763

Пример верстки в той теме специально не смотрела, хотела найти свое решение.

Спасибо за внимание, с наступающим Новым Годом

[ZI DAN]

Как обычно, отмечу привязку стилей к id у нескольких элементов. Я вместе с огромной армией других программистов категорически против этого. Стиль нужно назначать через класс.

В остальном, вроде, ничего страшного не заметил, но я не верстальщик - мне такое замечать не положено.)

Edited December 30, 2010 by ZI DAN

[undina]

Как обычно, отмечу привязку стилей к id у нескольких элементов. Я вместе с огромной армией других программистов категорически против этого. Стиль нужно назначать через класс.

В остальном, вроде, ничего страшного не заметил, но я не верстальщик - мне такое замечать не положено.)

Спасибо

Фуух, я думала, ругать будут в жестких выражениях

Насчет привязки стилей к id, я использовала готовый шаблон с сайта, ничего менять не стала.

В будущем лучше использовать классы?

[ZI DAN]

Фуух, я думала, ругать будут в жестких выражениях

Ещё не вечер.)

В будущем лучше использовать классы?

Только классы.

[Vlad]

Заголовки в левой колонке съехали чуть вправо, верхнее меню, наоборот, влево.

Transitional в наше время как-то не канает, надо на Strict переходить.

Таблица зачем? По сути данные не табличные, это список скорее, логичнее выводить его иначе.

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

[undina]

Заголовки в левой колонке съехали чуть вправо, верхнее меню, наоборот, влево.

Спасибо, завтра исправлю

Transitional в наше время как-то не канает, надо на Strict переходить.

Пойду гуглить, спасибо, об этом даже не задумывалась

Таблица зачем? По сути данные не табличные, это список скорее, логичнее выводить его иначе.

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

[deadrash]

#053B6B -moz-use-text-color #1173B0

Не знаю что это, но мне это понравилось =)

Странно, у меня края синих плашек не совпадают с краями белого фона, это только у меня или только на убунту?

Edited December 30, 2010 by deadrash

[Протуберанец]

Пойду гуглить, спасибо, об этом даже не задумывалась

http://htmlbook.ru/samhtml/struktura-html-koda

[undina]

Не знаю что это, но мне это понравилось =)

Ой, где вы такую красоту нашли? Какой-то глюк, в коде в упор не вижу

В каком браузере непорядок? Делалось в Хроме, проверялось в Опере, Сафари, Лисе, ИЕ (частично тестером)

[ZI DAN]

Насчет классов и id, на форуме идет активное обсуждение, что лучше и почему использовать

Хотел освежить в памяти аргументы "тёмной стороны", но потерял тему с обсуждением этой проблемы. Ссылочки не будет?

[deadrash]

Ой, где вы такую красоту нашли? Какой-то глюк, в коде в упор не вижу

Это на форме поиска очень красивый бордер, сразу в глаза бросилось.

В каком браузере непорядок? Делалось в Хроме, проверялось в Опере, Сафари, Лисе, ИЕ (частично тестером)

Под убунтой Лиса. убунту 10.10, лиса 3.6.13

[Juster]

Очень хочется конструктивной критики работы. На что обратить внимание?

вот ссылка http://undina.aiq.ru/Page-3.html

Макет брала отсюда http://forum.htmlbook.ru/index.php?showtopic=16763

Пример верстки в той теме специально не смотрела, хотела найти свое решение.

Спасибо за внимание, с наступающим Новым Годом

1. размер шрифта текстов очень мелкий

2. <li><a href="#"> Services</a></li>

<li><a href="#">Support </a></li>

<li><a href="#">Careers </a></li>

<li><a href="#"> About </a></li>

Почему здесь различное кол-во пробелов?

3. в css файле все неаккуратно налеплено. добавьте табуляцию, разделяйте описания строкой, не лепите } в той же строке где был текст стиля

ужасно:

h3 span {

color: #a20000;}

отлично:

h3 span

{

color: #a20000;

}

избавьтесь от лишних пробелов:

#header .menu li.Search { - в оригинале тут лишние пробелы, но на форуме они не отображены

выдерживайте общий единый стиль в css файле

color: #FFFFFF; - в uppercase, хотя повсюду lowercase

Edited January 1, 2011 by Juster

[undina]

Исправила съехавшие заголовки, привела в порядок структуру файла css.

Что касается размера шрифтов, извините, как в макете. Возможно, было бы логичнее использовать em.

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

Edited January 12, 2011 by undina