Cita: #menu{text-align:center; background:gold; float:left; width:150px ; margin:10px 0 5px 10px; padding:5px 0 0 0;}
#menu p{text-align:center; background:white; color:red; margin:10px;}
#menu h1{background:url(imagenes/menu-imagen-superior.png) #FF9900; text-align:center; text-decoration:underline; color:#ff0000; font-size:16px ; padding:1px 0;}
#menu li{list-style:none; text-align:left; border-bottom:1px solid silver;}
#menu a{font.size: 1.1em; background:gold; text-align:center; color:black; text-decoration:none ; display:block ; padding:1px 3px 1px 5px;}
#menu a:link{background:gold;}
#menu a:visited{background:#EEEEE0; color:#8B8989; border:0;}
#menu a:hover{color:red; background:#fff;}
#menu a:active{text-align:center; color:red; background:white; border:0;}
Ten presente el orden de los valores para la etiqueta a: "
link, visited, hover, active"
Te eh optimizado y mejorado mucho el código por ejemplo:
No necesitas colocar background-color, con solo background esta bien, te recomiendo que no uses "
gold, white" usa los valores
Hexadecimales. La anterior web puedes elegir un color y abajo de la paleta de colores dice RGB, coges el color y lo colocas con #. es decir: #934463.
No necesitas agregarle un valor a un elemento "0" es decir: 0px, con solo colocarle el 0 esta bien y también te corregi algunas cosas.
Saludos!