I set Maximum number of menu levels to display to 2, but superfish menu still show them all.
How to fix this? or it's my setting wrong?

CommentFileSizeAuthor
#4 2729165-superfish-max-depth.patch1.22 KBedb
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lilee created an issue. See original summary.

lilee’s picture

Component: Code » User interface
Anonymous’s picture

I had the same problem at first, but you can use one of Superfish' own settings to solve that.

Open the Superfish menu block from admin/structure/block and open the Advanced Settings section in the configuration form. Inside the Superfish section you'll find a Path Levels dropdown. Change this according to your needs and it should work.

I'm not sure if that's the best solution, as the default settings should be taken into account too, but for new it works (for me).

edb’s picture

Version: 8.x-1.0-rc2 » 8.x-1.x-dev
Component: User interface » Code
Assigned: lilee » edb
Category: Feature request » Bug report
Status: Active » Needs review
FileSize
1.22 KB

The issue here is the expand tree method does not use the maximum depth setting, it just expands all children recursively. I have patched it and it looks to be working fine now. Please review.

Jonah Fenn’s picture

Tested patch on rc4 and it worked great. Thanks!

  • mehrpadin committed 759e47a on 8.x-1.x
    Issue #2729165 by edb: Fixed the 'maximum depth' option.
    
DuaelFr’s picture

Status: Needs review » Fixed

This patch has been commited.
(I'm just cleaning the issue queue)

Status: Fixed » Closed (fixed)

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