Hi,
as far of jquery 1.9 jQuery.browser was removed, and admin_menu get this error:
"Uncaught TypeError: Cannot read property 'msie' of undefined" (admin_menu.js - line:223)

For quick fix use this code:
change line 223:
- if ($.browser.msie && parseInt(jQuery.browser.version) == 6) {
+ if (navigator.userAgent.match(/msie [6]/i)) {

CommentFileSizeAuthor
#2 admin_menu-ie6detect-1961178-2.patch583 bytesjcisio
#1 admin_menu-ie6detect-1961178-1.patch531 bytestompagabor

Comments

tompagabor’s picture

tompagabor commented
Status: Active » Needs review
StatusFileSize
new admin_menu-ie6detect-1961178-1.patch531 bytes

Here is the patch.

jcisio’s picture

jcisio commented
Version: 7.x-3.0-rc4 » 7.x-3.x-dev
Category: Task » Bug report
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new admin_menu-ie6detect-1961178-2.patch583 bytes

Patch works as expected. Tested with jQuery 1.10 too. Attached a more verbose patch.

tompagabor’s picture

tompagabor commented
Assigned: Unassigned » tompagabor
Issue summary: View changes

I assigned this issue to myself, os anything is happening with this issue, i do my best to fix this bug.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented

RTBC++ @tompagabor any eta on committing this to dev?

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)
Anonymous’s picture

Anonymous (not verified) commented

@jcisio & @tompagabor, the only way that last patch works is if you add a "}" in the end:

Drupal.admin.behaviors.hover = function (context, settings, $adminMenu) {
// Hover emulation for IE 6.
- if ($.browser.msie && parseInt(jQuery.browser.version) == 6) {
+ if (navigator.appName == 'Microsoft Internet Explorer' && navigator.userAgent.match(/msie 6/i)) {
$('li', $adminMenu).hover(
function () {
$(this).addClass('iehover');
}); //close function and hover!

Don't really know the state of this issue, but I still have it with all up to date and greeny.

jcisio’s picture

jcisio commented
Status: Closed (duplicate) » Closed (won't fix)

Correct issue status.

markhalliwell’s picture

markhalliwell
Primary language English
commented
Status: Closed (won't fix) » Closed (duplicate)

This issue duplicates the efforts in #1892074: remove $.browser (Remove IE 6 support) (which were added to the project).

scott.browne’s picture

scott.browne commented

Thanks for this quick fix. I found it very quickly too! Happy me!