[wp-trac] [WordPress Trac] #36337: Shift-Click to Edit on Custom Logo selects "Remove" button.

[WordPress Trac]

#36337: Shift-Click to Edit on Custom Logo selects "Remove" button.
-------------------------------------+--------------------------------
 Reporter:  mikeschroder             |       Owner:
     Type:  defect (bug)             |      Status:  new
 Priority:  normal                   |   Milestone:  Awaiting Review
Component:  Customize                |     Version:  trunk
 Severity:  normal                   |  Resolution:
 Keywords:  needs-patch 2nd-opinion  |     Focuses:  ui, accessibility
-------------------------------------+--------------------------------

Comment (by adamsilverstein):

 This happens because the focus is designed to hit the first visible button
 or link.

 [attachment:36337.diff] fixes this by explicitly excluding elements with
 the class `remove-button`. After applying the patch, shift clicking the
 logo focuses the 'Change Logo' button. This may not be the best solution;
 I think a more robust solution would enable sections to specify a focus
 target that would override the default 'first button' behavior.

 [[Image(http://cl.ly/3X1W0B1b2k0L/Customize_Dev_Test__Just_another_WordPress_site_2016-03-25_17-53-22.jpg)]]

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