[wp-trac] [WordPress Trac] #47053: Accessibility: Need to set proper 'tabindex' in 'Skip To Toolbar' HTML

[WordPress Trac]

#47053: Accessibility: Need to set proper 'tabindex' in 'Skip To Toolbar' HTML
-------------------------------------------------+-------------------------
 Reporter:  jankimoradiya                        |       Owner:  joedolson
     Type:  defect (bug)                         |      Status:  assigned
 Priority:  normal                               |   Milestone:  5.3
Component:  Toolbar                              |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  has-screenshots has-patch needs-     |     Focuses:  ui,
  testing                                        |  accessibility
-------------------------------------------------+-------------------------

Comment (by afercia):

 The scripting part is very old and basically not used as the vast majority
 of themes do use jQuery on the front end. Except Twenty Nineteen :) So
 this moving-via-JS doesn't really happen since years on most of the
 themes. It happens on Twenty Nineteen, even if the admin bar is _already_
 printed out at the bottom of the source.

 After some software archeology, going through relevant commits:
 https://core.trac.wordpress.org/changeset/17263
 https://core.trac.wordpress.org/changeset/16070
 https://core.trac.wordpress.org/changeset/16038

 I landed on this ticket:
 https://core.trac.wordpress.org/ticket/14772#comment:71
 > On the non-debug menu, JS does the following:
 >  - Append the admin bar as a child of the body element.

 and I really don't know what the "non-debug menu" is / was. Maybe someone
 with good historical knowledge may know. @azaozz? :)

 Personally, I think it can be safely removed.

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