[wp-trac] [WordPress Trac] #36325: Move #wpadminbar above #page

WordPress Trac noreply at wordpress.org
Fri Sep 1 07:09:02 UTC 2017


#36325: Move #wpadminbar above #page
-------------------------+---------------------------------
 Reporter:  wewlad       |       Owner:
     Type:  enhancement  |      Status:  new
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Toolbar      |     Version:  4.4.2
 Severity:  normal       |  Resolution:
 Keywords:               |     Focuses:  ui, administration
-------------------------+---------------------------------

Comment (by danieltj):

 Replying to [comment:2 cbutlerjr]:
 > I can understand how this might be an issue.  However, I'm not sure that
 moving this ID earlier in the stream is the best solution.
 >
 > For starters, there's no requirement to place the toolbar at the top of
 the page. It can be positioned at the bottom, and that can be done with
 just CSS.  Additionally, from a document perspective, I'm not sure I would
 agree that it makes much sense to have all of the toolbar HTML at the top
 of the document. That has the potential for an ugly page load in certain
 situations.
 >
 > This isn't really a bug so much as it is an enhancement request.

 A setting or hook might be a better idea in terms of moving the toolbar
 but simply changing its position will mean that page of the page is loaded
 before the real content. I've also never had an issue with moving things
 around the toolbar because there's an `.admin-bar` class on the `body` tag
 so you can still style the cascade.

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


More information about the wp-trac mailing list