Bonjour tout le monde,
Je viens soumettre le premier jet de mon portfolio http://cyrou.fr, les contenus rédactionnels ne sont pas encore définitifs, mais si vous avez des remarques sur les aspects techniques n'hésitez pas.
Je débute tout juste en CSS Smiley confused donc il doit surement y avoir pas mal de choses perfectibles.
Merci d'avance. Smiley smile
Salut,

Sur la page d'accueil, le "Soyez les bienvenus..." dépasse de la page.

Il y a des erreurs au niveau du code.

Vu que ton doctype est en XHTML 1.0 Strict, il faut que les balises <br> soient fermées (<br />).

Sur toutes (?) tes images, il manque le alt="". Il faut la remplir si l'image est porteuse de contenu et la laisser vide sinon.

Peut-être pourrais-tu regrouper <ul class="rubriques"> et <ul id="rubriquestext"> sous la même liste.

Sur tes <li class="reseaux"> il y a une virgule qui n'a pas sa place tout comme le target="_blank" (les gens n'aiment pas qu'on décide comment ils doivent naviguer).

Sinon, sur ta page contact, il n'est peut être pas judicieux de mettre ton mail au risque d'être submergé de spam.

Pour ton menu (Accueil Portfolio...), il y a quelque chose de gênant : dans ton code html ils sont inversés. Du coup quand tu navigues au clavier, tu le fais de droite à gauche.

Sur la page service, il y a un espace en trop avant les apostrophes (partie intégration / réalisation).

PS : il manque une balise méta description dans le header.
Modifié par Xulu (05 May 2010 - 18:48)
Bonsoir,
Je trouve le design pas mal.

Quelques remarques en vrac :
* Ton logo n'est pas très lisible : le "y", le "n" et le "a" seraient à retravailler.
* Dans la rubrique print, seule la première image de chacun de tes travaux est montrée, du coup il faut manœuvrer un clic en plus ou accéder à la seconde (et la troisième) image. Même si le nombre d'images est indiqué sur la page, il serait mieux de les exposer toutes (quitte à les disposer en plus petit que la première image) pour être accessibles dès la page actuelle : tout d'abord j'avais cru (je suis abruti comme la plupart des internautes) que les trois images de Perrier Fluo étaient les trois bouteilles de l'image (pas de commentaires svp !)
EDIT : ceci est valable aussi pour la rubrique illustration.

* le texte n'est pas agrandissable : dans la feuille CSS, affecter à l'élément body la propriété {font-size:100%} et définir toutes les tailles de police du reste du site en unités em : h2{font-size:0.6em;} signifie que les titres h2 seront de taille 60% par rapport à la taille de police de l'élément parent (body, probablement, souvent réglé à 16px par défaut pour l'internaute)
Modifié par darkstar2023 (06 May 2010 - 00:55)
Bonjour,

Avant de voir le site je vais corriger/développer quelques points évoqués :
darkstar2023 a écrit :
* le texte n'est pas agrandissable : dans la feuille CSS, affecter à l'élément body la propriété {font-size:100%} et définir toutes les tailles de police du reste du site en unités em : h2{font-size:0.6em;} signifie que les titres h2 seront de taille 60% par rapport à la taille de police de l'élément parent (body, probablement, souvent réglé à 16px par défaut pour l'internaute)
Euh...

Alors, déjà il faut préciser que ce n'est pas agrandissable sous IE 6 et 7 (via taille des caractères). Ensuite "em" n'est pas une obligation les % sont très bien aussi. Enfin le 100% ne sert à rien (enfin si à corriger les bêtises quand on ne maitrise pas em).

Xulu a écrit :
qui n'a pas sa place tout comme le target="_blank" (les gens n'aiment pas qu'on décide comment ils doivent naviguer).
Le principal problème de ton traget c'est que tu as choisi un doctype qui ne l'autorise pas, si tu tiens absolument à ce que tes visiteurs ferment la page de ton site lors de l'ouverture d'une autre (sans posibilité de retour vers ton site) alors utilises un doctype transitionel.

Xulu a écrit :
il manque une balise méta description dans le header.
La meta description a une et une seule utilité : être affichée par les moteurs de recherche dans les pages de résultats, sous le titre de ton site (sinon ce sont les N premiers caractères qui le sont). Il est donc important, non seulement de spécifier cette meta mais surtout de l'optimiser, de la rendre attirante pour le visiteur ; quand tu n'es pas une référence sur Internet, c'est souvent elle (et la pertinence de la talise title) qui va te démarquer d'un concurent.


Visuellement, le site est un peu trop terne, il n'y a aucune couleur en dehors du lien courrant, d'une portion de ton logo et des travaux (qui apparaient sous le fold avec ma config actuelle). Il faut absolument rajouter un peu de vie dans ce design.

Il serait bien de prévoir des alternatives aux PNG semi-transparents.

Ton animation est trop rapide (et répétitive), je m'attendais également à ce qu'elle soit cliquable.


Niveau code (il y a aura des répétitions mais c'est pour que ça rentre Smiley langue ) :
* Il manque les meta description et Content-Language.

* Il est préférable d'inclure les scripts (JS) en fin de document afin de laisser le contenu se charger avant son comportement.

* Ta hiérarchie de titres n'est pas pertinente. Sur ta page d'accueil, ton h2 et ton h3 ne sont pas des titres de section.

* L'inversement code/visuel des éléments de ton menu pose un problème lors de la navigation au clavier : elle n'est plus logique. Il te faut soit changer ton code soit ajouter des tabindex pour rétablir un ordre de tabulation logique.

* Ceci :
<div id="cont">
<div id="news">
<div class="mask1">
<div id="box1">
Ressemble fort à de la divite.

* Il manque effectivement des alt sur certaines de tes images, sur les autres ils ne sont pas toujours pertinents. L'attribut alt d'une image doit retranscrire exactement le même contenu que l'image elle-même.

* Ceci :
<li><a href="portfolio_web.html"><img src="images/icone_web.jpg"/></a></li>
Est un lien vide. Les images suport de lien doivent absolument avoir un contenu (une image sans alt n'est pas un contenu, ce n'est rien) pertinent !

* Pourquoi ne pas appliquer les liens sur les intitulés sous les images ?

* Ta doubles liste créerait une redondance si tes alt étaient bien présent, dans l'état actuel, c'est juste fouilli.

* Tu as un problème d'encodage sur la page "services"

Dans l'ensemble, tu as un site qui a l'air pro (bien que triste), par contre tu manque énormément de rigueur dans ton code.
Laurie-Anne a écrit :
Bonjour,

Avant de voir le site je vais corriger/développer quelques points évoqués :
Euh...

Alors, déjà il faut préciser que ce n'est pas agrandissable sous IE 6 et 7 (via taille des caractères). Ensuite "em" n'est pas une obligation les % sont très bien aussi. Enfin le 100% ne sert à rien (enfin si à corriger les bêtises quand on ne maitrise pas em).

Si on ne définit pas la taille de référence 100%, qu'est-ce qui va la définir ?
darkstar2023 a écrit :
Si on ne définit pas la taille de référence 100%, qu'est-ce qui va la définir ?
Le navigateur ou l'utilisateur du navigateur. D'ailleurs définir 100% c'est dire au navigateur d'utiliser sa taille par défaut...