Bon, il est pas 100% fini mais e gros est fait.
C'est un mec qui me paye pour ce site donc je voudrais savoir ce que vous en pensez, que ce soit au niveau design que sécurité ou autre...

C'est pas encore validé par le W3C pour le XHTML mais c'est pour bientot.

Mon site

J'attends vos critiques avec impatience
Modifié par djedie (19 Jul 2005 - 11:22)
Bonjour,

Je vais sans doute te paraitre très sévère mais malheureusement je ne vais te dire que l'exacte vérité.
Je ne parles pas du design, je n'ai aucune compétence en la matière, en revanche le code est absolument catastrophique.

Tu à fait l'erreur, sans préjuger de tes compétences (ce n'est pas du tout mon propos) de developper ce site "à la volée", c'est en tous cas ce que raconte ton code, en choisissant les moyens les plus rapides pour "faire tourner" la maquette.

Le résultat est un mélange de html et de xhtml, de mise en pages CSS et de mise en forme HTML dans une structure partiellement invalide noyée dans une masse de div inutiles.

Le travail à faire pour rendre cette structure et ce code acceptable est très important et pose la question de savoir si il ne serait pas plus rapide de tout reprendre de zéro et de considérer ce site comme une simple maquette.

Le soucis, évidemment, c'est que tu à déjà fait "tout le site" et qu'en conséquence chaque nouveau problème que tu vas rencontré va être considérablement amplifié passant de simple correction à de la réécriture difficile.

La première chose que tu dois faire est de corriger le plus flagrant, assainir autant que faire se peut et surtout procèder par étapes, tu est dans l'exacte configuration d'un "vieux site", dégradé par des interventions mutliples et le temps et qu'il te faudrais normaliser, c'est le plus difficile.

Je n'ai fait que parcourir ton code, voici une liste de ce qui me parait parait urgent de faire AVANT de passer en production, après ce sera trop tard :

1. Supprimer ce frameset ridicule et inutitle (mais bon on peut supposer que c'est juste pour visualiser le maquette)

2. Faire un choix : HTML ou XHTML, si tu ne maitrise pas bien XHTML et la mise en page CSS n'ai aucun état d'âme à utiliser du html, du html bien formé est bien meilleur que du Xhtml bidon simplement pour être dans l'air du temps.

Même remarque pour la mise en page CSS, le mélange entre du positionnement CSS et l'usage de balise center, height ou de br aux fins de structure montre que tu à eut des problèmes à résoudre "absolument".

De la même manière que pour le html, le recours à des tables de layout peut être une solution pour patienter le temps que tu acquières une maitrise suffisante du positionnement CSS.

Le risque que tu fais prendre à ton travail est énorme dans le cas contraire, car, en production et avec le temps, il deviendras simplement impossible de l'améliorer.

Il faut aussi que tu te poses la question de savoir si tu à "confiance" dans la solidité et la pérénité de ta mise en page. Ton code dis que non, tu n'à pas confiances et dans ce cas là poses toi la question des moyens qui seront les tiens quand ton "client" va te demander une modification.

La structure de tes pages est ton futur outil de travail, si il est mal fichus tu ne pourras faire que du mauvais travail.

3. Si le choix c'est du Xhtml remplacer le doctype 1.1 dont tu n'à aucune utilité et opter pour du 1.0.

4. Dans tous les cas : ces points sont extrèmement urgents si tu veux pouvoir corriger tes pages dans le temps, car ça va demander du temps :

- remplacer toutes les balises dépréciées de style et de mise en forme par leur équivalents CSS (font, i, b... ) et plus particulièrement les balises center ou les attributs nowrap qui vont avoir des effets très désagréables sur ta mise en page.

- corriger les graves erreurs de structures comme les balises non fermées et les tables mal formées, les défaut d'imlémentation (tous les messages d'erreur qui comment par "ce type d'élément n'est pas autorisé ici..").
- Commencer à essayer de supprimer les divs inutiles (par exemple pour soutenir l'aspect d'un formulaire, ce que tu fais avec le div tu le fera aussi bien et plus facilement en stylant directement le formulaire) et utiliser les élements pertinents, par exemple remplacer ton div pour les liens par un liste ul.

Ce n'est qu'une fois que tu aura terminé ce travail que tu pourras savoir si ta structure tiens le coup et remplis son rôle et si ce site peut passer en production.

Bon évidemment le résultat n'est pas très brillant, en revanche il y à certaines choses de qualité dans ton code qui montre ta motivation, donc ce n'est maintenant plus qu'une question de temps... Smiley smile

JP

Ps: Et tu trouveras toujours ici, j'en suis persuadé de très bon conseils... Smiley cligne
Modifié par jpv (19 Jul 2005 - 05:08)
Salut,

Rapidement 2 captures.
Ce site est tout simplement invisitable sous windows NT en écran 800*600

Capture1
Capture2 (j'ai viré le nom de la société d'où ce rectangle blanc)

Par contre, je ne reviendrai pas sur ce topic du fait du sujet de ton site.
Je suis spécialiste en nutrition et ton site porte sur une société en marge.
Dans le monde de la nutrition, elle fait partie des sociétés dites "diététiques commerciales qui vendent n'importe quoi, à n'importe qui, à n'importe quel prix".
Aucune preuve scientifique n'est apportée ni est apportable à leurs arguments commerciaux.

La nutrition, c'est apprendre aux gens à manger donc ... pas plus d'aide de ma part pour ce site ; c'est un problème personnel.
(tout comme j'aiderais volontier Linux mais pas Microsoft Smiley cligne )

Je précise que cela n'a rien de personnel avec toi.
Mais il est évident que si je faisais des sites pour les autres, j'aurais refusé ce contrat Smiley cligne
Modifié par dieteticien (19 Jul 2005 - 10:40)
a écrit :
1. Supprimer ce frameset ridicule et inutitle (mais bon on peut supposer que c'est juste pour visualiser le maquette)

en fait c'est la redirection du site qui est comme ca. Ca permet de garder formeetforme.net dans l'adresse du navigateur et non pas le nom de l'hebergeur. Mais si tu pense que je dois le virer ...

Pour le reste je vais faire du XHTML + CSS donc je vais virer tout ce qui est HTML.

a écrit :
le mélange entre du positionnement CSS et l'usage de balise center, height ou de br aux fins de structure montre que tu à eut des problèmes à résoudre "absolument".


j'ai virer les <center>, par contre les balises height sont présentes dans le tableau (la ou il y a le contenu) car il y a des <tr> et <td> qui demandent "height" ou "width" pour etre bien formés. Je vais mettre ca dans le CSS alors.

Bar contre j'ai souvent besoins de <br /> pour faire la mise en forme du texte, mais ca ej ne vois pas ou est le mal ?

a écrit :
e site est tout simplement invisitable sous windows NT en écran 800*600

Oui je sais, je vais faire une image qui retreci avec la resolution, ca sera moins moche.

Je continu les modif en suivant vos conseils. Continuez les critiques entre temps.

Merci beaucou p a jpv pour ses conseils.
Je repost dès que je fais une autre maj du site.
Bravo !

Maintenant il te reste à choisir un DOCTYPE adapté Smiley cligne :

http://www.pompage.net/pompe/declarations/

Et ajouter quelques « meta » importants tels :

<meta http-equiv="Content-Language" content="fr" />
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1" />
<meta http-equiv="Content-Style-Type" content="text/css" />
ok C est fait.
Mais pour le DOCTYPE, pourquoi le changer?
Je fais du XHTML 1.1 donc je peux laisser :
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
nan?
Là, tu fais du XHTML traité en tant que "text/html".

Le XHTML 1.1, il doit être servi en tant que "application/xhtml+xml".

<edit />
En fait, la très grande majorité des pages arborant un DOCTYPE XHTML ne sont en réalité que du HTML reformulé en XML.

XHTML 1.0 : Le langage de balisage hypertexte extensible
Une reformulation de HTML 4 en XML 1.0


http://www.la-grange.net/w3c/xhtml1/
Modifié par Stephan (20 Jul 2005 - 00:01)
ok
j ai mis :

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" 
"http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">


comme ca c'est nickel !

par contre pour la validation CSS ca ne passe pas car j ai utilisé :
padding:10px;
   -moz-border-radius-bottomleft: 10px;
   -moz-border-radius-bottomright: 10px;
   -moz-border-radius-topleft: 10px;
   -moz-border-radius-topright: 10px;
   border-radius: 1em;

pour avoir des cadres arrondies ...
y a une feinte possible?
bah... il y a des feintes, mais pourquoi ne pas assumer ton choix ?

Ta feuille de style est-elle valide sans les -moz-border ?

Si oui, alors tu sais pourquoi le validateur gueule. C'est tout.

À moins que tu ne souhaites afficher une jolie image « Valid CSS checked » Smiley biggol

<edit />
Lire ce post de Laurent Denis à ce sujet :
http://forum.alsacreations.com/topic.php?fid=4&tid=5368#p46740
Modifié par Stephan (20 Jul 2005 - 00:11)
Il y a des problèmes d'affichage dans IE je pense.

Et en général bcp trop d'espace blanc entre les éléments... (Bande, menu, titre, cadre...)
Du coup, même avec une grande résolution on ne voit qu'une toute petite partie du contenu.

Le tout est aussi assez hétérogène : arrondi, angles, applat de couleurs, reliefs... tout cela n'est pas très harmonieux.

Bon courage.
merci encore pour vos commentaires.

J'ai beaucup soigné mon code depuis mon precedent post.

Je voudrais savoir si le design et le code sont mieux maintenant ...

Site modifié ici
donc j'attends vos critiques.
;)
Une petite remarque par rapport au design:

je suis connecté via un portable dont l'écran n'est pas top top, et je vois essentiellement du blanc avant de scroller (je vois a peine la première ligne de contenu).

Vu la façon dont est disposé ton titre et le reste, tu pourrais peut e^tre remonter un peu tout ça Smiley smile
Idem, je trouve ça pire au niveau du design.
Ton header prend vraiment bcp trop de place...
Qqn en 800*600 doit vraiment s'arracher les cheveux...
Déjà qu'en 1152 je n'ai pas le menu en entier...

Un bon site passe par une bonne ergonomie.
En un coup d'oeil on doit pouvoir comprendre à qui on a à faire et comment naviguer...

Ton image de header est déformée volontairement ?
Le texte est un peu écrasé.
Il y a également un petit conflit entre les deux images du header.
A gauche, il me semble que c'est le logo... Hors l'image à droite prend également fonction de logo... il y a donc une concurrence.

Ci-joint une idée que je me fais de ta page en plus contracté :


upload/499-djedie.jpg

Désolée si c'est un peu trop critique, mais c'est pour toi Smiley smile
Bon encore une MAJ donc la ca devrait passé sous 800/600 et j'ai corrigé de nombreux bugs.

Pensez a faire F5 pour reactualiser et prendre en compte les modifications.

Dites moi ce que je dois encore changer ou bien ce qui ne va pas.
bizar!
Pourquoi avec Linux ca changerai?
T'es en quel resolution?

J'ai bien des float: left; donc ca devrait pas faire ca.
T as peut etre été sur le site quand je re uploadais les fichiers.

Fais F5 pour réactualiser au cas ou stp..