Menu item settings show up at any level but menu_minipanels_init only supports top level menus. This is a bit troublesome if you want to use your secondary nav as the menu items triggering the minipanel megamenu's.

Comments

damienmckenna’s picture

Status: Active » Postponed (maintainer needs more info)

Please explain more about your self. How are you loading the secondary navigation - via the normal Drupal secondary-menu mechanisms?

neclimdul’s picture

Yeah, sorry I wasn't clear. The site I was hoping to use this on has a primary level of navigation that contains 3 top level sections of the site and then the secondary navigation(the output of menu_secondary_links()) is actually the "primary" navigation for each section of the site. Because the code in menu_minipanels_init() iterates over each top level of the menus but never goes any deeper, the settings for any menus deeper in the structure never get added to the page even though they exist. The theme functions do correctly add the classes though.

damienmckenna’s picture

Assigned: Unassigned » damienmckenna
Status: Postponed (maintainer needs more info) » Active

Thanks for clearing it up, I see now where the problem is. I'll have to investigate menu_navigation_links() further and work out how to present that in the UI.

damienmckenna’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev
Status: Active » Needs review
StatusFileSize
new1.42 KB

This checks if the primary menu is enabled, and then checks if the primary and secondary menus are set to the same menu, then if so it loads the second level of that menu.

damienmckenna’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
StatusFileSize
new1.42 KB

Corrected the defaults for the menu source variables.

damienmckenna’s picture

Version: 6.x-1.x-dev » 7.x-1.x-dev

Wrong branch for #5.

Status: Needs review » Needs work

The last submitted patch, menu_minipanels-n1633136-5-d7.patch, failed testing.

damienmckenna’s picture

Status: Needs work » Needs review

#5: menu_minipanels-n1633136-5-d7.patch queued for re-testing.

damienmckenna’s picture

Version: 7.x-1.x-dev » 6.x-1.x-dev
StatusFileSize
new1.37 KB

D6 version of #5.

damienmckenna’s picture

Status: Needs review » Fixed

Committed.

Status: Fixed » Closed (fixed)

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

damienmckenna’s picture

Assigned: damienmckenna » Unassigned
Issue summary: View changes