Drupal 7's menus support the ability to include in page anchors and append query strings to menu links. This module strips them out in theme_superfish_build(), by not including them in the $link_options array before passing it to l().
The link_options array structure should look something like this when it is passed into l() :
array(
'attributes' => array( [array contents] ),
'fragment' => array( [array contents] ),
'query' => array( [array contents] ),
);
I'm going to do my best to write a patch for this in the next few days if I can find the time to do it.
Comments
Comment #1
mnlund commentedThat's true! I thought it was the menu system that stripped it out, and didn't realize this is a feature in D7. Fantastic! If you change line 1005 to
$link_options += $menu_item['link']['localized_options']the attributes, query and fragment will be merged into the $link_options.Comment #2
amitaibuHere's the patch based on #1, which works.
Comment #3
baby.hack commentedThank you so much. My page anchors were working fine before Superfish. I came straight to this tread and applied patch from #2. Problem solved!
Comment #4
jm.federico commentedScaling to major!
A site with broken links is the same as no site!
Patch works
Comment #5
mehrpadin commentedHey everybody,
Patch applied as of v1.9-beta3, thank you all :)