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

[WordPress Trac]

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

Comment (by bradyvercher):

 I think the entire script is short-circuited in IE8 since it doesn't
 support `addEventListener`, so the video or button shouldn't ever be
 injected in the DOM.

 Currently, the button is injected after the video is added to the DOM.
 Inserting it later would require bubbling up an event from the handler
 when a video can be played. For native video, that would be the `canplay`
 event. For YouTube, that would probably be in the `onReady` callback.

 @afercia That postMesage error isn't related to the Customizer. I'm pretty
 sure it's related to the `origin`
 [https://developers.google.com/youtube/player_parameters#origin parameter
 in the YouTube API], but it shouldn't affect playback.

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