Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
The small plus buttons that expand a menu in the admin toolbar don't work when overlay is present on a page.
That's the only problem I've noticed with admin's support for overlay.
Comment | File | Size | Author |
---|---|---|---|
#7 | admin.patch | 438 bytes | jurgenhaas |
#1 | admin-927944-overlay.diff | 772 bytes | skwashd |
Comments
Comment #1
skwashd CreditAttribution: skwashd commentedThe attached patch fixes the problem. The new <a> needs the overlay-exclude class to stop the overlay module hijacking the click event on it.
Comment #2
yhahn CreditAttribution: yhahn commentedThanks. http://drupal.org/cvs?commit=465668
Comment #4
drclaw CreditAttribution: drclaw commentedI'm still unable to expand the menu items once the overlay has been accessed... Am I the only one...?
Comment #5
caschbre CreditAttribution: caschbre commentedAre you using the jquery_update module by chance? I'm running into a similar issue that is a combination of overlay, jquery_update, and admin.
#1064198: jQuery 1.5 and Devel Module
Comment #6
drclaw CreditAttribution: drclaw commentedNope, no jQuery update. Just admin and overlay.
Comment #7
jurgenhaasHere you go, I finally found the real issue and the patch that resolves it is attached.
The problem is that there is a menu.data('disableMenu', true); being used when clicking on a link, which is required to prevent the menu.drilldown('setActive') being called, because otherwise the link wouldn't be opened. Normally, this is no problem, because a new page gets loaded and the assigned data gets lost anyway.
However, if overlay is active, the page doesn't get reloaded and the data disableMenu=true remains assigned to the menu which prevents further execution of menu.drilldown
Well, I believe that disabling the menu is only required once, so the solution in the patch just unsets that setting immediately and I now have it working nicely and as expected.
Comment #8
paulgemini CreditAttribution: paulgemini commentedDidn't fix it for me.
Comment #9
nithinkolekar CreditAttribution: nithinkolekar commented#7 patch worked with 7.x-2.0-beta3+8-dev , Jquery > 1.7 and Drupal core 7.34.