Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This module breaks the special menu items module as it overwrites the nolink and separator string. This same issue happens in superfish menu system.
There was a similar fix here:
https://drupal.org/node/1273602
tb_megamenu has to account for rewritten strings such as nolink and separator.
screenshot: http://prntscr.com/19shja
Comment | File | Size | Author |
---|---|---|---|
#11 | tb_megamenu-special_menu_items_integration-2019163-11-D7.patch | 1.89 KB | VladSavitsky |
Comments
Comment #1
themebrain CreditAttribution: themebrain commentedHi duckx,
Thanks for your information. This is in fact a special issue we need to take care of, since this is clashing between modules. Next release will have this fixed!
Comment #2
themebrain CreditAttribution: themebrain commentedDear Duckx,
Our alpha version has been released and fixed your reported bug. Please have a look and let me know if there's any other problem.
Cheers and have a nice weekend!
Comment #3
themebrain CreditAttribution: themebrain commentedComment #4
patoshi CreditAttribution: patoshi commentedthanks! really really amazing job on the module.. its the best one i would say...
also, you fixed the nolink> URL option, but there is also a separator> one too.
Comment #5
t_radigan CreditAttribution: t_radigan commentedI have the DEV version of this module installed but the 'nolink' option does not appear to function as expected. My URLs are still getting written as
http://<sitename>/<nolink>
. Any ideas? Is this issue not fixed in DEV? Only the Alpha release?Comment #6.0
(not verified) CreditAttribution: commentednew
Comment #7
tenuis CreditAttribution: tenuis commentedI wanted to use TB Mega Menu with special menu items, so I have modified a file to make it work (it's a WIP, as the CSS have to be updated as well to style those elements and the separator could be something else), I share it now in case someone is in need to use this ASAP.
In file : modules/tb_megamenu/templates/tb-megamenu-item.tpl.php
Comment #8
andjsmit CreditAttribution: andjsmit commented#7 works great for me. Only addition I made was to include a class on the no link span and remove redundant no link check.
Comment #9
loparr CreditAttribution: loparr commentedHi,
I confirm #8 works (class on nolink span is useful). This should be committed.
Comment #10
VladSavitsky CreditAttribution: VladSavitsky commentedVersion 7.x-1.0-rc2 bug appears. Please reopen issue.
Comment #11
VladSavitsky CreditAttribution: VladSavitsky commentedI've used code from comment 8 to make the patch.