Avis sur mon site : www.tyforn.com

Bonjour,

Pour le centrage, il faut mettre les margin left et right à auto.

Par exemple :

#wrapper{
	text-align:left;
	width:960px;
	background:#014c93 center;
	position:relative;
	border:3px solid #014c93;
	[b]margin:20px auto 0 auto;[/b]
}

ah ouai cool ca marche!! quand je pense au temps passé a essayer de comprendre pouquoi ca fonctionnait pas, et toi tu me sers la reponse en moins de 2 min!! je te remercie vraiment...

Bonjour,

Le coup du centrage effectif sur IE et non FF m'a mis la puce à l'oreille. Ce n'est pas normal que quelque chose fonctionne sur IE et pas sur FF Le problème vient du fait qu'à cause de la déclaration XML en début de document (première ligne) IE passe en mode quirks (en gros le mauvais mode de rendu). Supprime cette première ligne de ton code, elle ne sert strictement en rien (puisque tu n'utilise pas d'XML, malgré ton doctype XHTML...), tu risque par contre de devoir recorriger certains problèmes d'alignements mais ce sera une bonne chose.

Pour le design, c'est effectivement simple, mais ça passe bien. Attention tu utilise des images qui une un copyright inscrit dessus (la croix sur couché de soleil à gauche), j'en déduis que tu n'as pas les droits sur ces images. Tu te mets donc dans l'illégalité en la diffusant sur ton site. TU as donc deux solutions : demander à l'auteur l'autorisation d'utiliser son image (mais en général peu de photographe donnent cette autorisation après que l'image ait déjà été utilisée...) ou utiliser une photo qui t'appartiens.

Pour le code, il y a effectivement quelques petites choses à revoir, mais ce n'est pas trop catastrophique :
* Tu as 25 erreurs de code, à corriger !

* Le titre de la page d'accueil est bien trop long, d'ailleurs il ne s'affiche pas complètement dans mon navigateur. Cela ne sert à rien d'avoir un titre aussi long. "Ty Forn (Plogoff) : Restaurant-Pizzeria" suffirait amplement. De plus dans ce titre (et certainement dans le reste du site) attention à la position des espaces !

* La meta keyword est totalement inutile, car inutilisée par les moteurs de recherche, et peut donc être supprimée.

* Idem pour distribution

* Idem pour robots, all étant sa valeur par défaut, cela n'apporte rien (si ce n'est une ligne de code supplémentaire, de le préciser.

* Les tailles de polices sont précisées en unité fixe, ce qui empêche le redimensionnement des caractères sous IE6 et 7; il serait préférable d'utiliser des unités relatives (em ou %).

* Les éléments vides en HTML sont TOUJOURS des aberrations :

<div id="banniere"></div>

Plus encore lorsqu'il comporte une image porteuse de contenu placée en fond. Les images qui ont du texte dessus doivent TOUJOURS se trouver dans le code HTML accompagnée d'un attribut alt dûment remplis (avec le même contenu, au moins). À ton avis que se passe t'il si les images ne peuvent être chargées (pour plein de raisons) ? Ton site n'a plus de titre, plus rien pour l'identifier... C'est particulièrement embêtant.

* Cependant, l'attribut alt ne doit pas toujours être remplis, il peut tout à fait être vide :

<a href="viandes.html"><img src="theme/hermine.png" alt="Viandes"/> NOS VIANDES </a>

Ici et dans les autres liens de ce menu, le alt spécifié pour l'image ne correspond absolument pas à l'image qui est purement décorative, de plus le contenu du alt est redondant avec le contenu qui le suis. Ces alt doivent être laissés vide : alt="".

* Même remarque que pour le header pour ces liens :

<a href="poissons.html"> </a>

Des éléments vides sont des éléments qui ne servent à rien. Des images qui doivent servir de support de clic doivent se trouver dans le code HTML avec un attribut alt, sinon le lien est vide.

* Tous tes intitulés de liens et certains titres sont en majuscule dans le code, il serait préférable de spécifier cette mise en majuscule via CSS, ce qui faciliterait une future mise à jours, si tu décide de ne plus les mettre en majuscule.

* La hiérarchie des titres du site est incorrecte. Elle commence par H2, alors qu'elle devrait, toujours, commencer par H1. La hiérarchie de titres doit être correcte et ne pas comporter de sauts.

* Il y a effectivement un peu trop de div à partir de la partie "bienvenue" :

<div id="bienvenue">
  <div class="corps"></div>
</div>
<div id="actu">
  <div class="corps">
    <div id="news"></div>
  </div>
</div>
<div id="saviezvous">
  <div class="corps"></div>
</div>

Tu pourrais largement te contenter de :

<div id="bienvenue"></div>
<div id="actu"></div>
<div id="saviezvous"></div>

* Il manque quelques labels sur ton formulaire de contact, qui est d'ailleurs totalement riquiqui. Il faudrait agrandir la zone de saisie du message. Tu as également oublié un champ e-mail, sans lui, tu va avoir du mal à répondre aux questions posées...

* Toujours au sujet du formulaire de contact, celui-ci n'est absolument pas protéger contre le spam, et affiche l'adresse mail en clair, il faudrait te renseigner sur le traitement en PHP des formulaires et sur les captcha inversé.

* La section liens utiles est mal affichée sur IE6.

* Les livres d'or, c'est assez has-been...
Modifié par Laurie-Anne (30 Dec 2009 - 09:20)

Ok un grand merci a toi Laurie-Anne, je vais du coup pouvoir me pencher sur les différents points que tu as soulevé.