IE / Safari support

At the moment, IE is generally not supported. So I'm marking this as feature request.

IE does not understand the pseudo-class :hover, on which the CSS of this module is based.

Did you try the IE7 script from http://dean.edwards.name/IE7/ ? IE needs support for dynamic pseudo classes that can be extracted from IE7 package by using ie7-core and some other scripts (see documentation there).

If you'll get this working with IE7 script I won't hesitate to commit a patch for IE support.

Attached is a patch that introduces support for Internet Explorer.

In order to let this happen you need to download the already mentioned IE7 script available from http://dean.edwards.name/download/ and place the files of that archive into a new folder /misc/ie7 of your drupal installation.

Since ie7 script might be used by other modules or themes I thought it would be best to store it in /misc, so other things might re-use it without the need of reloading it from scratch.

CSS :hover support can be added to IE easely,
read http://www.xs4all.nl/~peterned/csshover.html how to do it.
You just need to download csshover.htc file and put a couple of lines to HTML code to use it.
That is where magic happens, IE can process :hover pseudo-class on any TAG

Thanks for this hint, vladimir.

I just realized that there's no notice yet that Admin Menu is actually relying on the technique introduced by Sons of Suckerfish Dropdowns.

Although Suckerfish Dropdowns are in fact providing IE support (that supersedes usage of htc files) I've ripped that some time ago in favor of the ie7 script by Dean Edwards, because that makes IE even more standards-compliant (regarding transparent pngs and so on...).

After doing some research on htc techniques I found several forum posts that complain about mime-type problems on Apache webservers, as already stated by Peterned:

NOTE 1: The rules for HTCs have changed a bit in Windows XP SP2. Users with SP2 installed may not see it working correctly, because webservers have to send htc files with the mime-type set to text/x-component.

Anyway, implementing a settings page to let the admin choose the preferred technique for Drupal Administration Menu seems to be the silver bullet. No code hacks needed, just place the corresponding file(s) into the folder /misc and enable the technique of your choice.

Anyone agreeing?

Attached patch introduces full IE and Safari support.

While Safari only does not like bold formatted hyperlinks in expandable list items (yay, seems to be a bug in Safari finally), IE needs really ugly tweaking (relying on original Sons of Suckerfish Dropdowns javascript code).

Committed.

Path for IE7 styles is not correct for sites in subdir (example.com/mysite). We need modify:

      // IE fix
      $ie_header = "<!--[if lt IE 7]>\n";
      $ie_header.= '<script type="text/javascript" src="/' . drupal_get_path('module', 'admin_menu') . '/admin_menu.js' . '"></script>';
      $ie_header.= "\n<![endif]-->";

into

      // IE fix
      $ie_header = "<!--[if lt IE 7]>\n";
      $ie_header.= '<script type="text/javascript" src="'. base_path() . drupal_get_path('module', 'admin_menu') . '/admin_menu.js' . '"></script>';
      $ie_header.= "\n<![endif]-->";

Committed, thanks!

btw: http://drupal.org/diffandpatch