Un site test à critiquer (FlorentG, gatsu, plainsofpain..)

Un site test à critiquer (FlorentG, gatsu, plainsofpain..) - HTML/CSS - Programmation

Marsh Posté le 22-07-2005 à 13:25:26    

Bonjour
J'ai fait rapidement mon premier site test xhtml/css, et j'aimerais que vous regardiez si mon code est bien utilisé, si il y a pas des choses à simplifier, des choses oubliées, enfin des trucs que je fais mal, et qu'il faudrait que je sache avant d'attaquer des projets plus importants  :p  
 
Vous pouvez aussi m'aider graphiquement parce que c'est pas terrible, si vous avez des petits trucs à faire partager pour donner + de coherence ou autre  :p  
 
http://skyhead.free.fr/test/index.html
 
N'hesitez pas à être dur, j'aime çà  [:cupra]

Reply

Marsh Posté le 22-07-2005 à 13:25:26   

Reply

Marsh Posté le 22-07-2005 à 13:28:30    

Il te faudrais une liste pour le premier menu également AMHA

Reply

Marsh Posté le 22-07-2005 à 13:32:07    

pluzun.
Et au lieu de  

Citation :


<div id="header1"><img class="droite" alt="logo" src="design/logo.gif" /></div>


 
J'aurais directement mis

Citation :


<img id="header1" alt="logo" src="design/logo.gif" />


---------------
Can't buy what I want because it's free -
Reply

Marsh Posté le 22-07-2005 à 13:38:47    

Ok pour skeye j'y ai pas pensé :p
 
Mais pour le menu horizontal (c'est bien celui-là dont tu parles Ayuget ?) comment mettre une liste et empecher le saut de ligne ? à moins de ne mettre que les <ul> et pas les <li> mais çà ne se fait pas j'imagine, donc expliquez moi svp :p

Reply

Marsh Posté le 22-07-2005 à 13:52:17    

Pouir le  

<div id="header1"><img class="droite" alt="logo" src="design/logo.gif" /></div>


J'aurais mis :

<h1><img class="droite" alt="logo" src="design/logo.gif" /></h1>


C'est le titre du site, qu'on met généralement dans un titre de niveau 1 (d'où le h1).
 
Pour ton menu :

#menu li {
  display: inline;
}


Ou si tu veux contrôler leur dimension :

#menu li {
  display: block;
  float: left;
}

Reply

Marsh Posté le 22-07-2005 à 13:54:11    

Et pour l'image logo, un alt="logo" est inutile, il faut plutôt remplacer par le nom du site :

<h1><img alt="aoyo" src="design/logo.gif" /></h1>


Et jamais de class ou d'id qui indique une position (genre class="droite" ). Si un jour tu restyle ton site et que tu met ton logo en bas à gauche, ça va être cool un truc en bas à gauche de classe droite ;)

Reply

Marsh Posté le 22-07-2005 à 15:17:13    

J'arrive après la bataille, saleté de FT ^^'
 
Pour les menus, il n'y en a qu'un à chaque fois. Je mettrais donc un id plutot qu'une classe, mais c'est du pinaillage. :)
 
Pas de connexion dans la boite pendant une matinée, et au moment ou le technicien de ft me dit par téléphone : "je vois pas de problème, votre ligne fonctionne", j'ai à nouveau internet ... Bizarre non ?
 
C'est la deuxième fois en plus ...
 
Je les soupconne de bloquer les lignes, pour qu'on appelle au 1015 et que ca puisse payer les techniciens :o
 


---------------
http://www.alsacreations.com , http://www.openweb.eu.org. Mon CV : http://cv.roane-irkana.net/. Exemple à ne surtout pas suivre : www.worldinternet.be
Reply

Marsh Posté le 22-07-2005 à 15:28:49    

[HS]quelle contrat FT ??[/HS]
 
sinon, pour le site : a href=URI ... là c'est vide. mais pariel, c'est pour chipoter
 
@+


---------------
:Moultiplayers:.:Quatuor Chevalier:.:Admin G15-france.com:.:Wiki -G15-france:.:-Papa-:.
Reply

Marsh Posté le 22-07-2005 à 15:30:21    

[HS]FT fournit internet depuis plus de 6 ans dans la boite. Et depuis 2 jours, plus de connexion. Juste pendant que le mec vérifie la ligne, ca remarche. Fort hein ? [/HS]


---------------
http://www.alsacreations.com , http://www.openweb.eu.org. Mon CV : http://cv.roane-irkana.net/. Exemple à ne surtout pas suivre : www.worldinternet.be
Reply

Marsh Posté le 22-07-2005 à 16:00:51    

Reply

Marsh Posté le 22-07-2005 à 16:00:51   

Reply

Marsh Posté le 22-07-2005 à 20:31:38    

FlorentG a écrit :

Pouir le  

<div id="header1"><img class="droite" alt="logo" src="design/logo.gif" /></div>


J'aurais mis :

<h1><img class="droite" alt="logo" src="design/logo.gif" /></h1>


C'est le titre du site, qu'on met généralement dans un titre de niveau 1 (d'où le h1).
 
Pour ton menu :

#menu li {
  display: inline;
}


Ou si tu veux contrôler leur dimension :

#menu li {
  display: block;
  float: left;
}



 
Donc mon <h1> je le mets en <h2>. Merci je ne savais ni pour le display: inline (j'ai encore pas mal de balises non maitrisées) ni pour l'image dans un <h>. Merci  :p  
 

FlorentG a écrit :

Et pour l'image logo, un alt="logo" est inutile, il faut plutôt remplacer par le nom du site :

<h1><img alt="aoyo" src="design/logo.gif" /></h1>


Et jamais de class ou d'id qui indique une position (genre class="droite" ). Si un jour tu restyle ton site et que tu met ton logo en bas à gauche, ça va être cool un truc en bas à gauche de classe droite ;)


 
Bon le titre du site c'est logo désolé  [:cupra]  En fait aoyo c'est logo avec une police bizarre donc ca aurait été ecrit autre chose dans le logo j'aurais pas crit logo  [:cupra]  
Je pensais pour le class="droite" que comme il a dedans qu'un float: right j'aurais pu l'utiliser pour un autre element que je voudrais mettre à droite, mais t'as raison il y a pas grand chance que j'ai besoin d'un float: right tout seul et autant refaire un class et pas se melanger.  :)  
 

plainsofpain a écrit :


Pour les menus, il n'y en a qu'un à chaque fois. Je mettrais donc un id plutot qu'une classe, mais c'est du pinaillage. :)


 
Oui mais ca me permet d'en rajouter si besoin sans avoir à rechercher mon id et de le repasser en classe, je pense que çà je vais le garder parce que c'est utile je trouve  [:spamafote]  
 

bl@p_psx a écrit :


sinon, pour le site : a href=URI ... là c'est vide. mais pariel, c'est pour chipoter
@+


 
J'ai pas compris  [:cupra]  
Si c'est <a href=""> qu'est vide c'est normal, si c'est autre chose j'ai pas compris  [:hello_cass]  
Merci pour le texte en latin, c'est pour çà que en fait c'est jamais le meme mais que ca commence toujours pareil (en fait j'avais pas vu avant de poser la question que c'etait pas pareil, je lisais que le début de la ligne  [:cupra] )
 
Merci pour tous vos commentaires, ca me fait avancer,  je reposterais un nouveau truc demain je pense  :hello:

Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed