Оцените верстку. Судите строго.

[webmax7]

Оцените верстку - http://dl.dropbox.com/u/15995074/portfolio/verstka_8/index.html

ie7+ и последние версии всех остальных

Из того что хочу еще доработать:

эмулировать плэйсхолдер для поля password, отобразить маркеры в блоке для ie7

Буду рад конструктивной критике, рекомендациям и советам относительно верстки, написания кода, используемых методов.

Спасибо.

Edited November 2, 2011 by webmax7

[Анна]

А правый верхний угол таким и должен быть?

[webmax7]

А правый верхний угол таким и должен быть?

Да.

[yacor]

для чего надо было ДОКТАЙП хтмл5 выбирать...

[SelenIT]

yacor, а для чего вообще доктайп в верстке?

По сабжу — 1) невнятные имена классов, 2) зря не используете text-transform:uppercase, 3) по-моему, спан с неразрывным пробелом ради оформительской картинки — излишество. Ну и JS-костыль на замену плейсхолдера для инпутов я бы не в атрибуты пихал, а грузил отдельно лишь там, где он нужен (по событию загрузки DOM, например). И еще лично мне непонятно, зачем списку новостей (нужен ли там список — отдельный холиворный вопрос, ну да ладно) сбрасывать дефолтный маркер, чтобы с помощью фоновой картинки для :first-letter нарисовать... практически такой же

[webmax7]

По сабжу — 1) невнятные имена классов, 2) зря не используете text-transform:uppercase, 3) по-моему, спан с неразрывным пробелом ради оформительской картинки — излишество. Ну и JS-костыль на замену плейсхолдера для инпутов я бы не в атрибуты пихал, а грузил отдельно лишь там, где он нужен (по событию загрузки DOM, например). И еще лично мне непонятно, зачем списку новостей (нужен ли там список — отдельный холиворный вопрос, ну да ладно) сбрасывать дефолтный маркер, чтобы с помощью фоновой картинки для :first-letter нарисовать... практически такой же

1) имена классов согласен невнятные - часть идеи БЭМ подхода, не отточенная до нужного уровня

2)про text-transform - почему зря, можно подробней

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

И спасибо за критику.

[Gaspode]

2)про text-transform - почему зря, можно подробней

Заглавные буквы - это, в вашем случае, элемент оформления. Так что по уму бы следовало перенести это в стили.

И да, зачем вам html5 в доктайпе, если вы используете его только для указания кодировки?

Edited August 11, 2011 by Gaspode

[SelenIT]

зачем вам html5 в доктайпе

Gaspode, да где, где там в доктайпе HTML5? В упор вижу только HTML А это что, не HTML разве?

[Vlad]

Ресет у вас тупой используется. Точнее вы его применяете тупо в лоб, даже не задумываясь над смыслом. vertical-align: baseline; умилило применительно к блочным элементам. Вы разве не знаете, что vertical-align для блочных не работает? А если знаете, зачем пишите?

Стилевой файл какой-то неуклюжий и избыточный. К примеру, на сайте нет тега ins, зачем ему стиль указывать.

При увеличении шрифта SUBMIT "проваливается".

[webmax7]

Ресет у вас тупой используется. Точнее вы его применяете тупо в лоб, даже не задумываясь над смыслом. vertical-align: baseline; умилило применительно к блочным элементам. Вы разве не знаете, что vertical-align для блочных не работает? А если знаете, зачем пишите?

Стилевой файл какой-то неуклюжий и избыточный. К примеру, на сайте нет тега ins, зачем ему стиль указывать.

При увеличении шрифта SUBMIT "проваливается".

Ресет Эрика Мэйера. Не оптимизирую его под каждый макет сознательно.

Избыточен стилевой файл или ресет?

Каким образом вы увеличиваете шрифт и в каких браузерах, искал настройках не нашел, раньше вроде, в Опере и ie можно было - хочу увидеть проваливающийся submit.

По поводу доктайпа - смотрите видео Вадим Макеев "Доктайп.Точка"

[Gaspode]

Ресет Эрика Мэйера. Не оптимизирую его под каждый макет сознательно.

Ну какбэ Мейер завещал его оптимизировать под каждый макет

[yarickyarick]

Ну серенько всё