Bonjour,
Le design est sympa, mais l'animation du header est vraiment brutale (du moins sur ma machine) et rapide, ça me donne mal à la tête.
Il manque un élément extrêmement important sur la page d'accueil (et ailleurs) : la localisation de ton amie ou du moins sa zone "d'intervention", pour un client potentiel, c'est important.
Pour le code :
* C'est loin d'être aussi propre que le design
33 erreurs de code avec un doctype transitionnel, c'est énorme. Il faut corriger cela.
* Tu as des metas inutiles : Robots (deux fois présente, deux fois inutile car présentant ses valeurs par défaut), Keywords (qui ne sert strictement plus à rien depuis un paquet d’années), description (car avec la description actuelle elle n’apporte pas un plus, cette meta a pour but d’attirer le visiteur, par le robot).
* Sur les pages internes du site, on perd, dans le title, le nom du site, ça pourrait être embêtant.
* Il est préférable de placer les scripts en fin de document et de les regrouper en aussi peu de fichiers que possible (1 fichiers + scripts provenant de sources externes, dans l’idéal).
* Title n’as rien à faire sur une image !
* Title est un élément pouvant être utilisé sur les liens pour apporter une information nécessaire à la bonne navigation du visiteur et supplémentaire à l’intitulé du lien. À ton avis, est-ce que le title est nécessaire ici :
<a href="http://www.valoriz.com" title="Valoriz"><img src="/img/logo.gif" alt="Valoriz" title="Valoriz"/></a>
Le title ne doit pas non plus prendre le visiteur pour un idiot :
<a href="/realisations.html" title="realisation decoration valoriz">Voir toutes les réalisations</a>
* Contrairement à title, alt, lui est obligatoire sur les images !
* Ceci est un lien vide, et donc, qui, dans l’absolu, n’existe pas :
<a href="http://www.valoriz.com/blog"><img src="/img/btn_blog.gif" /></a>
* La div <div class="header_r"> est inutiles puisqu’elle ne contient que la liste qui constitue le menu.
* L’utilisation d’une div class clear pourrait être évitée.
* Ta hiérarchie des titres est fausse, elle commence à h2 au lieu de h1...
* L’utilisation de <br> successifs (et sans consistance dans leur syntaxe) est le signe qu’un nouveau paragraphe aurait du être créé et/ou qu’il faut jouer sur les marges.
* Alt n’a strictement rien à faire sur un lien ! (et je rajouterais bien quelques « ! » pour signifier l’horreur que m’inspire cette vision)
Il faut vraiment que tu revois ton code, des erreurs pareilles c’est...
