If running the latest jquery_update you get a browser error. Mainly because of the removal of $.browser.

In short a simple extra if condition will fix this.

Patch coming right up.

CommentFileSizeAuthor
#1 admin_menu.code_.2114587-1.patch953 bytesericduran

Comments

ericduran’s picture

ericduran commented
Status: Active » Needs review
StatusFileSize
new admin_menu.code_.2114587-1.patch953 bytes

Nice and simple.

deryck.henson’s picture

deryck.henson commented
Status: Needs review » Reviewed & tested by the community

Patch works w/ 7.x-3.0-rc4. jQuery 1.10.1 used on Drupal-7.23.

Will work on 1.9+ (version $.browser left the building).

Clean and simple. Thank you.

quicksketch’s picture

quicksketch
he/him
commented
Issue summary: View changes

+1 here too. It'd also probably be okay to just remove that entire section, we don't need to support IE6 at this point surely, especially for admin-only functionality like admin_menu.

js’s picture

js commented

@quicksketch nailed it.

bhavikshah9’s picture

bhavikshah9 commented

I noticed the bug earlier. I just commented entire section for IE hover emulation and it worked fine then. But the patch provided here is better than my tweak and its working fine too.
Thanks a bunch

markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Reviewed & tested by the community » Closed (duplicate)
Related issues: +#1892074: remove $.browser (Remove IE 6 support)

This has already been removed by the related issue and this patch (in theory) shouldn't apply anymore.