+ Todo el CSS para un archivo externo.
+ Para que son esos comentarios del CSS? Hacks?
+ Identa el código, así te será as fácil si quieres modificarlo.
+ No es necesario usar JS para efectos rollover.
+ Es innecesario que uses tanto la etiqueta <span>, incluso hay algunos <span class ="">...
+ <br> en XHTML debe estar cerrada, por lo tanto es <br />
+ El atributo color, bgcolor y todos los que den estilo, entan descontinuados, para dar estilos usa CSS.
+ No maqutes con tablas, las tablas son pra eso, datos tabulados. Usa <div> para maquetar.
Visualmente no es una mala web, pero el código es un desorden total, se podría ahorrar mucho tiempo de carga y renderización en el navegador si no hubieran tantas etiquetas inútiles, mal anidadas, cosas como </... y nada mas!, maquetacion con tablas, uso de atributos descontinuados y tantos <span> que la verdad la mayoría son inútiles.