[wp-trac] [WordPress Trac] #38678: Video Headers: Improve Basic Accessibility

[WordPress Trac]

#38678: Video Headers: Improve Basic Accessibility
---------------------------+----------------------------
 Reporter:  davidakennedy  |       Owner:
     Type:  defect (bug)   |      Status:  new
 Priority:  normal         |   Milestone:  4.7
Component:  Themes         |     Version:
 Severity:  normal         |  Resolution:
 Keywords:  has-patch      |     Focuses:  accessibility
---------------------------+----------------------------

Comment (by afercia):

 Tested a bit. Couple of thoughts.

 For now the button is hidden from assistive technologies and intended to
 be used with the keyboard. Thinking the button should be revealed on focus
 then, unless I'm missing something. Also, when the video is a youtube
 video it should probably use `aria-hidden="true"` as well, maybe
 dynamically adding the attribute on the video iframe or wrapping the
 iframe in a container with `aria-hidden="true" could work?

 Ideally, also Youtube videos should have a Pause/Play button. I've played
 a bit with it, not a great expoert about YT APIs, but seems pretty simple,
 unless I'm missing something. See:

 Playback controls:
 https://developers.google.com/youtube/iframe_api_reference#Playback_controls

 How to check the playback status:
 https://developers.google.com/youtube/iframe_api_reference#Playback_status

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