This only appears to happen with a certain combination of DHTML Menu settings. Here's how to replicate:
1) fresh install of Drupal 6.6;
2) turn on DHTML Menu;
3) in DHTML Menu Settings, uncheck every box;
4) now, check "Keep only one menu open at a time." it should be the only one on.
5) save configuration.
Now, when you click a menu item with children, it still dynamically loads the children, but it also loads the page you just clicked on, which makes the whole thing pretty much moot. The default link behavior isn't being intercepted anymore. If you click on an already expanded menu item, though, it does still close the tree without loading a new page. Same for IE7 and FF3.
This bug only happens with "Animated sliding effect" turned off. If you were to turn that on, the module works as expected again.
| Comment | File | Size | Author |
|---|---|---|---|
| #1 | dhtml_menu_337242.patch | 497 bytes | les lim |
Comments
Comment #1
les limFound it. There's a syntax error in dhtml_menu.js that causes a break in execution before it gets to return false. Patch is attached. Since it's just a typo fix, I'm guessing it's okay that we skip the regular patch review, but someone should feel free to change the status if that's not kosher.
Comment #2
les limoops.
Comment #3
cburschkaCommitted to DRUPAL-6--3. Thank you for your help! =)
This part of the code has been rewritten in each current branch, so this is the only branch that has the typo.