On 24 fév, 09:48, totalstranger <totalstran...@not.yahoo.netwrote:
Can someone please take a look at this and give some suggestions.http://www.arnb.org/testmenu.html
Hello,
I checked your stylesheet and I saw a few problems:
/* Set up the first level list items */
#nav li {
position:relative; /* this allows positioning of second level */
display: inline; /* show as a horizontal line */
}
but that's not the correct CSS rule for the first level items. The
correct rule is:
#nav li {
position:relative; /* this allows positioning of second level */
display: inline; /* show as a horizontal line */
}
and when correcting such CSS rule, that makes this other CSS hack
[
background:transparent url(IMAGES/transparent.gif);/*forces IE7 to
work without scripting*/
/* otherwise secondary menu blanks out on a hover*/
]
... completely useless, pointless: so, you can remove that
background: transparent url(...);
declaration entirely.
Some other points:
#nav, #nav ul { /* all lists */
padding: 0;
margin: 0;
list-style: none;
font-weight:normal;
text-align: center;
}
can be better replaced by
ul
{
padding: 0;
margin: 0;
list-style: none;
text-align: center; /* although we explained centering is not best */
}
Also
The selector
#nav li ul li a, #nav li ul li a:visited { ... }
can be better replaced by
#nav ul a, #nav ul a:visited { ... }
You may think those changes are cosmetic but they help in fact parse
the CSS code faster and with lesser chances of triggering bugs in
browsers.
I then checked in Opera 9.26 and my demo works .. although Opera 9.26
is a bit iffy. It works better in Opera 9.50 build 9807 though.
Regards, Gérard