1. Чем именно верстка то не угодила?
Во первых, спасибо за адекватную реакцию на комментарий, я думал, что это будет холивар :)
По вёрстке:
1. Пожалуйста, откажитесь от табличной вёрстки. Блочная модель куда более привлекательна.
2. Появление css ознаменовало прекрасную возможность разделения структуры и представления. Использование презентационного кода абсолютно не оправдано.
3. Тегов огромное количество, и это не просто так. Например, тег заголовка - h1-6, ну вы понимаете к чему я.
4. Вот это, например, вообще убило.
<table cellpadding="0" cellspacing="0" border="0" class="theme_layer1"><tr><td align="center" valign="top">
<table cellpadding="0" cellspacing="0" border="0" class="theme_layer2"><tr><td align="center" valign="top">
<table cellpadding="0" cellspacing="0" border="0" class="theme_layer3"><tr><td align="center" valign="top">
<table cellpadding="0" cellspacing="0" border="0" class="theme_layer4"><tr><td align="center" valign="top">
5. Радует, что css-файл организован чисто и аккуратно, даже комментарии есть.
Однако, при этом не было найдено ни одного использования id, как способа привязки стиля к элементу и ни один класс не привязан к конкретному тегу.
5.1. Хорошо, что указано несколько шрифтов.
font-family: tahoma, arial;
Но, в любом случае, завершать список должен sans-serif (гротеск) или serif (любой шрифт, который есть у юзера с засечками).
5.2. Зачем четыре раза?
padding: 0px 0px 0px 0px;
5.3. Это можно записать в одну строчку (уже придираюсь):
background-image: url(images/theme_light_bg.jpg);
background-position: top;
background-repeat: repeat-x;
P. S.: <!-- Да, мне нечем заняться кроме как просматривать код страницы :) -->