Bonjour,
ça fait quand même très pub gratuite le premier message qui poste un lien vers son site
Visuellement, c'est sympa, mais j'ai une impression de trop compressé. Il faut laisser de l'espace !
Sans vouloir t'offenser, la photo ne donne pas envie de t'embaucher, principalement parce qu'elle ne te fait pas honneur (mais aussi un peu à cause de la coupe de cheveux (ton coiffeur s'appelle coup de vent, j'imagine). Une photo prise en intérieur, avec un bon éclairage et pas en contre plongée, serait mieux.
Le menu en bas de fenêtre peut être qualifié de mystery meat, même s'il y a des intitulés au survol. C'est part hasard que je me suis rendue compte de sa présence. Ce menu doit être visible en permanence quelque part.
Il n’y a pas vraiment de justification à l’utilisation d’autant d’images-texte. Font-face fonctionne très bien !
Pour le code :
* href sur p...
* Le title de ta page est trop long, il ne sera pas affiché en entier et l’information s’y perd.
* La meta description ne doit pas servir à placer des mots clés : 1. ça n’améliorera pas ton référencement / positionnement ; 2. ça risque au contraire de repousser de potentiels visiteurs / clients.
* La meta keywords ne sert plus à rien, inutile de s’acharner dessus.
* Les meta author, robots (valeurs par défaut) et language ne servent à rien. Pour la dernières, c’est Content-Language qu’il faut utiliser à la place (sauf si le serveur envoie déjà cette information).
* Il est préférable d’inclure les scripts en fin de page... et surtout d’en limiter le nombre : 8 scripts rien que dans le head... c’est beaucoup trop.
* Idem pour les CSS, à part pour les correctifs appelés par commentaires conditionnels, il est préférable de n’avoir qu’un seul fichier.
* Ton site est un vrai défi niveau accessibilité et de référencement (puisque tu le cite comme étant une de tes compétences) : les titres en image c’est bien joli, mais si l’information est perdu, autant ne rien mettre... Le alt d’une image DOIT TOUJOURS être le même contenu que celui porté par l’image... Accessoirement, une image peut très bien se trouver dans un titre hn... Ton détournement des Hn en display none ne risque que de t’apporter une chose : un blacklistage de google si un de tes concurrent regarde ton code et te dénonce).
Donc :
<h2>About Me : Design, Develop, Share</h2>
<img src="img/aboutMe.png" alt="e-wok.fr, Melting pot du Web"/>
Pas bien !
<h2><img src="img/aboutMe.png" alt=" About Me : Design, Develop, Share "/></h2>
Bien !
* Dans le même ordre d’idée, ton « technos diverses + café = réalisation », n’a absolument aucun sens dans le code... (et donc pour les lecteur d’écran et les robots).
* Attention à bien baliser les changements de langue (même si je ne vois pas trop l’intérêt du « coffee »).
* Le double <br> est le signe qu’un nouveau paragraphe aurait du être créé.
* Ta hiérarchie des titres n’est pas logique le h3 « tweet » devrait être un h2.
* Les tableaux de présentation, c’est le loose en 2011...
* Tes séparateurs sont bien gentils, mais il n’aurait pas été plus simple de styler une hr ou juste de les regrouper en une seule image?
* Le traget blank, c’est le meilleur moyen pour que l’utilisateur ferme l’onglet de ton site.
Alors, dans l’ensemble, il y a de grosses améliorations à faire dans le code, ne serait-ce que pour le référencement naturel de ton site.
ps. : "Chef de projet", en sortie de master, oublies, vise un poste de développeur qui évoluera dans 3-5 ans à un poste de chef de projet. D'autant que tes compétences de gestion de projets sont celles d'un analyste fonctionnel.
Modifié par Laurie-Anne (31 Jan 2011 - 14:57)
