If you click a menu item that is not a tree, Firebug will report an error (right before the page changes to go to your requested click) about "elem" not having properties or being defined, or some such. Adding in breakpoints suggests line 27:

   Drupal.dhtmlMenu.switchMenu($('#'+ id)[0], $(this).parents()[0]);

I suspect doing something like "if clicked menu has class 'leaf', don't fire." would solve this.

CommentFileSizeAuthor
#1 selectorfix.patch611 bytesmorbus iff

Comments

morbus iff’s picture

morbus iff
they/them
commented
Status: Active » Needs review
StatusFileSize
new selectorfix.patch611 bytes

Patch attached.

Anonymous’s picture

Anonymous (not verified) commented
Status: Needs review » Fixed

fixed in HEAD.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)
cburschka’s picture

cburschka
they
commented
Status: Closed (fixed) » Closed (duplicate)

I originally thought http://drupal.org/node/157718 was a regression of this, but examination of the CVS log reveals that this patch was never actually implemented. The only bugfix commit in that time frame is 1.8, and did not contain it.

The other issue has been fixed, so this can be marked a dupe now.