[wp-trac] [WordPress Trac] #25425: Last separator in network and user menus not removed

WordPress Trac noreply at wordpress.org
Fri Sep 27 14:49:19 UTC 2013


#25425: Last separator in network and user menus not removed
-----------------------------+------------------------------
 Reporter:  johnjamesjacoby  |       Owner:
     Type:  defect (bug)     |      Status:  new
 Priority:  normal           |   Milestone:  Awaiting Review
Component:  Network Admin    |     Version:  3.0
 Severity:  normal           |  Resolution:
 Keywords:  has-patch        |
-----------------------------+------------------------------
Description changed by johnjamesjacoby:

Old description:

> In `/wp-admin/network/` and `/wp-admin/user/` the last separator is not
> trimmed off.
>
> This is caused by the ID of those separators being `wp-menu-separator-
> last` and not `wp-menu-separator`
>
> Screen shots attached to outline the problem.
>
> * First patch renames the final separators, since `wp-menu-separator-
> last` is not referenced anywhere else in WordPress core.
> * Second patch keeps the ID's the same (for backpat with who-knows-what)
> and tweaks the code that removes the final separator in `/wp-
> admin/includes/menu.php` instead.
> * Bonus patch to adjust a separator capability from `read` to `exist` to
> match the other two.
>
> Related'ish: #23182, #24104.

New description:

 In `/wp-admin/network/` and `/wp-admin/user/` the last separator is not
 trimmed off.

 This is caused by the ID of those separators being `wp-menu-separator-
 last` and not `wp-menu-separator`

 Screen shots attached to outline the problem.

 * First patch renames the final separators, since `wp-menu-separator-last`
 is not referenced anywhere else in WordPress core.
 * Second patch keeps the ID's the same (for backpat with who-knows-what)
 and tweaks the code that removes the final separator in `/wp-
 admin/includes/menu.php` instead.
 * Bonus patch to adjust a separator capability from `read` to `exist` to
 match the other two.
 * Final patch combines all three patches together.

 Related'ish: #23182, #24104.

--

--
Ticket URL: <http://core.trac.wordpress.org/ticket/25425#comment:1>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list