hemulll Сообщение 06/02/2009 07:45 Копия темы
Оцените верстку Просьба оценить верстку по шкале от -5 до 5 желательно с комментариями.
www.free-lance.ru/users/h...
www.metroregion.ru/
P.S. Кто найдет пасхалку, тот молодец :)

girichev Сообщение 06/02/2009 07:52 Копия темы
За использование таблиц без необходимости — такому верстальщику руки бы оторвать, и то что пасхальное...

hemulll Сообщение 06/02/2009 07:56 Копия темы
Ну вообще верстал я, я не проф верстальщик, но нужда заставила – поэтому и оценить попросил. Помоему таблица в шапке использована как раз по необходимости :).

s-design Сообщение 06/02/2009 08:06 Копия темы
Вплане, что таблицы использовать для представления данных.

Я вот недавно таки использовал таблицу для реализации одной фигни, получилось меньше мозгое* чем с блоками.

hemulll Сообщение 06/02/2009 08:11 Копия темы
Ну да я сделал также – использовал для реализации одной фигни :)

frost_lab Сообщение 06/02/2009 08:15 Копия темы
Никакой необходимости в этом нет, можно было сделать списком.
<div class="hline"/> – только ради фона? а ведь можно было этот вон добавить к общему для header'a

<div class="caption"> – зачем оно там?

<div id="extra"/> в меню, я так понима, тоже только ради фона... можно было задать фон для <div id="menu">

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

s-design Сообщение 06/02/2009 08:22 Копия темы
а почему не сделать просто ul id="menu" ?
меньше оберток — лучше

hemulll Сообщение 06/02/2009 08:23 Копия темы
спасибо за комментарии – учту
Список съезжает при уменьшении ширины экрана, а с таблицей появляется горизонтальная прокрутка – так и не придумал как сделать со списками такое же – повторюсь верстать не люблю и не умею, но вот пришлось :)
<div class="hline"/> согласен лишнее – уберу
<div class="caption"> ну не знаю для управляемости отображением заголовка :)
<div id="extra"/> предполагается под меню потом пустить какие-то виджеты – пока еще не ясно какие – это задел на будущее

frost_lab Сообщение 06/02/2009 08:24 Копия темы
Затем, что я не спал уже 24 часа, и думать рационально не в состоянии... ))

UPD: ну вот ТС уже озвучил причину, по которой сделать просто ul id="menu" не вариант )))

s-design Сообщение 06/02/2009 08:25 Копия темы
Верстальщик всегда должен трезво оценивать ситуацию, лол :D
Я могу верстать 72 часа, хоть и на автомате код вылетает )

frost_lab Сообщение 06/02/2009 08:28 Копия темы

hemulll Сообщение 06/02/2009 08:33 Копия темы
<div id="menu">

<div id="extra"></div></div>
вэкстра будут виджеты, если делать прочто ул, то я не знаю как расположить потом под ними виджеты :)

CSR Сообщение 06/02/2009 08:48 Копия темы
Такой макет можно было сделать намного проще, есть лишние элементы. Ну и... HTML 4.01 Transitional как-то не модно уже о_О
Таблица вообще не в кассу, это списком делается )

Empresso Сообщение 06/02/2009 08:58 Копия темы
-2, без комментариев.

Hansito Сообщение 06/02/2009 09:14 Копия темы
щас набегут защитники диких животных и заставят вса сменить аватару)

hemulll Сообщение 06/02/2009 09:18 Копия темы
Картмана не сменю никогда :)

hemulll Сообщение 06/02/2009 09:19 Копия темы
жаль, что без комментариев

hemulll Сообщение 06/02/2009 09:23 Копия темы
блин не модно... придется делать модно

xadminx Сообщение 06/02/2009 09:44 Копия темы
- таблицы не для таблиц
скрипты раскиданы где угодно
стили в тэгах
невалидно

hemulll Сообщение 06/02/2009 09:48 Копия темы
спасибо за комментарий
скрипты поправлю, стили тоже попробую свести к минимуму, с валидностью сложнее...