Kiddo

1. Зачем логотип в тегэ img? 2. Нет IR в телефоне, куда звонить человеку который серфит с отключенными картинками?Image Replacement 3. Нет media для файла цсс 4. Новости это список 5. Шрифт в px 6. Для поисковой оптимизации центральную часть в коде нужно было выводить до сайдбара. 7. "Наши разработки" в идеале тоже список. 8. Невалидный цсс. 9. Неструктурированные заголовки, на одной странице должен быть один h1, а не 3. 10. Презентационные имена классов (modLeft, modCenter...) 11. Нексроссбраузерное объявления font-weight:100; 12. Написание многих цсс параметров можно сократить сокращения в CSS 13. Все картинки в гиф формате. Например, лого можно было сохранить в пнг-8, было б 5кб, а не 9. 14. Для ссылок можно было обнулить outline, но это дело вкуса. Outline 15. Для ссылок копирайта хорошим тоном задавать target="_blank" 16. Неотформатированная типографика, но это у вас наверно к дизайнеру, хотя этим в идеале должен и верстальщик заботится. 17. Ссылка реклама на сайте сделана блоком с падингом, активный получается и отступ между ней и навигацией.

С точки зрения незнания юзабилити: 1. Попав на главную, если б не картинка холодильника, пришлось долго думать что за сайта. 2. Логотип очень нечеткий, а слоган не распознаваемый. 3. Ссылка "запомнить сайт" находится в ключевом месте и берет на себя почти весь фокус сразу, зачем? 4. Надпись в футере "ПластХладо - детали, что надо!!!" не внушает доверия, три и более знака восклицания подряд ставят истеричные пользователи аси, для акцента достаточно одного. 5. Передавать идентификатор сесси в сылке плохо, нету ЧПУ. 6

1. Используйте доктайп, без доктайп браузер в quirk режиме работает Doctype 2. Некоторые атрибуты уже считаются устаревшими Deprecated тэги 3. Для ячеек таблицы есть css атрибут vertical-align

А зачем добавлять пробел?

А зачем использовать устаревший доктайп? Если задать xhtml 1.0 transitional будет проходить валидацию без контейнеров для элементов формы. При стриктовом доктайпе элементы формы нужно брать в контейнер (таблица,блок,параграф...)

2klierik Если о .itemsGalery, то мне кажется целесообразно делать на списке (список товаров), а не на блоке. Но я это рассматриваю как таблицу товаров. Спасибо за ссылочку, но цсс перед выпуском будет полностью сжимается(уберутся лишние проблемы, величины переменных после 0 значения, ; у последнего атрибута), по поводу комментирования соглашусь, но я это писал с расчетом на то, что все правки делать буду только я, ленивый, эгоистичный подход. Разделение стилей на отдельные файлы в этом случае мне кажется лишним, так как это только создает лишнюю нагрузку на сервер(выполняется больше запросов к серверу), мне кажется, этот подход целесообразно выполнять при больших размерах файла стилей.

2rus Пустая конструкция спанов, это IR(Image Replacement) Park, когда под картинкой прятают текст, для того чтоб пользователь с отключенной графикой смогу понять назначение этого блока, также этим способом порой злоупотребляют "черные" оптимизаторы. Подробней об этом IR и других способах: Как подставить текст под бекраунд А как вам удалось увидеть вертикальное дублирование бекраунда? Я при масштабировании его не вижу, там стоят спрайты, поэтому видно фон футера. В каком браузере, ОС увидели дублирование? Фот сделать шириной до 2-3 пикселя считаю лишним, потому что там стоит патерн, и при сильном уменьшении он плохо дублируется, но чуть-чуть ужать все-таки постараюсь. yopopt Спасибо за address, хороший тег, о нем совсем забыл, в следующий напишу еще с учетом микроформатов. А где вы увидели лишние брейклайны? Как по мне там лишний только один в хедере в блоке авторизации, а все остальные уместны, мне кажется в ряде случаев это лучше чем брать все подряд в параграф и обнулять отступы, или я не прав?

Спасибо всем кто уже откликнулся. 2Vindex10 А где конструктивная критика? Там всего 2 таблицы и все остальное на блоках и списках... Если вы считаете что таблицы были где-то применены неуместно, обоснуйте свою точку зрения адекватными доводами. 2homm Вариант когда товары оформляются плавающими блоками и в строке количество товаров зависит от разрешения экрана был откинут заказчиками, захотел чтоб в центре всегда было два товара. Дизайн был сделан по принципу - как хочет заказчик, а не как будет лучше, поэтому много графических ляпов. Прошу критиковать верстку, а не дизайн.

Прошу прощение. Поправил, случайно скопировал папку с названием с большой буквы

Здравствуйте. Покритикуйте пожалуйста верстку. http://zodios.net/portfolio/karefun/index.html Буду благодарен всем, кто не полениться потратить свое время и напишет замечания. Попутно еще один вопрос. Есть сервисы для сжатия цсс файлов, например http://www.cleancss.com/ , он убирает с файла все лишние пробелы, табы, единицы измерения после нулевых значений параметров, точку с запятой в конце последнего свойства селектора. Пользуйтесь ли вы им? Знаете ли программную реализацию подобного инструмента? Есть ли в ней функция обратная сжатию?

Если нету возможности поставить основные браузеры можно воспользоватся сервисом http://browsershots.org/, о позволяет увидеть верстку не только в разных браузерах а и на разных платформах.

Можно сделать через бекграунд который будет на 2 пикселя короче длины списка, или через повторяющийся бекраунд для ul, а последним li перекрывать фон на 2 пикселя. А для какой цели нужно это применить, а то тяжело ориентироваться когда задача вырвана с контекста.

Смотря для чего нужно применить мин-видс, если для боди, чтоб блоки не съезжались в разрешении меньше 1024, то экспрешины мне кажутся самым лучшим выходом, а если нужно для какого-то внутреннего блока, то можно сделать через распорку? или через дополнительные блоки с эмулировать мин-видс... Для мин-хейт есть более красивый вариант который проходит валидацию Особенности свойства height

Можно, или использовать експрешены в цсс, или подключать джаваскрипт который будет пересчитывать размеры Эмуляция min-width, max-width под IE6

Аналог - overflow:hidden; Что кому ближе к сердцу, мне этот вариант нравиться больше с точки зрения логики.