In IE7 I'm getting error "Object does not support this property or method". 

Drupal.admin.attachBehaviors = function (context, settings, $adminMenu) {
  if ($adminMenu.length) {
    $adminMenu.addClass('admin-menu-processed');
    $.each(Drupal.admin.behaviors, function() {
      this(context, settings, $adminMenu);  // in this line error happens
    });
  }
};

I have fixed it for ie, look at my patch maybe you can provide better solution.

Thanks.

CommentFileSizeAuthor
admin_menu-ie-fix.patch549 bytesddrozdik

Comments

robinwal’s picture

robinwal commented

Thanks for your patch! It finally gets rid of the error message. However, since I applied the patch I got an extended slide bar at the bottom of the window by approximately 20px or something. Does anyone else experience the same thing? I'm using a very customized zen theme, so it might also be a problem in there, but I just want to double check first!

thanks

ddrozdik’s picture

ddrozdik commented

We have same problem on all our sites in all browsers, and this problem exist from last or penultimate version of module(http://drupal.org/node/1683132), and we fix it in our custom css. This is not a problem from my patch.

vinmassaro’s picture

vinmassaro commented

This appears to be fixed in 7.x-3.0-rc4.

truls1502’s picture

truls1502
he/him
Primary language Norwegian Bokmål
commented
Issue summary: View changes
Status: Active » Postponed (maintainer needs more info)
Issue tags: +postponed2w

I am sorry for no reply until now.

There are many issues regarding this module admin_menu which is a bit difficult for us to follow up since some of the issues might be already outdated, or is already fixed by the module or any other modules or itself core which means that the problem might no longer need to be fixed.

We can see that the issue has been created for a few years ago, I hope it is okay for you that I am postponing the issue, and give you around two weeks. If you still face the problem, could you tell us the step by step when until you get the error message or what is frustrated you, and a list of modules you are using related to admin_menu and a screenshot that might help us? So it makes us easier to reproduce your issue.

However, after two weeks with no feedback - we will close this issue. So in case, you noticed it after the issue is closed, do not hesitate to reopen it like and fill information which is mentioned above.

So before giving us a feedback, do you mind to test it again with our latest 7.x-3.x-dev? Since on #3 mentioned it is already fixed?

Thank you for understanding! :)

ddrozdik’s picture

ddrozdik commented

5 years later... :)

I think we can safely close this issue since IE7 is almost unused already

truls1502’s picture

truls1502
he/him
Primary language Norwegian Bokmål
commented
Priority: Major » Normal
Status: Postponed (maintainer needs more info) » Closed (outdated)

Yes, I am sorry. I have recently jumped into the project, so it is not easy to handle with quite old issues.

However, thank you for letting us know. Again, if you remember or someone here has the similar experience. Then just open the issue and change the status :)

Then I will close this issue now.

truls1502’s picture

truls1502
he/him
Primary language Norwegian Bokmål
commented
Issue tags: -postponed2w