Bonjour,
Le splash screen est totalement inutile et donne une image très ringard du site avant même de le voir ("y a encore des gens qui font des splash ?").
Les robes sont très belles, le site sobre les met plutôt bien en valeur, par contre (même si je ne pense pas que ce soit ta responsabilité, ça fait quand même partie du tout) les photos pas : elles sont trop "ternes", les quelques couleurs ne "flashent" pas, un peu de post-prod ne ferait pas de mal.
Dans les pages de contenu court, c'est un peu dérangeant d'avoir une sidebar aussi longue et pas de fond continu pour la zone de contenu, des colonnes factices rendrait mieux.
Il y a un bug d'affichage sur les fiches des robes, s'il y a plusieurs photos elles sortent de leur cadre (ajouter un overflow sur le cadre ?), idem pour la page "qui sommes nous", les images sortent du cadre (parfois, il faudrait, je pense rajouter les dimensions des images).
Le title est identique sur toutes les pages, c'est une très mauvaise pratique : 1. pour Google qui aime bien les titre et description de page différents et uniques ; 2. pour le visiteur qui voudrait bookmarquer plusieurs pages et qui ne saura plus les différencier.
Les mots clés en bas de page, c'est bien joli, mais ça n'apporte rien au visiteur et ça peut être considérer comme une technique de tentative de positionnement abusive
Pour le code, je me base sur la page « qui sommes nous » (il manque d’ailleurs un « ? ») :
* Il y a deux erreurs de code assez grave : alt non présents sur deux images porteuses de contenu très important.
* Tu ne précise pas la langue du document, il faut ajouter lang sur html et une meta Content-Language (sauf si le serveur envoie déjà cette information).
* Quel est l’intérêt de demander à IE8 et supérieurs de se comporter comme IE7 ? C’est totalement idiot de dire à un navigateur qui interprète presque correctement les CSS2 de ne pas le faire ! Si tu as des problèmes pour gérer les deux versions d’IE, il est préférable d’utiliser des commentaires conditionnels. Pour rappel : la meta X-UA-Compatible a été ajoutée pour faciliter la migration des vieux sites codés pour IE5.5 pas pour des sites tout neufs !
* Les scripts doivent, de préférence, être placés en fin de document.
* Il est dommage d’utiliser des styles en lignes alors que tu fais appel à une feuille de styles externe.
* Ta
hiérarchie" rel="noopener" >http://www.alsacreations.com/astuce/lire/952-bien-construire-sa-hirarchie-de-titres.html]hiérarchie des titres n’est pas correcte : elle commence à h2 alors qu’elle devrait commencer à h1 et ne pas comporter de sauts.
* J’étais dans le juste pour les dimensions d’images manquantes.
* Le alt d’une image doit reprendre le même contenu que celui porté par l’image. Sur l’image points-de-vente.jpg qui porte le texte « Recherche de magasin », le alt « Points de vente Annie Couture, en France et à l'étranger » n’est pas vraiment identique...
* Pou l’image robe-du-mois.jpg qui a pour texte alternatif : « Annie Couture, robe du moment – Alia », un alt vide serait préférable, en effet, toute l’information (sauf le « Annie Couture », qui ressemble fortement à une tentative de référencement/positionnement hasardeuse et abusive) se trouve dans le texte encadrant l’image. Le contenu du alt devient redondant.
* Les h3 du footer ne sont pas des h3, dans la hiérarchie actuelle de la page, ils sont des h2.
* Attention au mailto, c’est le meilleur moyen de récupérer du spam
Ce qui me dérange le plus dans ce code c’est le replacement de class par style sur les éléments (principalement flottants)...
pour avoir suggérer une chose pareille !