3246 sujets

Critiques de vos sites: code et design

Http://www.pomelojob.com

Bonjour,

55 erreurs de code avec un doctype transitionnel, c'est beaucoup. À corriger.

Il y a également de la divite dans l'air.
einboubou a écrit :

heuu c'est quoi de la divite ?

Excès de div, on dit aussi la divite aiguë Smiley cligne
Appart ça ton champ de recherche est trop petit en hauteur et ton bouton go trop basique, pas assez travaillé.
einboubou a écrit :
C'est grave la divite ? ça se soigne ?

Oui c'est assez grave ; ça créé du code en trop.

Graphiquement, le centrage maladroit me pose problème. Un vrai centrage (en gros un qui fonctionne sur d'autres écrans que le tiens) serait bien plus agréable visuellement.

Les textes en vert sont un peu clairs pour une lecture facile.

L'utilisation du PNG avec transparence alpha n'est pas justifiée pour le logo (d'ailleurs as-tu les droits de cette image ?), le fond du site étant uni, ce fond pourrait être intégré dans une image sans transparence alpha (et sans transparence du tout d'ailleurs). Ce qui serait un plus visuellement pour IE6. et un moins pour le poids de la page.

Le code ne semble pas fonctionner :
"siteweb" a écrit :
Warning: SolrClient::query() [solrclient.query]: in /srv/d_pomelojob/pomelojob/public/pomelojob_prod/model/joboffers.php on line 312

Fatal error: Uncaught exception 'SolrClientException' with message 'Unsuccessful query request : Response Code 400. <html> <head> <meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1"/> <title>Error 400 </title> </head> <body><h2>HTTP ERROR: 400</h2><pre>org.apache.lucene.queryParser.ParseException: Cannot parse ' +text Smiley decu accessibilit? +lang Smiley decu en OR fr OR de OR nl) ': Encountered "&lt;EOF&gt;" at line 1, column 51. Was expecting one of: &lt;AND&gt; ... &lt;OR&gt; ... &lt;NOT&gt; ... "+" ... "-" ... "(" ... ")" ... "*" ... "^" ... &lt;QUOTED&gt; ... &lt;TERM&gt; ... &lt;PREFIXTERM&gt; ... &lt;WILDTERM&gt; ... "[" ... "{" ... &lt;NUMBER&gt; ... </pre> <p>RequestURI=/solr/select/</p><p><i><small><a href="http://jetty.mortbay.org/">Powered by Jetty://</a></small></i></p><br/> <br/> <br/> <br/> in /srv/d_pomelojob/pomelojob/public/pomelojob_prod/model/joboffers.php on line 208


Pour le code :
* La langue du document n'est pas précisée dans html.

* Les meta author, email, date-creation, keyword et identifier-url ne servent à rien et peuvent donc être supprimées.

* Il est préférable de placer les scripts JS en fin de document.

* Ceci est de la divite aiguë : 
<div id="menu">
	<div id="lang">
		<span class="navigation">
			<a  href="http://www.pomelojob.com/fr/">[FR]</a>
			<a  href="http://www.pomelojob.com/de/">[DE]</a>
			<a class="active" href="http://www.pomelojob.com/en/">[EN]</a>
		</span>
	</div>
	<div id="pages">
		<ul class="navigation">
			<li><a class="active" href="http://www.pomelojob.com/en/home" title="Home">HOME</a></li>
			<li><a  href="http://www.pomelojob.com/en/joboffers" title="About">JOBOFFERS</a></li>
			<!--<li><a href="http://www.pomelojob.com/blog/en/" title="Blog">BLOG</a></li>-->
			<li><a  href="http://www.pomelojob.com/en/about" title="About">ABOUT</a></li>
			<li><a  href="http://www.pomelojob.com/en/contact" title="Contact">CONTACT</a></li>
		</ul>
	</div>
</div>
Et pourrait se soigner ainsi : 
<div id="menu">
	<ul id="lang">
		<li><a  href="http://www.pomelojob.com/fr/">[FR]</a></li>
		<li><a  href="http://www.pomelojob.com/de/">[DE]</a></li>
		<li><a class="active" href="http://www.pomelojob.com/en/">[EN]</a></li>
	</ul>
	<ul class="navigation">
		<li><a class="active" href="http://www.pomelojob.com/en/home" title="Home">HOME</a></li>
		<li><a  href="http://www.pomelojob.com/en/joboffers" title="About">JOBOFFERS</a></li>
		<!--<li><a href="http://www.pomelojob.com/blog/en/" title="Blog">BLOG</a></li>-->
		<li><a  href="http://www.pomelojob.com/en/about" title="About">ABOUT</a></li>
		<li><a  href="http://www.pomelojob.com/en/contact" title="Contact">CONTACT</a></li>
	</ul>
</div>
Ainsi, on supprime un niveau de div, qui se dupliquait à ul.

* Attention à l'utilisation de title sur les liens : 
<a class="active" href="..." title="Home">HOME</a>
L'attribut title peut être utilisé sur les liens pour fournir une information supplémentaire et nécessaire à la bonne navigation du visiteur. Il ne doit pas être utilisé pour répéter biessement l'intitulé du lien ! Title n'est absolument pas obligatoire sur les liens. Ici tous tes titles sont redondant avec les intitulés de lien, tu peux (dois même) les supprimer.

* Il est dommage d'avoir recours à des styles en ligne.
Modifié par Laurie-Anne (18 Jan 2010 - 09:49)
Salut,

Rôle du candide :
J'arrive sur le site et ma première impression, c'est "je suis où, ca sert à quoi ?"
Il y a "job" dans le nom et je vois un très discret "joboffers" complètement en haut à droite de mes 1920 pixels de large d'écran.
Donc ça doit être un site de recherche d'emploi.
Je suis pizzaïolo, et justement je cherche du boulot, je tape "pizza paris" et je vois des annonces en anglais, qui semblent parler de diplomes d'éducation supérieure, une annonce apple, ou autre...
Comme je ne parle pas anglais, je clique sur l'aussi discret [FR] en haut à gauche de la page (toujours sur mes 1920 pixels de large d'écran). Mais à part quelques informations éparses qui passent en français, les annonces auxquelles je ne comprends rien sont restée en anglais...

Bref tout ça pour dire que j'ai déja passé 2mn sur le site, que je ne sais absolument pas quelle information j'aurais du y trouver et qu'en réalité ca fait déja 1mn 55s que j'ai du quitter le site... pour plusieurs raisons en plus.
On est loin d'un outil pratique... A repenser au niveau communication, ergonomie et graphisme.
Merci pour vos conseils.

Je n'ai pas trop de temps, alors pour le moment je n'ai corrigé que ce que Laurie-Anne m'a indiqué.

5 Divisions ont été supprimées.

Le JS est passé en bas.

Les métas ont été netoyées.

Le vert clair un peu assombri.

Le code php corrigé

La langue du document spécifié, c'est bon comme ça ?
<html xmlns="http://www.w3.org/1999/xhtml" lang="en">


Les titres dans les liens retirés.

Les styles en ligne passé en CSS.

L'image convertis en jpeg et 15ko de gagné plus un plus joli rendu sous ie6. Je sais pas pourquoi elle était en png ...

J'attaquerai les recommandations de Mikachu, qui demande plus de réflexions, quand j'aurais un peu plus de temps.

Une petite question pour terminé, quelqu'un sait pourquoi sous ie6 la liste des offres d'emplois prend toute la page quand les sidebar sont terminées ? Ça ne fait pas ça sous ie7 et firefox... Si quelqu'un a une piste à me donner parce que je ne vois pas trop le problème ...

Merci d'avance.
Ben voilà, je le trouve "joulijouli" ton site, maintenant il est bien centré, on y voit clair pour la couleur verte, mais je suis comme Mikachu, je ne sais toujours pas à quoi sert ce site donc… je zappe. C'est un nouveau Google ?

einboubou a écrit :
La langue du document spécifié, c'est bon comme ça ?
<html xmlns="http://www.w3.org/1999/xhtml" lang="en">


Normalement ce serait :

<html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en">
ou bien les deux ?? xml:lang="en" et lang="en". Pour ma part je n'indique que le premier et je ne sais pas si la différence a une grande importance.
Sujet clos
Aller à