[wp-trac] [WordPress Trac] #31487: Semantic elements for non-link links: /wp-admin/customize.php

[WordPress Trac]

#31487: Semantic elements for non-link links: /wp-admin/customize.php
--------------------------+--------------------------------------------
 Reporter:  Cheffheid     |       Owner:  afercia
     Type:  defect (bug)  |      Status:  closed
 Priority:  normal        |   Milestone:  4.6
Component:  Customize     |     Version:
 Severity:  normal        |  Resolution:  fixed
 Keywords:  has-patch     |     Focuses:  ui, accessibility, javascript
--------------------------+--------------------------------------------
Changes (by afercia):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"37230"]:
 {{{
 #!CommitTicketReference repository="" revision="37230"
 Accessibility: Customizer, improve UI controls in `customize.php`

 - makes the mobile preview/customize toggle a button
 - changes the "Close" link hidden text from 'Cancel' to 'Close the
 Customizer and go back to the previous page'
 - adds a missing `type="button"` attribute
 - removes unnecessary `keydown` events and `preventDefault()`: buttons
 don't need a keydown event and when they have a `type="button"` attribute
 there's no default action to prevent

 Props Cheffheid, afercia.

 Fixes #31487.
 }}}

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