Direi che va bene. Solo farei una modifica al CSS per il tag <a>:
codice:
li.menu1 a{
display:block;
text-align: center;
text-decoration: none;
padding-top:6px;
background: url(Immagine.bmp) no-repeat top left;
width:100%;
height:100%;
}
li.menu1 a:hover{
background: url(Immagine2.bmp);
}
oppure:
codice:
li.menu1 a {
display:block;
text-align: center;
text-decoration: none;
padding-top:6px;
background: url(Immagine.bmp) no-repeat top left;
width:100%;
height:100%;
}
li.menu1 a:link, li.menu1 a:visited {
background-image: url(Immagine.bmp);
color: #???;
}
li.menu1 a:hover{
background-image: url(Immagine2.bmp);
text-decoration: none;
color: #???;
}
Ci sono alcune proprieta` che hanno un default nelle varie condizioni dei link (:link, :visted, :hover, :active); in linea di massima sono: color, text-decoration . Tali proprieta` conviene ripeterle nelle pseudoclassi; le altre invece non serve (creano solo confusione, e la confusione e` amica degli errori).