Bonjour Benjamin,
Ton site est visuellement agréable. Mais, ajouter ton nom en gros en début de page serait un plus (personnellement, je ne lis pas la barre de titre de mon navigateur, je ne dois pas être la seule).
Ton formulaire de contact ne gère pas très bien les erreurs. Un message me criant de vérifier les champs du formulaire en cas d'erreur de saisie n'est pas très pratique, ni agréable. Il serait bon 1. d'indiquer quels champs sont obligatoire, 2. de signaler exactement les champs à corriger en cas d'erreur. De plus, il serait préférable d'avoir des intitulés de champs en dehors des champs, surtout que si l'on clique dans le champs, puis en dehors du champs l'intitulé est perdu
Bref, le formulaire est
absolument à revoir ; après tous, c'est un moyen de te proposer un travail.
L'encadrement de tes compétences apporte une grosse confusion : ils ressemblent à tes liens et semble donc cliquables... mais non. C'est très décevant pour un visiteur de ne pas pouvoir cliquer sur ce qu'il pense être un lien.
Il serait préférable de présenter un nombre de réalisations, dans ton portfolio, qui soit un multiple de 4 (ou 2), le 9ème élément faisant un peu "cheveux dans la soupe".
Le "HTML Sctrict" n'apporte rien. Il serait préférable de préciser dans tes compétences que tu code en suivant les standards.
Pour le code :
* Tu utilise une pléthore de meta. La plupart sont totalement inutiles et peuvent être supprimés (gain de poids de page) : Title (ne sert à rien), Keywords (plus/pas utilisé par les moteurs), Subject (ne sert à rien), category (ne sert à rien) author (ne sert à rien, n’apporte rien, n’a aucune valeur), reply-to (ne sert à rien, si ce n’est attraper du spam), revisit-after (gros mythe, ne sert à rien), language (ne sert à rien, préférer Content-Language), Robots (valeurs par défaut), Pragma (???), Refresh (si la valeurs est « non » ne sert à rien). La meta restante Description devrait avoir un nom en minuscule (première lettre en majuscule), c’est plus propre et apporter un contenu intéressant... Ce n’est pas le cas actuellement.
* Il manque une meta Content-Language
* 3 fichiers externes de styles, c’est bien en développement ; quand tu passes ton site en production (c’est le cas) une seule suffit et est préférable (moins d’appel serveur = gain de temps de chargement).
* 5 fichiers JS c’est beaucoup trop, comme les CSS, il faut les regrouper en un seul fichier. Afin de ne pas bloquer le chargement du contenu, il est également préférable de les placer en fin de document.
* Quel est l’intérêt de cette « chose » :
<a href="" title="BE - Print, Webdesign & Développement web"></a>
Un lien vide sans href... Qu’est ce que ça t’apporte (à part bien sûr la mauvaise idée de vouloir placer une image porteuse de contenu dans le CSS...) ? L’image doit être placée dans le code HTML avec un attribut alt correctement rempli (et qui rendra le title inutile).
* Les titles sur ton menu sont totalement inutiles : title
PEUT être utilisé sur les liens pour apporter une information
supplémentaire à l’intitulé du lien ET
nécessaire à la bonne navigation du visiteur. Un title se contentant de répéter l’intitulé du lien ne remplis pas ces deux conditions.
* Les div « clear » pourraient effectivement être supprimés (utiliser overflow sur le parent des éléments flottants).
* Les span class « gras » pourraient être remplacés par <b> ou <strong>.
* Ta hiérarchie des titres n’est pas correcte : elle débute à h3 alors qu’elle doit toujours débuter à h1 et ne pas comporter de sauts (pas de h1 suivi de h3 sans h2 entre les deux).
* Plutôt que de faire suivre des </p> d’un <br>, il serait préférable de jouer sur les marges en CSS.
* Tu n’a aucun label sur ton formulaire.
En bref : des balises inutiles (il est possible d’optimiser certains éléments imbriqués comme <a href="#contact" title="Contact"><span class="grasvert">contacter</span></a>, où le span n’apporte rien, sa class pouvant être placée sur le lien), des titles qui ne servent à rien dans tous les sens...
). Tu ne peux pas proposer une version "hd" pour montrer les détails?