3246 sujets
Critiques de vos sites: code et design
Hello,
Je trouve ça plutôt propre, juste quelques remarques :
Le header en positionnement fixe déstabilise un peu selon moi, vaudrait mieux rester sur quelque chose de plus classique (ie : positionnement relatif), ça gênera moins l'internaute lambda selon moi.
Il y a un espace de 1 ou 2 pixels entre le header et le menu, ça fait bizarre quand on scolle de voir le contenu derrière.
Ta typo grise est peut-être un peu trop proche du teint du jaune derrière, je pense que quelqu'un qui a les yeux fatigués aura un peu de mal à lire.
Les logos sont un peu en bordel dans la page partenaires, tu devrais essayer de ranger ça mieux.
Sinon je trouve ça clair et les infos sont faciles à atteindre.
Modifié par Skoua (02 Jul 2009 - 13:17)
Je trouve ça plutôt propre, juste quelques remarques :
Le header en positionnement fixe déstabilise un peu selon moi, vaudrait mieux rester sur quelque chose de plus classique (ie : positionnement relatif), ça gênera moins l'internaute lambda selon moi.
Il y a un espace de 1 ou 2 pixels entre le header et le menu, ça fait bizarre quand on scolle de voir le contenu derrière.
Ta typo grise est peut-être un peu trop proche du teint du jaune derrière, je pense que quelqu'un qui a les yeux fatigués aura un peu de mal à lire.
Les logos sont un peu en bordel dans la page partenaires, tu devrais essayer de ranger ça mieux.
Sinon je trouve ça clair et les infos sont faciles à atteindre.
Modifié par Skoua (02 Jul 2009 - 13:17)
Bonjour,
Première remarque, le code ton site n'est pas valide, le validateur trouve 13 erreurs, certaines dûes à la capitalisation de certaines balises ou attribut. Tu as choisi un doctype XHTML 1.0 Strict, le code HTML doit donc être rédigé en minulcules uniquement.
Seconde remarque avant de passer au code, ton site est déformé sous IE6, qui est encore utilisé par une bonne partie des internautes (contre leur gré ou pas).
* Tu utilise quelques tableaux de présentation, peut-être pourrais-tu les transposer en div.
* La hiérarchie de titre de ton site est plus que bancale, tu passe de h3 à h5 puis h1... Tu dois absolument conserver une hiérarchier logique (qui va de h1 à hn sans "saut"). Il ne faut pas se reposer que le style graphique d'un hn pour son choix, mais bien par sa hauteur hiérarchique.
* Ton menu déroulant que tu à je suppose voulu sans JavaScript ne fonctionne pas sous IE6. C'est fort embetant... Un menu déroulant DOIT intégrer du JS pour une meilleurs intéropérabilité, mais également accessiblité au clavier. Florent explique ça très bien.
* Le lien "Haut de Page" en bas de page renvoie vers le début du contenu et non vers le haut de la page. Il faudrait soit changer l'intitulé du lien, soit son pointage, histoire de ne pas perturber les visiteurs (qui veux aller en haut de la page).
* Tu ne peux pas mettre du texte dirrectement dans une div, il faut passer par un <p>aragraphe et éviter les <br /> superflus.
* De même plutôt que de simuler une liste en commençant une ligne avec "-" et en la finissant par un retour à la ligne forcé, utilise <ul> et <li>. Chaque balise doit être utiliser pour ce pour quoi elle est designée.
* Pense à rajouter une meta description sur chaque pages, avec, au possible, une description unique pour chaque pages.
* Sur ta page contact, fais attention ! Laisser toutes les adresses mail en clair sur la page, c'est attirer les spam. Un formulaire de contact serait déjà un peu plus sûr.
En conclusion, ton balisage est très hétérogène, certaines portions de textes sont correctement balisées, d'autre non, il faut homogénéiser tout cela.
Première remarque, le code ton site n'est pas valide, le validateur trouve 13 erreurs, certaines dûes à la capitalisation de certaines balises ou attribut. Tu as choisi un doctype XHTML 1.0 Strict, le code HTML doit donc être rédigé en minulcules uniquement.
Seconde remarque avant de passer au code, ton site est déformé sous IE6, qui est encore utilisé par une bonne partie des internautes (contre leur gré ou pas).
* Tu utilise quelques tableaux de présentation, peut-être pourrais-tu les transposer en div.
* La hiérarchie de titre de ton site est plus que bancale, tu passe de h3 à h5 puis h1... Tu dois absolument conserver une hiérarchier logique (qui va de h1 à hn sans "saut"). Il ne faut pas se reposer que le style graphique d'un hn pour son choix, mais bien par sa hauteur hiérarchique.
* Ton menu déroulant que tu à je suppose voulu sans JavaScript ne fonctionne pas sous IE6. C'est fort embetant... Un menu déroulant DOIT intégrer du JS pour une meilleurs intéropérabilité, mais également accessiblité au clavier. Florent explique ça très bien.
* Le lien "Haut de Page" en bas de page renvoie vers le début du contenu et non vers le haut de la page. Il faudrait soit changer l'intitulé du lien, soit son pointage, histoire de ne pas perturber les visiteurs (qui veux aller en haut de la page).
* Tu ne peux pas mettre du texte dirrectement dans une div, il faut passer par un <p>aragraphe et éviter les <br /> superflus.
* De même plutôt que de simuler une liste en commençant une ligne avec "-" et en la finissant par un retour à la ligne forcé, utilise <ul> et <li>. Chaque balise doit être utiliser pour ce pour quoi elle est designée.
* Pense à rajouter une meta description sur chaque pages, avec, au possible, une description unique pour chaque pages.
* Sur ta page contact, fais attention ! Laisser toutes les adresses mail en clair sur la page, c'est attirer les spam. Un formulaire de contact serait déjà un peu plus sûr.
En conclusion, ton balisage est très hétérogène, certaines portions de textes sont correctement balisées, d'autre non, il faut homogénéiser tout cela.
Bonjour djeimel,
Tout d'abord, le design de ton site est sympathique et agréable à l'œil.
Néanmoins, ton code CSS n'est pas valide, il contient une erreur.
Essaie de la corriger
.
Ensuite, je trouve que l'espacement entre les lignes de tes paragraphes est trop petit à mon goût.
Tu peux remedier à l'espacement entre les lignes via la propriété CSS suivant :
Bonne continuation pour ton site.
Modifié par jQz (02 Jul 2009 - 18:27)
Tout d'abord, le design de ton site est sympathique et agréable à l'œil.
Néanmoins, ton code CSS n'est pas valide, il contient une erreur.
Essaie de la corriger
.Ensuite, je trouve que l'espacement entre les lignes de tes paragraphes est trop petit à mon goût.
Tu peux remedier à l'espacement entre les lignes via la propriété CSS suivant :
line-height: Une hauteur, exprimée en pixels ou en pourcentage.
Bonne continuation pour ton site.
Modifié par jQz (02 Jul 2009 - 18:27)
Le contraste entre la couleur de police principale et le fond du site est beaucoup trop faible pour des yeux fatigués. il serait beaucoup plus confortable d'accentuer la différence entre les deux.
J'ai noté un autre point important : La zone réactive des liens dans le menu est beaucoup trop réduite. Je veux dire par là que la zone "cliquable" de ton menu est trop fine et demande une certaine adresse à la souris que tout le monde n'a pas.
De même, à propos du menu : les sous-menus s'effacent aussitôt que l'on sort de leur zone. Assez désagréable si on combine avec la zone réactive réduite. Il serait intéressant de laisser un temps (genre 2 secondes) à la sortie du sous menu avant de le fermer.
Le champs de saisie pour la newsletter est beaucoup trop proche du menu, même remarque que précédemment, il nécessite une certaine adresse pour, par exemple, ne pas cliquer involontairement sur "partenaires".
Encore une dernière chose : Il y a une petite incohérence à mon avis qui concerne la séparation des articles.
Visiblement, chaque article est séparé par une bande bleue. Mais le titre "actualité" aussi ? Ou bien... il s'agit d'une séparation de l'article du dessous ? Pas évident.
La rubrique "Festival" a l'air de délirer un peu, le menu passe par dessus les articles et se retrouve décalé.
J'ai noté un autre point important : La zone réactive des liens dans le menu est beaucoup trop réduite. Je veux dire par là que la zone "cliquable" de ton menu est trop fine et demande une certaine adresse à la souris que tout le monde n'a pas.
De même, à propos du menu : les sous-menus s'effacent aussitôt que l'on sort de leur zone. Assez désagréable si on combine avec la zone réactive réduite. Il serait intéressant de laisser un temps (genre 2 secondes) à la sortie du sous menu avant de le fermer.
Le champs de saisie pour la newsletter est beaucoup trop proche du menu, même remarque que précédemment, il nécessite une certaine adresse pour, par exemple, ne pas cliquer involontairement sur "partenaires".
Encore une dernière chose : Il y a une petite incohérence à mon avis qui concerne la séparation des articles.
Visiblement, chaque article est séparé par une bande bleue. Mais le titre "actualité" aussi ? Ou bien... il s'agit d'une séparation de l'article du dessous ? Pas évident.
La rubrique "Festival" a l'air de délirer un peu, le menu passe par dessus les articles et se retrouve décalé.