IE7/Safari 2.x: Misalignment of 2nd level menus

I remember experiencing this bug with SuckerFish dropdowns a while ago. I somehow fixed it in the stylesheet, but I'm not able to recall how today. I only know that it was related to a really weird style definition. Let's try this:

Could you try to remove or comment out all instances of

font-weight: bold;

in admin_menu.css?

I have the same problem in Safari. An older version of this module from May 2007 worked fine.

FYI... I just replaced the admin_menu.css file with my old one from the March 30th (5.x) build.

Seems to be working OK. Definitely better than the misaligned, newer version of the css. If I run into any major issues, I'll report back.

Attached is a diff between admin_menu.css rev. 1.7 and the latest version. While the diff contains all changes, I'll outline those that may have caused this issue:

 /* all lists */
+#admin_menu ul { position: static; }

AFAIK, added for IE7 support.

 /* all lists */
-#admin_menu a { ... padding: 4px 7px; ... }
+#admin_menu ul a { ... border-bottom: none; padding: 4px 8px; ... text-align: left; }

AFAIK, added to support various themes (border/text-align) and better readability (padding).

 /* all list items */
 /* width needed or else Opera goes nuts */
-#admin_menu li { ... }
+#admin_menu li { ... height: 100%; ... }

AFAIK, added for IE7 support (very strange behaviour).

I hope this helps.

@okeedoak: The .diff file provided in #6 was not a patch to solve this issue. It was a diff to outline all changes since March up to now. You may use this information to debug, if one of the mentioned styles are the cause for the misalignment of second level menus in Safari.

http://cvs.drupal.org/viewvc.py/drupal/contributions/modules/admin_menu/...

I used Rev. 1.7.4.1 of admin_menu.css.

Like I stated, this was the only file I replaced, and it works for me in Safari.

Thanks for clarifying this once more. However, we need to know what exactly causes Safari to misalign second level menu items, because there have been many CSS bugs fixed since 1.7.4.1. That was the reason why I outlined all possibly related changes in #6.

Did you also try later versions? If you could check them, we might be able to narrow the guilty styles down.

Safari 3.0.4 (Mac OS X 10.4.11) does not have this issue.

I don't have a Mac. So I don't know if we can still consider this a bug (compare it to issues related to IE v6 / 7) or if users can simply upgrade their browsers (compared to issues related to Firefox v1 / 2).

I'll keep this issue open for a while. If someone discovers the style that is causing this bug, please let me know.

Adding yourself by posting a "subscribing" (or the like) won't hurt, too, because I have absolutely no idea of how many Safari 2.x users are experiencing this bug.

I just took out the height:100% on the #admin_menu li and it seems to have solved this.

Ok, so attached patch should fix this issue for Safari.

However, height: 100%; has been added to fix exactly the same bug in IE7. Thus, I need someone having access to IE7 to confirm that this patch does not break it again for IE7.

I tried with IE7 and drop-downs are not working at all anymore.

IE7, Windows Vista,
Drupal 5.7
Admin Menu 5.x-2.3 + patch #17

omg, we have missed to rename CSS ids in admin_menu.js.
@Standart: Could you please test attached patch which should fix IE-support in 5.x-2.3?

Attached is a screenshot of IE6 with patch #19 (not patch #17).

A similar effect happens in IE7 with patch #19 AND patch #17 (without patch #17 it's ok).

IE-related bugs in 5.x-2.3 have been fixed. Please update to 5.x-2.4.

Resuming normal operations. Patch in #17 needs review from both IE7 and Safari 2 users.

No replies - seems like #17 is no longer needed. Feel free to re-open this issue if you don't think so.