Niveau design et ergonomie, je trouve ça pas mal, ça tient la route. Le menu pourrait êtr plus soigné par contre, et la bordure blanche montrant le menu actif n'est pas une excellente idée, on dirait vraiment un problème d'affichage de la barre orange. L'ombre générale est un peu grossière aussi, essaye quelque chose de plus discret. Ton gris de texte n'est pas fameux non plus, l'interlignage est trop important. Toujours pour la typo, ça manque cruellement de contraste. Essaye peut-être une sérif pour tes titres parce que là c'est trop "plat". C'est vraiment au niveau de la gestion des espaces, des blancs et des typos que j'ai le plus de mal. Pour le reste, ça va, les couleurs marchent bien ensemble, la structure est bonne aussi.
Au niveau du code, je ne suis pas 100% convaincu... Ce que je n'ai pas du tout aimé, c'est la tricherie sur ton premier h1, j'ai vraiment un problème avec ce genre d'approches...
Sinon, pourquoi tant de h1? Il existe six degrés de titrage permettant de créer des niveaux de lecture cohérent, utilise-les.
Les title sur les images et autres accesskey, franchement, tu peux virer... Pour les images, tu gagnerais à utiliser du png à la place du jpg, du moins pour les petits et moyens formats. Je trouve également que tu utilises beaucoup de div, ce n'est pas bien méchant, mais ça alourdi inutilement ton code et le rend par conséquent moins lisible.
Sinon, globalement, c'est sur la bonne voie, ça c'est sûr.
