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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

rimen’s picture

rimen CreditAttribution: rimen commented
FileSize
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 CreditAttribution: rimen commented
FileSize
dhtml_menu.js.txt10.96 KB

.txt

julien.reulos’s picture

julien.reulos CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: nagy.balint commented
FileSize
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 CreditAttribution: acidaniel commented

the #6 works really well thanks

pacome’s picture

pacome CreditAttribution: pacome commented

Thanks for patch #6 !
-p-

bensey’s picture

bensey CreditAttribution: bensey commented

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

vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
CreditAttribution: vuil at Bulpros Consulting 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 🇧🇬 🇪🇺 🌍
CreditAttribution: vuil at Bulpros Consulting commented
Assigned: jessejesse » Unassigned

I unassigned the issue of @jessejesse.