[wp-trac] [WordPress Trac] #41104: Add "Site" to the "New" admin-bar dropdown

WordPress Trac noreply at wordpress.org
Mon Feb 12 19:55:57 UTC 2024


#41104: Add "Site" to the "New" admin-bar dropdown
-------------------------------------------------+-------------------------
 Reporter:  johnjamesjacoby                      |       Owner:  joedolson
     Type:  enhancement                          |      Status:  accepted
 Priority:  normal                               |   Milestone:  6.5
Component:  Toolbar                              |     Version:  3.0
 Severity:  normal                               |  Resolution:
 Keywords:  has-patch has-testing-info 2nd-      |     Focuses:  multisite
  opinion                                        |
-------------------------------------------------+-------------------------
Changes (by audrasjb):

 * keywords:  has-patch needs-testing has-testing-info 2nd-opinion => has-
     patch has-testing-info 2nd-opinion


Comment:

 I tested both patch and they work fine.

 I prefer the approach in `41104.2.diff`. Adding a separator is not
 mandatory since multisite owners probably know what a new Site is in their
 context. Plus, it only shows for users who are able to create new sites
 (admin/super admins).

 Also, the separator in the alternate patch looks a bit weird (I think the
 bottom padding of the "User" item is too small).

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/41104#comment:13>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list