[buddypress-trac] [BuddyPress] #4629: New function which combines primary/secondary nav menus into one

buddypress-trac noreply at wordpress.org
Sun Oct 28 20:33:55 UTC 2012


#4629: New function which combines primary/secondary nav menus into one
-------------------------+------------------------------
 Reporter:  DJPaul       |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Core         |     Version:
 Severity:  normal       |  Resolution:
 Keywords:               |
-------------------------+------------------------------

Comment (by DJPaul):

 Replying to [comment:7 modemlooper]:
 >Why create the function bp_nav_menu when wp_nav_menu exists?
 wp_nav_menu always goes and finds a (WordPress) Menu, finds a theme
 location, and gets the Menu's items. That's overhead that we don't need,
 and there's also some logic in the WordPress function that bails out if it
 can't find a menu or if the menu it finds has no menu items, which we
 can't work around.

 If/when in a future release, BP registers a menu location and integrates
 fully into WordPress' Menus admin screen, I think there's a reasonable
 chance that bp_nav_menu() could become a simple wrapper for wp_nav_menu(),
 but I don't think it's possible yet.

-- 
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/4629#comment:8>
BuddyPress <http://buddypress.org/>
BuddyPress


More information about the buddypress-trac mailing list