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

 After more thought and review from the accessibility review, having the
 video container and button hidden with `aria-hidden` can potentially cause
 problems. Using a screen reader doesn't necessarily mean somebody is
 totally blind; so the video could still be a visual problem, even if
 there's no sound.

 In this case it's better to make the same content available to all users.

 In [attachment:38678.9.patch], the `aria-hidden` on the button is removed,
 bouncing off of [attachment:38678.8.diff] by @bradyvercher. Worth noting,
 I didn't have to remove the `aria-hidden` on the video container because
 it looks like it was inadvertently removed somewhere along the way.

 So the biggest accessibility improvement here is the addition of the
 visible play/pause button.

 See: https://wordpress.slack.com/archives/accessibility/p1479147742000608

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