[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 adamsilverstein):

 I did some testing of the code with Twenty Seventeen and various browsers.
 Mostly things seemed to work well, however I don't really like the way it
 looks overlaid on the video, it seems like this needs more design review.
 I'm not sure I even like having control buttons, i'm curious what others
 think.

 I notice a couple of issues as well:

 * The button did not work correctly in windows 7/ie 9, I see a console
 error:
 https://cl.ly/0g091W0l1Y2K/Dashboard_2016-11-11_17-00-01.jpg
 (Video doesn't play in ie8 in my testing)

 * When I click the button before the video loads, it doesn't work and I
 sometimes got the video in a hung state. Can we hide or disable the button
 until the video starts playing? This is pretty easy to reproduce if you
 throttle your connection in chrome inspector.

 [[Image(https://cl.ly/040L1C1z2t04/wpdev__Just_another_WordPress_site_2016-11-11_17-09-27.jpg)]]

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