Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
With IE7 and IE8, hovering over an item displays no sub items. Also the sub menu on the right of the screen isn't shown.
Im using Drupal 7.0-alpha3 with Admin Menu 7.x-3.x-dev (2010-Mar-18) and the default garland theme.
Comment | File | Size | Author |
---|---|---|---|
#4 | 759406.patch | 529 bytes | fietserwin |
Comments
Comment #1
cian73 CreditAttribution: cian73 commentedThis seems to only happen when admin-menu-toolbar is active, so if you don't need the extra css just disable the "Admin menu Toolbar style" module.
The ms-filter shadow-box was killing the sub items for me; this is how I fixed it;
remove lines 26 and 27 from admin_menu_toolbar.css
filter: progid:DXImageTransform.Microsoft.Shadow(color=#000000, direction='180', strength='10');
-ms-filter: "progid:DXImageTransform.Microsoft.Shadow(color=#000000, direction='180', strength='10')";.
I double checked the line syntax for each filter and tried some variations, but the only thing that fixed it was removing the lines.
I also removed lines 23, 24 and 25 as well to standardise the look and feel for other browsers.
If you still wanted the shadow-box in IE you could add another div directly under toolbar, and shadow it, but probably not worth it
Comment #2
fietserwinI can confirm this error and the solution. These lines seem to be taken from the core toolbar module. The difference being that the toolbar from core does not drop down.
I think it is indeed best to remove these filters.
Comment #3
JamesK CreditAttribution: JamesK commentedThis issue is also occuring in IE9 RC. These filters are not required for IE9, since it supports the box-shadow property.
Comment #4
fietserwinOK, we're a bit further now. The error is still there in the current dev version. However IE9 (final) works fine with me, so reverting title. But IE9 now supports the box-shadow property, so the filter is not needed at all anymore for IE9.
So removing both lines altogether is the easiest and best solution. Patch attached, please review and RTBC if satisfied, so we can get this fixed and closed.
Comment #5
fietserwinComment #6
acromediainc CreditAttribution: acromediainc commentedHey All,
Just thought I'd mention that a patch isn't necessary, if you're developing your own theme. I personally try to stay away from core/module modifications as much as possible, but that's just a personal thing. Anyway, my fix was to simply put two lines in my "style.css" for my theme:
Just overrides those two lines in the module's css file.
Of course this only works if you're using your own theme throughout the back-end including the admin pages. Otherwise yeah, the patch would be necessary.
Cheers =D
Comment #7
artis CreditAttribution: artis commentedacromediainc: your solution worked perfectly.
Although it would be nice to get this patch committed to the module.
Comment #8
whitefluffy CreditAttribution: whitefluffy commentedTried out patch from comment #4. Worked great and solved the problem for IE7 and IE8.
Comment #9
sunThanks for reporting, reviewing, and testing! Committed to 7.x-3.x and 8.x-3.x.
A new development snapshot will be available within the next 12 hours. This improvement will be available in the next official release.
Comment #11
hiramanpatil CreditAttribution: hiramanpatil commentedI tried patch from comment #4. It worked for me. Now solved this issue for IE7 and IE8.
Thanks