Originally I posted this as an issue with Colorbox, but Im thinking now that it is a problem with DHTML Menu.

When setting "Static Navigation" in DHTML Menu Settings, to "None"... everything works fine initially. But after opening a Colorbox on a page, and closing it, the menu no longer expands and collapses. Refreshing the page fixes it. When "Static Navigation" is set to anything else it works as expected. I really would like it set to None however.

I'm using the latest dev releases of DHTML Menu, Colorbox and Jquery Update.

Any thoughts?

CommentFileSizeAuthor
#6 fix-trigger-duplication-1495496-6.patch1.99 KBnagy.balint
#2 dhtml_menu.txt10.96 KBrimen
#1 dhtml_menu.zip3.47 KBrimen

Comments

rimen’s picture

rimen commented
StatusFileSize
new dhtml_menu.zip3.47 KB

This is a patched dhrml_menu.js

Added .dhtml-processed to prevent re-attach listeners and double-click option fixed.

rimen’s picture

rimen commented
StatusFileSize
new dhtml_menu.js.txt10.96 KB

.txt

julien.reulos’s picture

julien.reulos commented
Status: Active » Reviewed & tested by the community

Nice! The conflict with Colorbox module was affecting whatever Static Navigation option you would apply to your dhtml menu.
With this js version, both modules are playing well together.

xl_cheese’s picture

xl_cheese commented
Issue summary: View changes

I tried to use this to fix my issue, but it didn't make a difference.

https://drupal.org/node/2208577

christopher james francis rodgers’s picture

christopher james francis rodgers commented

this module is the only one I have installed on my D7 core
and so I understand that the various settings for it can drive you
crazy.

All I can say is that the options that I stick with are set out
as best I could do at https://drupal.org/node/2126031.

...the only options I use ...

... I know that the 'other' various options are
goofy... if not a waste of time ...

And believe me .. the key-board strokes I now make
are intended only to possibly help you, my friends...

nagy.balint’s picture

nagy.balint commented
StatusFileSize
new fix-trigger-duplication-1495496-6.patch1.99 KB

Created a patch from the above.

Did not carry over the removal of the if block that checks for the margin-left class as that would be a change that breaks existing designs.

However with this patch now if using an ajax view or colorbox, seems to work fine.

acidaniel’s picture

acidaniel commented

the #6 works really well thanks

pacome’s picture

pacome commented

Thanks for patch #6 !
-p-

bensey’s picture

bensey commented

Patch in #6 also works well for me.
Thanks!

vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented
Status: Reviewed & tested by the community » Needs work

The patch needs re-roll to work on the latest 7.x-1.x-dev branch.

vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented
Assigned: jessejesse » Unassigned

I unassigned the issue of @jessejesse.