Оцените музыкальный блог. Нужна критика

[Nikita Kirpo]

Рассказываю:

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

<div class="top-menu">
  <ul> 
      <li><a href="http://moope.ru/">Главная</a></li> 
      <li><a href="http://moope.ru/ob-avtore.html">О проекте</a></li> 
      ........
      <li><a href="http://moope.ru/pravoobladatelyam.html">Юристам</a></li>
  </ul> 
  <div style="clear: both;"></div> 
</div>

2. Какова цель связки <a href=""><span>? Кого нужно запутать? Другое дело, если текущий пункт меню вместо ссылки будет обернут элементом SPAN с приданием соответствующих стилей. Но, к сожалению, большинству пользователей SMC такой код не под силу.

3. Говнокод №1:

<div align="center">

Во-первых, нельзя смешивать структуру документа с его представлением. Последнее лежит на плечах стилей.
Во-вторых, чего вы хотите этим добиться?
Если выровнять блок по центру родителя, пропишите ему ширину и авто поля (margin:auto).
Если выровнять линейное содержимое блока, пропишите ему стиль text-align:center.
Если выровнять блоковое содержимое, поступите с ним, как в первом случае (ширина и авто поля).

4. Вот это:

<span class="fot-block3"><a href="http://moope.ru/zrock/244-deep-purple-smoke-on-the-water.html">Deep Purple - Smoke On the Water</a></span>

хотя и не является нарушением правил, однако сводит на нет смысл применения элемента SPAN, как выделителя части строки.
Почему бы не сделать просто:

<a class="fot-block3" href="http://moope.ru/zrock/244-deep-purple-smoke-on-the-water.html">Deep Purple - Smoke On the Water</a>

5.

<div class="itro-text"><br><br><br><br><br><br><br><br><br>

О, маркиз! Вы перещеголяли самого короля!
неужели padding-top уже отменен?

6. вместо

<i>Среда 18 мая</i>

пишите

<em>Среда 18 мая</em>

а вместо

<b>Май 2011 (53)</b>

пишите

<strong>Май 2011 (53)</strong>

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

7. Элемент <center> запрещен стандартами W3C. Чем его заменить - читай п. 3.

8. Заявленный доктайп XHTML требует обязательного закрытия слешем пустых элементов (BR, INPUT, IMG, ...)

9. В таблице какие-либо данные могут быть только в ячейках, но никак не между ними:

<table width="100%"> 
    <tr> 
        <div id="vk_poll"></div> 
    </tr> 
</table>

так неправильно, нужно так:

<table width="100%"> 
    <tr> 
        <td>
            <div id="vk_poll"></div> 
        </td>
    </tr> 
</table>

10. Скриптам нужно обязательно указывать type="text/javascript", а вот language писать не нужно



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

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

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

ап теме