Bonjour,

Globalement c'est pas mal, on voit tout de suite ce que vend le site (ceci dit, elle casse les prix ta traductrice... Smiley ohwell ).

Deux remarques :
- Tu pourrais soigner le code de tes tableaux (summary, ...)
- Le message à l'attention des utilisateurs IE6 / IE7 : ça servira juste à faire fuir d'éventuels clients qui ne connaissent pas grand chose à la technique.
Bonjour,

Ericf a écrit :
Le message à l'attention des utilisateurs IE6 / IE7 : ça servira juste à faire fuir d'éventuels clients qui ne connaissent pas grand chose à la technique.
+1 D'autant plus que le message (couleurs, taille) est particulièrement agressif. Ce genre de message me dit, à moi, je ne suis pas capable de faire un site qui passe sous IE6 alors j'essaye de me dédouaner en racontant des cracks. Après tout ce n'est quand même pas difficile de faire une feuille de style corrective pour réduire de quelques px un des blocs pour IE6... ps. : considérer IE7 comme obsolète c'est un peu fort...


Niveau design, c'est propre et clair (malgré le défaut d'affichage pour IE6).

Le mouvement sur le logo au survol est plus perturbant qu'autre chose et il n'apporte rien.

Les textes sont un peu fins pour une lecture confortable.


Niveau code :
* Tu as oublié une fermeture de tag (meta).

* La langue du document n'est pas déclarée (la meta content-language ne suffit pas). Il faut ajouter un attribut lang sur html.

* Le titre de la page est bien trop long.

* La meta description n'apporte pas vraiment d'information intéressante. C'est en partie grâce à cette meta que peut se faire le choix de visiter un site plutôt qu'un autre depuis un moteur de recherche.

* La meta robots a ses valeurs par défaut, elle ne sert donc à rien.

* Il est préférable de placer les scripts en fin de document afin de ne pas ralentir le chargement du contenu de la page.

* Une CSS corrective prendrait moins de place dans le code que l'avertissement.

* l'utilisation de <q> pour placer une tagline/slogan sous le h1 est mauvaise. <q> sert à baliser les citations en ligne, ce n'est pas le cas ici. Il faut utiliser <p>.

* Il faut baliser les changements de langue dans la page.

* title, sur les liens sert à apporter une information supplémentaire à l'intitulé du lien et nécessaire à la bonne navigation du visiteur. Un title qui reprend exactement l'intitulé du lien est un title inutile.

* Ton menu est inaccessible, je corrige, tous les liens en image sont inaccessibles : si les images sont désactivés, mais les CSS toujours présents (cas typique du problème serveur ou de la connexion qui déconne), plus de menu ! Tu utilises déjà du JavaScript sur le site (et pour le menu), pourquoi ne pas avoir placé les images porteuses de contenu dans le code HTML plutôt que dans le CSS ?

* Les h4 dans le footer ne sont pas justifiés et rendent ta hiérarchie des titres incorrecte. Ces titres doivent être des h2, si tu considère qu'ils sont sous le titre général, ou h1 sinon.

En soi, le code est propre et pro, mais il y a quand mêmes des détails gênants d'un point de vue accessibilité/interopérabilité qui doivent être corrigés pour une prestation pro.
Bonjour,
Graphiquement c'est simple et bien fait, mais je laisserais personnellement une petite marge "de sécurité" sur les côtés (je suis en résolution 1024), pour une raison esthétique, mais aussi pour prévoir pour ceux qui ne naviguent pas toujours en plein écran (ou fenêtre agrandie) ou d'autres éventualités non prévues.

Sinon, j'ai un coup de coeur pour l'identité graphique/logotype, vraiment superbe et bien pensé Smiley love

Je n'ai pas vu le code en détails, je laisse le soin aux spécialistes. EDIT : Ah bah Laurie-Anne a déjà fait !

Bonne journée.
Modifié par darkstar2023 (21 Sep 2010 - 16:12)
Merci à tous pour vos conseils Smiley cligne

J'ai corrigé un certain nombre de problèmes. Le site s'affiche désormais correctement avec IE6, je dois encore tester IE7.

a écrit :
* Il faut baliser les changements de langue dans la page.


Peux-tu me dire si j'ai procédé correctement ? Merci.

a écrit :
* Ton menu est inaccessible, je corrige, tous les liens en image sont inaccessibles : si les images sont désactivés, mais les CSS toujours présents (cas typique du problème serveur ou de la connexion qui déconne), plus de menu ! Tu utilises déjà du JavaScript sur le site (et pour le menu), pourquoi ne pas avoir placé les images porteuses de contenu dans le code HTML plutôt que dans le CSS ?


J'ai compris le problème. Les images sont dans le CSS car j'utilise la technique des portes coulissantes. Si je place les images directement dans le code HTML, mon animation javascript/jQuery fonctionnant sur la balise <a> ne foncitonne pas si je la place sur la balise <img>.

a écrit :
* Les h4 dans le footer ne sont pas justifiés et rendent ta hiérarchie des titres incorrecte. Ces titres doivent être des h2, si tu considère qu'ils sont sous le titre général, ou h1 sinon.


Corrigé.

Merci pour l'aide Smiley smile
Modifié par alw (24 Sep 2010 - 07:38)