En avant pour la critique maintenant :
Visuellement :
* Sous IE6 (indépendament de la résolution) j'ai une petite déformation. Je suppose que le design prévoit d'évoir 4 images côtes-à-côtes sur un ligne, je n'en ai que trois et ai donc un espace vide sur la droite. Autre défaut visuel, le pied de page remonte et se retrouve sous les images (manquerait'il un overflow ?).
* Sans même fouiller le code, je sais déjà que tu as utilisé des pixels puor définir les tailles de polices. C'est embêtant, je ne sais donc pas grossir les textes.
* Les liens du footer en gris moyens sur gris moyen à peine plus clair, c'est illisible.
* Idem pour les liens du menu, le contraste est trop faible pour une lecture confortable (d'autant plus que les textes sont petits et non-agrandissables).
Le code :
* Avoir des titres identiques pour toutes les pages du site c'est moyen : pour le référencement et pour le bookmarking des visiteurs. Il est recommandé d'avoir pour chaque pages d'un site un titre et une description différents, tous deux caractérisant le contenu de la page en question.
* En parlant de description :
<meta name="description" content="Portfolio" />
Il faut absolument étoffer la tienne. Dans sont état actuelle, non seulement elle n'apporte rien, mais risque au contraire de te déservir. Cliquerait-tu sur un lien, dans les résultats d'un moteur de recherche, dont la seule description est "Portfolio" ? (Il n'est d'ailleurs pas sûr que Google considère cette description comme intéressante, dans ce cas, il affichera les premiers textes de ta page : ton menu).
* Les 152 lignes de code JS seraient bien, au chaud, dans un fichier séparé, en plus ça permettrais au navigateurs de tes visiteurs de les mettre en cache.
* Attention aux textes alternatifs de tes images :
<img src='./wp-content/themes/maxkv2/img/header4.gif' alt='maXK' />
Ici l'image porte le texte "MaX K Porfolio", le alt doit contenir, au moins, la même information que l'image.
* Dans le même registre, attention à l'utilisation abusive de title sur les liens :
<li class="cat-item cat-item-4"><a href="http://portfolio.maxk.fr/?cat=4" title="Voir tous les articles classés dans Coucher de soleil">Coucher de soleil</a></li>
<li class=""><a href="" title="Voir tous les articles classés dans Noir et blanc">Noir et blanc</a></li>
<li class=""><a href="" title="Voir tous les articles classés dans Panoramique">Panoramique</a></li>
<li class=""><a href="" title="Voir tous les articles classés dans Paysage">Paysage</a></li>
<li class=""><a href="" title="Voir tous les articles classés dans Scène urbaine">Scène urbaine</a></li>
Ici tes titles n'apporte aucune information concernant le lien, ils sont redondant entre eux et avec l'intitulé des liens auxquels ils se rapportent. Title, sur les liens, est un attribut
facultatif, parfois (souvent même) ne rien mettre c'est mieux.
* Attention à ta hiérarchie de titre. Le premier titre (Hn) qui apparait dans ta page est un H2, ce devrait être un H1. La hiérarchie de titres d'une page HTML doit être complète, logique et suivie ; cela veux dire qu'il ne peut pas y avoir de sauts entre les différents Hn (on ne passe pas de H2 à H6). Cette hiérarchie de titre est particulièrement importante pour les utilsateurs de lecteurs d'écran qui naviguent grâce aux titres. Si la hiérarchie est incomplète, le logiciel ne sait pas âller d'un titre à l'autre.
* Les images que tu présente sur ton site sont son contenu, de fait ce ne sont pas des images purement décorative, mais porteuse de contenu, elle ne
doivent donc PAS être appelées en CSS via background mais se trouver dans le HTML via la balise img. Tu pourra laisser l'attribut alt de ces images vide car les span qui présentent ces images seraient redondants.
* Il serait peut-être intéressant de faire en sorte que wordpress limite les classes miltiples sur les balises :
class="featured hentry p7 post publish author-admin category-noir-et-blanc category-paysage category-scene-urbaine tag- y2007 m10 d21 h14"
