[wp-trac] [WordPress Trac] #32678: Audit toolbar links and content

WordPress Trac noreply at wordpress.org
Sat May 7 21:51:34 UTC 2016


#32678: Audit toolbar links and content
-------------------------------------------------+-------------------------
 Reporter:  helen                                |       Owner:  helen
     Type:  enhancement                          |      Status:  assigned
 Priority:  normal                               |   Milestone:  Future
Component:  Toolbar                              |  Release
 Severity:  normal                               |     Version:
 Keywords:  has-patch make-flow needs-testing    |  Resolution:
  has-screenshots needs-refresh                  |     Focuses:  ui
-------------------------------------------------+-------------------------

Comment (by celloexpressions):

 @kosvrouvas [attachment:32678.5.diff] is a better starting point than the
 plugin - the plugin was derived from that but has some issues such as the
 one you pointed out above. You don't need to patch the minified files -
 they're generated automatically. To force use of unminified scripts,
 `define( 'SCRIPT_DEBUG', true );` in wp-config.php.

 The patch only needs a few tweaks to match the [attachment:wpadmin-bar-
 concept-i10.png] concept that we seem fairly agreed upon (see also
 [comment:50]). But I think the bigger thing that @helen wants explored is
 how to better integrate the admin menu with the admin bar on a technical
 level. My preference would be to lean more on the admin bar API than the
 admin menu "API", since it's structured better, but not sure how exactly
 those could be merged best. There are also some related changes besides
 the changes to toolbar contents that have been proposed, but I'm  not sure
 whether or not it makes sense to hold off on the content changes for that.

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


More information about the wp-trac mailing list