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.

CommentFileSizeAuthor
#4 759406.patch529 bytesfietserwin
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

cian73’s picture

This 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

fietserwin’s picture

Component: User interface » CSS / Browser Support
Priority: Normal » Major

I 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.

JamesK’s picture

Title: [IE7/IE8] Admin Menu shows no sub items » [IE7/IE8/IE9] Admin Menu shows no sub items

This issue is also occuring in IE9 RC. These filters are not required for IE9, since it supports the box-shadow property.

fietserwin’s picture

Title: [IE7/IE8/IE9] Admin Menu shows no sub items » [IE7/IE8] Admin Menu shows no sub items
FileSize
529 bytes

OK, 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.

fietserwin’s picture

Status: Active » Needs review
acromediainc’s picture

Hey 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:

#admin-menu {
  filter: none;
  -ms-filter: none;
  }

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

artis’s picture

Status: Needs review » Reviewed & tested by the community

acromediainc: your solution worked perfectly.

Although it would be nice to get this patch committed to the module.

whitefluffy’s picture

Tried out patch from comment #4. Worked great and solved the problem for IE7 and IE8.

sun’s picture

Status: Reviewed & tested by the community » Fixed

Thanks 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.

Status: Fixed » Closed (fixed)

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

hiramanpatil’s picture

I tried patch from comment #4. It worked for me. Now solved this issue for IE7 and IE8.

Thanks