The horizontal menus (dropping down) are having z-index issues in both IE6 and IE7. This means that in garland the you can lose the bottom of the menus behind the top "page" image where the breadcrumbs typically go as well as going behind the logo image and even the site name text. The vertical menus that pop from the side seem to work fine.

CommentFileSizeAuthor
#3 nice-menus-z-index-183924-3.patch532 bytesadd1sun
nice_menus_z.jpg29.8 KBadd1sun

Comments

herb’s picture

herb commented

FWIW, I see the same issue in both vertical and horizontal menus in IE6 and IE7 when using bluemarine. Most obviously for me when viewing the ?q=image or gallery page. Yet, when viewing a particular gallery, ie ?q=image/tid/1 the menus hover over the images quite nicely even though everything is just a li?? and both seem to "hasLayout -1" (whatever that means...)

It's great the the IE7 ghosts are gone, and that the menus work well with IE6 - This seems to be the last remaining glitch. I'm afraid I can't help debug this, but will help test the next release.

add1sun’s picture

add1sun commented
Assigned: Unassigned » add1sun
Priority: Normal » Critical

Moving this to crtical since this needs to be fixed before a stable release.

add1sun’s picture

add1sun commented
Title: Hor. menus z-index issue in IE » Menus z-index issue in IE
Status: Active » Needs review
StatusFileSize
new nice-menus-z-index-183924-3.patch532 bytes

OK, well I spent a good chunk of time futzing with this. Bleh. Anyway, I've attached a patch for anyone that wants to do it from patch but it is a one line change to nice_menus.css. Just add position: relative; to ul.nice-menu:

 ul.nice-menu,
 ul.nice-menu ul{
     z-index: 5;
+  position: relative;
 }

Please see if that works for you and if ti does I think we are pretty close to rolling a stable release.

herb’s picture

herb commented

Fantastic!! - I never would have figured that out. Just tested the patch in IE6 and IE7 and all menus seem to work as they should.
Big Thanks for your efforts!

add1sun’s picture

add1sun commented
Status: Needs review » Fixed

Committed to HEAD and 5 dev.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.