Salut,
<div id="header">
<ul>
<li class="actif1"><a href="index.html" title="Revenir à la page d'accueil"></a></li>
<li class="actif2"><a href="index.html" title="Accueil"></a></li>
<li class="actif3"><a href="index.html" title="Revenir à la page d'accueil"></a></li>
<li class="actif4"><a href="index.html" title="Accueil"></a></li>
<li class="actif2"><a href="index.html" title="Revenir à la page d'accueil"></a></li>
<li class="actif1"><a href="index.html" title="Accueil"></a></li>
<li class="actif4"><a href="index.html" title="Revenir à la page d'accueil"></a></li>
<li class="actif3"><a href="index.html" title="Accueil"></a></li>
<li class="actif1"><a href="index.html" title="Revenir à la page d'accueil"></a></li>
</ul>
</div><!--fin header-->
1. Liens vides
2. Liens -hyper- redondants
3. Title différent alors que la destination du lien est différente
4. Un gros sentiment que quelqu'un avec une synthèse vocale va pleurer en entendant 4 + 5 fois le même intitulé (l'attribut title remplacant l'intitulé du lien je crois bien)
Bref... Tout ca pour un truc parfaitement inutile en terme d'information, et purement graphique. Mettre des <span> sans lien, avec une position :hover ferait le même effet avec un seul lien tout autour, avec un intitulé clair.
Quand tu veux mettre du texte en majuscule, tu le tape comme le veux la convention, en minuscules avec une majuscule au début des phrases et des noms propres. Et ensuite tu utilise un text-transform dessus pour le passer en capitales.
Pourquoi utiliser une classe "black" dans ton div "right ? Tu aurais le même effet en sélectionnant tous les <p> situés dans "right", et du coup tu éviterais d'alourdir ton code de classes inutiles.
Pourquoi ne pas externaliser la partie des styles concernant IE6 ? La tu oblige les utilisateurs modernes à se taper le chargement d'un code qui n'est utile qu'aux vieux dinosaures.