Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
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.
Comment | File | Size | Author |
---|---|---|---|
#6 | 1985486-6.patch | 739 bytes | vuil |
Comments
Comment #1
gitesh.koli CreditAttribution: gitesh.koli commentedPatch for the reported issue
Comment #2
julien.reulos CreditAttribution: julien.reulos commentedBrillant! 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!
Comment #3
rumblewand CreditAttribution: rumblewand commentedWorked for me as well. DHTML did not work out of the box. The above patch fixed it in the latest Drupal 7 release.
Comment #4
Christopher James Francis Rodgers CreditAttribution: Christopher James Francis Rodgers commentedWould 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.
###
Comment #5
SGhosh CreditAttribution: SGhosh commentedConfirming. The module doesn't work without this patch.
Comment #6
vuilReroll the patch of #1.
Comment #8
vuilAlready submitted to the latest 7.x-1.x-dev branch.
Comment #10
vuil