Closed (fixed)
Project:
Administration menu
Version:
7.x-3.x-dev
Component:
CSS / Browser Support
Priority:
Major
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
1 Apr 2010 at 13:49 UTC
Updated:
28 Feb 2013 at 06:37 UTC
Jump to comment: Most recent file
Comments
Comment #1
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 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 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 commentedacromediainc: your solution worked perfectly.
Although it would be nice to get this patch committed to the module.
Comment #8
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
hiramanpatilI tried patch from comment #4. It worked for me. Now solved this issue for IE7 and IE8.
Thanks