[buddypress-trac] [BuddyPress Trac] #3419: bp_core_remove_nav_item() support WP admin bar as well?

buddypress-trac noreply at wordpress.org
Tue Aug 23 19:55:38 UTC 2016


#3419: bp_core_remove_nav_item() support WP admin bar as well?
------------------------------------+-----------------------
 Reporter:  r-a-y                   |       Owner:
     Type:  enhancement             |      Status:  reopened
 Priority:  normal                  |   Milestone:  2.7
Component:  Core                    |     Version:  1.5
 Severity:  normal                  |  Resolution:
 Keywords:  dev-feedback has-patch  |
------------------------------------+-----------------------
Changes (by r-a-y):

 * keywords:  reporter-feedback dev-feedback has-patch => dev-feedback has-
     patch


Comment:

 @imath - I like `02.patch` quite a bit!

 I've made some changes to it:
 - `'show_in_admin_bar'` now defaults to `true`.
 - I created a helper method - `BP_Component::setup_admin_nav()` - so we
 won't have to duplicate so much code when used across all components.
 - Some components like the Settings component have dynamic nav items that
 are only applicable when on a profile page.  The following admin bar items
 are still registered manually - `Settings > Email`, `Settings > Delete
 Account`, `Groups > Create a Group`.

--
Ticket URL: <https://buddypress.trac.wordpress.org/ticket/3419#comment:13>
BuddyPress Trac <http://buddypress.org/>
BuddyPress Trac


More information about the buddypress-trac mailing list