[WebCV] Avis sur le code

Avis sur le code [WebCV] - HTML/CSS - Programmation

Marsh Posté le 23-07-2006 à 14:56:03    

Voilà, conjointement avec mon sujet dans la rubrique Web Design ( http://forum.hardware.fr/hardwaref [...] 4050-1.htm ) j'aimerais avoir ici plutôt des avis sur le code de la version en ligne de mon CV.

 

Adresse: http://www.zeross.info/cv/2

 

Il y a une CSS d'impression permettant de sortir le CV afin de pouvoir l'envoyer directement a des entreprises, il faut par contre activer l'impression des images et des couleurs de fond, c'est plus joli ^^.

 

Merci d'avance.


Message édité par Cheinz le 23-07-2006 à 23:12:13
Reply

Marsh Posté le 23-07-2006 à 14:56:03   

Reply

Marsh Posté le 23-07-2006 à 18:53:51    

Si ton code est conforme au W3C, alors je ne vois pas ce qu'il y a à dire dessus...

Reply

Marsh Posté le 23-07-2006 à 19:01:30    

Bah ca peut être conforme aux normes sans pour autant être correct ou bien optimiser... C'est pour ca que je demandes.

Reply

Marsh Posté le 23-07-2006 à 20:24:39    

C'est clean, je ne vois rien à redire à priori. Par contre, vire la photo, ça sert juste à faire rire les drh.

Reply

Marsh Posté le 23-07-2006 à 20:52:07    

Ok merci, je notes, ca m'étonnes quand même, on m'a toujours conseiller de mettre la photo sur le CV mais bon...

Reply

Marsh Posté le 23-07-2006 à 21:45:42    

pas grand chose a dire si tu veux "optimiser" quelques trucs
 

Code :
  1. padding-bottom: 5px;
  2. padding-left: 0;
  3. padding-right: 0;


 
ceci est equivalent a

Code :
  1. padding : 0 0 5px 0;


 
edit:
voire a padding-bottom:5px; (regarder plus bas)
 
 

Code :
  1. border-style: solid;
  2. border-color: #000000;
  3. border-width: 1px;


ou

Code :
  1. border: 1px solid black;


 
 
souvent tu declares un padding ou un margin juste pour le mettre a zero
c est a faire dès le debut avec  
 

Code :
  1. * { margin:0; padding:0 }


 
ce qui t evitera de de le preciser a chaque fois que tout ne s affiche pas comme tu veux
 
sinon
 

Code :
  1. <div id="header">
  2. </div>
  3. <div id="photo">
  4.  <img src="photo.jpg" alt="Photo"/>
  5. </div>


 
bon on aime ou pas
je prefere eviter les div vides quand meme et image est un bloc qui peut etre stylé
 

Code :
  1. <div id="header">
  2. <img src="photo.jpg" id="photo" alt="Photo"/>
  3. </div>


 
voila parce tu as demandé a ce qu on chipote :D
ca devrait legerement diminuer le poids de ta css mais bon ca reste du pinaillagemental


Message édité par mIRROR le 23-07-2006 à 21:51:00
Reply

Marsh Posté le 23-07-2006 à 23:10:45    

:)  merci mirror, si il n'y a que ca alors ca va c'est que ca doit pas etre trop mal ^^ .

 

J'ai fais les modifications pour les border.


Message édité par Cheinz le 23-07-2006 à 23:11:13
Reply

Marsh Posté le 24-07-2006 à 09:04:21    

Une remarque quand même, on ne doit jamais mettre de texte directement dans un div (celui-ci n'ayant aucune valeur sémantique). Dans ton cas tu pourrais plutôt utiliser des listes, voire des listes de définitions.
 
Quant aux icones, celles-ci n'ont pas de valeurs informatives, et doivent donc être placées via la propriété background en CSS.


---------------
♈ ♋ ♌ ♍ ♎ ♏ - Agora Fidelio | Galerie d'art Toulousaine
Reply

Marsh Posté le 24-07-2006 à 12:21:57    

Cheinz a écrit :

Ok merci, je notes, ca m'étonnes quand même, on m'a toujours conseiller de mettre la photo sur le CV mais bon...


 
Ben tu peux mettre une photo, mais pas celle-là...  :o  Va chez le coiffeur et fais une photo "normale". [:dawa]

Reply

Sujets relatifs:

Leave a Replay

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