http://vilia.4html.ru/

[Иван Шумов]

http://vilia.4html.ru/

вот, оцените можете со всей строгостью конечно интересует только верстка, ибо я ни разу не дизайнер и дизайн не мой , но красивый

сразу скажу

1) фулл валид

2) гибридная верстка

3) оптимизация под 1024

4) отключив картинки вы увидите что сайт очень милый)

[Sotnikov]

если строго, то вот

1. При отключении картинок, "Вилия лого" можно было бы засунуть в h2 на мой взгляд, понимаю.. что там alt выводит, но лучше логотип внести в атрибут к стилю.

2. ss_center в принципе можно было бы организовать на div'ах, но возможно в этом случае и не нужно, но раз начал без таблиц, так закончи.

3. левое меню... а если будут добавляться менюшки? Бэкграунд ведь не растягивается.. сплошная картинка замощенная..

в целом здорово. Правда не понятно, для чего там синий и оранжево-желтые квадраты...но это уже задумка дизайнера.

[Иван Шумов]

1) ну, это в общем имхо, так что н этой почве спорить не стоит думаю

2) ну, да - можно было, но не стал извращаться. К тому же в таблицах плохого ничего нет.

3) установлено что не будут

[Sotnikov]

...

3) установлено что не будут

ну у клиента или у директора проекта сегодня одно на уме, завтра другое

[Иван Шумов]

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

[Jenek]

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

Меню и клиенты это списки, под одним даже написано "список", почему сделаны не списком не понятно.

Таблица с единственной ячейной ? это верный признак хорошей верстки.

lm и center ? это не названия для блоков.

Телефоны сделаны картинкой (скорее всего конечно просто еще не готовы), и если я отключю картинки по вашему совету то не увижу их, толку с того что страница при этом замечательно выглядит.

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

"в данном случае после сдачи проекта - не волнует" даже если разработчик так думает, то уж озвучивать точно не стоит.

Максимум строгости, как заказывали, хотя можно и продолжить.

[Иван Шумов]

ну и где вы увидели таблицу с одной ячейкой?

[Jenek]

Прошу прощения, не обратил внимания, что там две ячейки.

Тем не менее, таблица с двумя ячейками, в данном случае тоже не уместна.

Сравним два кода, сейчас:

<table border="0" cellpadding="0" cellspacing="0" style="margin-bottom: 30px;">
 <tr><td style="vertical-align: top; width: 60px; text-align:left;">
 <img src="http://vilia.4html.ru/images/default/b1.png" alt="" />
 </td>
 <td style="vertical-align: top; color: #4880c2;">
 <h2>Агентство ?Вилия?</h2>
 <p>Биржа труда ?Вилия? (ООО ?Вилия?) была создана в 1989 году. Уже более 15-ти лет нашей основной задачей является помощь предпринимателям при наборе квалифицированного и ответственного персонала, специалистам в поиске достойной работы и властям города в разрешении острых социальных проблем - проблем занятости и безработицы.</p>
 </td></tr>
</table>

должно быть:

<div style="color: rgb(72, 128, 194); padding-left: 60px; margin-bottom: 30px; background: url(http://vilia.4html.ru/images/default/b1.png) no-repeat;">
 <h2>Агентство ?Вилия?</h2>
 <p>Биржа труда ?Вилия? (ООО ?Вилия?) была создана в 1989 году. Уже более 15-ти лет нашей основной задачей является помощь предпринимателям при наборе квалифицированного и ответственного персонала, специалистам в поиске достойной работы и властям города в разрешении острых социальных проблем - проблем занятости и безработицы.</p>
<div>

хорошо бы даже так:

<div class="entry">
 <h2>Агентство ?Вилия?</h2>
 <p>Биржа труда ?Вилия? (ООО ?Вилия?) была создана в 1989 году. Уже более 15-ти лет нашей основной задачей является помощь предпринимателям при наборе квалифицированного и ответственного персонала, специалистам в поиске достойной работы и властям города в разрешении острых социальных проблем - проблем занятости и безработицы.</p>
<div>

в таблицах плохого ничего нет.

пока они используются как таблицы, конечно, см. пример выше.

[Vlad]

Внутренние стили типа style="margin: 0 32px 0 64px;" лучше выкинуть в файл со стилями.

В одном меню

нет, в другом они есть. Получается, что в перемешку разные стили верстки применяются. Это добавляет сложности при изменении кода.

Юзабилити как обычно страдает, но раз про верстку только речь зашла, ничего про это не скажу.

[homm]

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

Кажется это называетя халтура.

[@ndrey]

1. убрать стили из кода (как уже отмечалось выше);

2. меню в список;

3. зачем ссылки обрамлять блоками, в то время как есть такая штука как display?

4. зачем картинки с абсолютным позиционированием обрамлять блоками?

5.

 <b style="margin-bottom: 22px; display:block; font-size:11px;">Наши клиенты:</b>
					 ?Адамант?

в список;

6. таблицу убрать;

7. таблица стилей ужасна, плохо читаемая избыточная!

в целом - плоховато, избыточно.