I had an inconsistent behaviour with the dhtml_menu_preprocess_menu_link function not getting picked up immediately. Had to clear cache multiple times to get this to work.

problem seemed because the dhtml_menu.theme.inc file was included in the dhtml_menu_init function

I moved it out of there and it seemed to work fine.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gitesh.koli’s picture

Patch for the reported issue

julien.reulos’s picture

Status: Active » Needs review

Brillant! The patch solves all the issues I had with this module, in particular since my Drupal site has "Aggregate and compress CSS files" and "Aggregate JavaScript files" options activated. I can now use any of the Static navigation options: Expand on bullet, Expand on hover,...
Thanks!

rumblewand’s picture

Worked for me as well. DHTML did not work out of the box. The above patch fixed it in the latest Drupal 7 release.

Christopher James Francis Rodgers’s picture

Would someone kindly

- add this patch ;
- post new dev ;

===

Yes, I could learn what 'patching' entails,
but life is short.

I seek only to be an end-user,
and teacher for all day-one users
until Drupal becomes completely user-friendly
for all day-one users ...so to speak.

: o )

- Chris

backTo: https://drupal.org/node/1933800

===

Please, would someone kindly take command
of this module.

I am already crying to think
DHTML Menu might not make it to d8.

Thanking you in advance
for your attention, time, and help.

###

SGhosh’s picture

Confirming. The module doesn't work without this patch.

vuil’s picture

Issue summary: View changes
FileSize
739 bytes

Reroll the patch of #1.

  • ilchovuchkov committed 28bd03e on 7.x-1.x
    Issue #1985486 by ilchovuchkov, giteshkoli: Menu link preprocess...
vuil’s picture

Status: Needs review » Fixed

Already submitted to the latest 7.x-1.x-dev branch.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.

vuil’s picture