Ergonomie
La lisibilité est plutôt bonne, sauf sur un écran un peu large (au dessus de 1024px de large), où la longueur de la ligne de texte dans le bloc principal est
vraiment trop importante. Utiliser une largeur maximale serait un plus.
Les blocs (en-tête, bloc principal, menu) sont collés entre eux, et ne se distinguent pas directement au premier regard. Un peu d'espace serait appréciable.
Graphisme
Effectivement, c'est très basique. Le logo du jeu manque d'un visuel qui tienne la route, et la charte graphique, globalement, est assez simpliste. Peut mieux faire (comme vous en êtes conscient, je pense).
Code et accessibilité
Hum, zéro pointé ?
Plus sérieusement, et comme vous le savez sans doute déjà, les pages composées de tableaux (et surtout de tableaux imbriqués) posent de sérieux problèmes d'accessibilité. Or, c'est bien simple, vous avez utilisé des tableaux pour tout et n'importe quoi, sans doute par ignorance de ce qu'il est possible de faire par d'autres moyens.
Utiliser un tableau de mise en forme pour séparer dans l'en-tête le logo du blocs des statistiques, pourquoi pas. Un autre tableau pour séparer le menu du bloc de contenu, admettons.
Utiliser un tableau à neuf cellules (dont une seule accueille vraiment du contenu !) pour dessiner des jolies bordures et des coins arrondis pour les blocs, c'est hélas une pratique courante, même s'il existe d'autres solutions :
http://web.covertprestige.info/test/05-boite-avec-bordures-en-html-et-css.html
http://web.covertprestige.info/test/06-page-fluide-avec-bordures-en-html-et-css.html
(À noter que la gestion des bordures décorées et autres coins arrondis d'une boîte de largeur fluide est assez compliquée, tandis que le cas d'une boîte de largeur fixe est bien plus simple.)
Mais utiliser un tableau à l'intérieur d'un bloc de contenu pour séparer des lignes de texte, c'est une erreur de conception flagrante !
Non seulement c'est très mauvais pour l'accessibilité, mais en plus cela donne un code très difficile à maintenir/mettre à jour.
Pourquoi ne pas utiliser tout simplement des éléments HTML tels que les titres de niveau (éléments
h1 à
h6), des paragraphes (élément HTML
p), des listes non ordonnées ou ordonnées (
ul et
ol), etc. ?
Sur ce sujet, on peut se reporter à différents tutoriels sur Alsacréations :
http://css.alsacreations.com/Faire-une-mise-en-page-sans-tableaux/
Pour finir, on peut signaler plusieurs erreurs de syntaxe, relevées par le validateur de syntaxe HTML du W3C :
http://validator.w3.org/check?uri=http%3A%2F%2Fwww.planetswars.com
La plus importante de ces erreurs est
l'omission d'une déclaration de type de document (doctype), qui n'est pas accessoire, loin de là !