3246 sujets

Critiques de vos sites: code et design

Un avis pour mon premier site "pro"

Salut Smiley smile

Je ne suis pas fan des splash pages en général, qui bloquent l'utilisateur avant d'entrer sur le site lui-même. La tienne se justifie par l'étape du choix des langues, qui semble légitime mais pourrait peut-être être zappée puisque tu proposes après la possibilité de changer de langue une fois dans le site. Par ailleurs, il est conseillé de ne pas utiliser les drapeaux des pays pour les boutons de changement de langue mais plutôt le nom de la langue elle-même, en toutes lettres.

J'ai parcouru rapidement ton code source, sans entrer dans le détail tu utilises beaucoup de styles CSS inline, il serait mieux de les externaliser tous dans une feuille de style afin de séparer nettement fond et forme.
Hello,

sur cette fameuse page de choix de langue un alt pertinent fait cruellement défaut sur ton image (ainsi qu'un titre à la place d'un simple paragraphe).

Plus globalement (après être allé sur la page d'accueil en français) et à part un H1 tu ne structures pas du tout ta page (cf. Respecter la sémantique XHTML sur openweb ).

Et en vrac :

* pourquoi cette vilaine table de mise en page (cf. Construire un site sans tableaux ) ?

* c'est une bonne idée (pour la relecture / maintenance) de donner des noms significatifs aux class et aux id ce qui n'est pas le cas de f12 ou r12.

* il manque les LABEL associés aux INPUT.

* tu affectes des class à presque tous les éléments alors qu'il suffit de les cibler en fonction de leur conteneur (cf. les sélecteurs et Comprendre l'héritage et la parenté des styles CSS ).

* comme dit précédemment il faut externaliser tes css ce qui, en plus de séparer le contenu et la présentation, facilitera la maintenance et permettra d'avoir les styles en cache plutôt que de les recharger à chaque fois.

* l'attribut alt sur une image permet de fournir un texte alternatif en cas d'images désactivées/indisponibles (ce qui est notamment le cas des lecteurs d'écrans) et donc alt="image" ne sert à rien Smiley cligne .

* dans la page en français tu as laissé xml:lang="nl" lang="nl"

* lorsqu'on désactive les images (en utilisant par exemple l'extension web developer de Firefox) la plupart des textes disparaissent : penser à indiquer une couleur de background différente de celle de color.

* un lien sans href ne sert à rien (par exemple <a name="debutcontenu">Xanthos</a>). Si c'est juste pour réutiliser un style utiliser les css.


voili, voilà pour moi. Smiley smile
Modifié par Heyoan (20 Mar 2009 - 10:44)
une petite remarque sur les boutons des menus : je trouve la couleur des caractères et celle du fond du bouton un peu trop proche et ça pourrai gêner certaines personnes


... et puis j'aime pas la couleur de fond du formulaire de contact (mais bon ... les goûts et les couleurs...) Smiley lol

sinon pour le reste je trouve la charte graphique agréable
Merci pour vos remarques très constructives. J'ai modifié la couleur d'arrière plan et ajouté le "alt" sur la page d'accueil.
a écrit :
pourquoi cette vilaine table de mise en page
Pour ce qui est des tableaux, je ne les utilise que pour l'affichage des données qui sortent de la BDD. Parfois, les tableaux sont quand même bien pratiques pour ce genre de données: on écrit une seule tr et le mysql_fetch_array fait le reste. Sinon, toute la structure de la page est positionnée en float.
Je vais quand même me plonger dans les tuto CSS.

Pour les couleurs, le client aime bien, et comme c'est lui le roi...
Salut,

De toutes les métas, la seule vraiment utile est celle de description car les moteurs de recherche se basent sur celle-ci. Il faudrait alors la remplir plus spécifiquement avec 250 caractères maximum, car juste 'sellerie' à l'intérieur n'est pas assez pertinent Smiley cligne .

Sinon pour la gestion des langues, les moteurs de recherche vont avoir du mal à savoir si ton site est en français ou en néerlandais puisque quand il arrive sur le site il voit du français ET du néerlandais. Donc tout sera confus pour lui..., peut être réserver un deuxième nom de domaine en .nl qui en fonction de l'adresse fournie met le site dans la langue désirée. Pas forcément très facile à mettre en place mais cela te permettra d'avoir un référencement optimisé car quand on arrivera sur la page d'accueil on trouvera le contenu réel de ton site. De plus un nom de domaine ne coûte pas trop cher ^^.

Sinon pour l'aspect graphique, quand tu affiche les produits sous forme de liste avec leurs prix, tu met un contour autour de la case avec un fond blanc, on a envie de cliquer et de changer la valeur tellement ça ressemble à un input.

Bonne journée ^^
Pour ce qui est de la gestion des langues en page d'accueil (si tu tiens à la garder), je ferais comme ceci (vue que la page est déclarée en français) :

<p class="c12"><img src="images/xanthos-index.gif" alt="" /></p>
	<p class="pied-menu" lang="nl" xml:lang="nl" style="text-indent : -50px; font-size : 1.2em"><a href="nl-a-accueil.php" hreflang="nl">Welkom op de website van de ruitershop Xanthos - verder in het nederlands</a></p>


En bref : suppression du contenu du alt qui ne sert à rien (l'image est décorative et l'information est redondante), ajout de xml:lang sur le <p> en nl et ajout de hreflang="nl" sur le lien. Smiley cligne
Une remarque concernant un choix que vous avez fait (assez éloignée des question de code et d'esthétisme, mais bon ...) concernant le nom des fichiers. Ils sont tous en français : accueil.php, images/* ...

Autant sur un site purement francophone ça ne dérange pas ; autant, quelqu'un parlant Néerlandais ne pourrait-il pas être un peu gêné d'avoir une URL francisée ? Tout dépend du public visé, vous connaissez vos clients mieux que moi, mais dans le cas d'un site multi-lingue, l'anglais ne serait-il pasplus adapté ?

Je ne remets pas votre choix en question, j'apporte simplement ce que je perçois de plus que les remarques précédentes.

Amicalement,
Tahmahal.
Les noms des fichiers sont structurés ainsi pour des raisons techniques. Ils sont analysés par un script PHP ce qui me permet :
- de créer l'url de la page équivalente dans l'autre langue plutôt que de ramener le visiteur sur une page par défaut en cas de changement de langue
<a href="
<?php
//récupérer le nom du script
$nomfichier = basename($_SERVER['PHP_SELF']);
//On prend le 2 premiers caractères
$newlangue = substr($nomfichier, 0, 2);
//On change les 2 premiers caractères
$newlangue = str_replace($newlangue, "fr", "nl");
//On garde le reste dans une varialble
$finscript = substr($nomfichier, 2);
$newlangue .= $finscript;
print ($newlangue);

//Récupérer la query string si il y en a une
if ($_SERVER["QUERY_STRING"]<>"") print ("?".$_SERVER["QUERY_STRING"]);
?>
" title="nederlands"> <img src="images/nl.png" alt="nederlands" /> </a>


- d'ouvrir le menu correspondant la bonne catégorie.
<?php
//déterminer le nom du script et du menu
$nommenu = $_SERVER['SCRIPT_NAME'];
$nommenu = basename($nommenu);
$nommenu = substr($nommenu, 3, 1);

?>




<p class="<?php if ($nommenu =="a") print("menu-actif"); else print("menu-inactif"); ?>">
Sujet clos
Aller à