[wp-trac] [WordPress Trac] #42078: Customize: fix the color hue picker HTML and accessibility

WordPress Trac noreply at wordpress.org
Sat Dec 9 21:38:49 UTC 2017


#42078: Customize: fix the color hue picker HTML and accessibility
---------------------------------------+-----------------------------
 Reporter:  afercia                    |       Owner:
     Type:  defect (bug)               |      Status:  new
 Priority:  normal                     |   Milestone:  Future Release
Component:  Customize                  |     Version:  4.7
 Severity:  normal                     |  Resolution:
 Keywords:  has-screenshots has-patch  |     Focuses:  accessibility
---------------------------------------+-----------------------------

Comment (by dlh):

 [attachment:42078.diff] builds on the previous patches with approaches for
 almost all of the issues described.

 > the label is actually empty

 This appears to be a bug in Twenty Seventeen, which doesn't pass a label
 to the control. I assume that no label is used because the control is
 meant to be visually part of the "Color Scheme" control above it.

 The patch adds a label along with JavaScript that applies the `screen-
 reader-text` class to maintain the existing appearance.

 > it should not wrap the hidden input field and should use for/id
 attributes as per the WordPress accessibility standard

 The patch moves the hidden input field out of the label and adds the
 attributes.

 > the iris picker should not be inside the label

 The change to the label in the patch should also cause the Iris picker to
 appear outside the label.

 > the hidden input field should not be hidden, and should be used as an
 alternative way to enter a hue value

 I encountered the same difficulty as @subrataemfluence here: The field
 responds to the slider, but not to manual input. If I had to guess, I
 would say the `_addInputListeners()` function in `iris.min.js` isn't
 handling the values correctly, but that's only a guess.

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


More information about the wp-trac mailing list