Drupal core has theme_menu_link, whcih can be easily used to add special classes etc. to menu items.

Superfish has no such thing, you have to override theme_superfish_build(), which is excessive.

It would be nice even to just wrap the l() call in a theme function for easy override.

#1 superfish-theme_link-1534734-1.patch1.89 KBrooby
Members fund testing for the Drupal project. Drupal Association Learn more


rooby’s picture

Status: Active » Needs review
1.89 KB

Here is a patch to add said theme function.

mehrpadin’s picture

Hey Reuben,

Thanks for this :) added to my to-do list for review.

jeffschuler’s picture

Status: Needs review » Reviewed & tested by the community

@rooby: I needed this too.
Patch looks good and works as designed. Thanks!

markcarver’s picture

Category: feature » support
Priority: Normal » Major

#1 works perfectly and needs to be committed ASAP. This is a major hindrance in providing integration support between Menu Views and Superfish.

#1617802-4: Create a release from 7.x-1.x-dev to support Menu Views
#1404802-10: Not working with Superfish.

samhassell’s picture

+1 we need this too :)

mehrpadin’s picture

Added! please try the development release :) thanks for everything!

mehrpadin’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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

kenorb’s picture