AX: usability clean up on media controls accessibility Bug 121990 resolved the accessibility regressions in the media controls, but some of the button roles/descriptions could be improved. For example, there is a "Display Full Screen" check box, that would be better exposed as "Exit Full Screen, button" than "Display Full Screen, checked check box". Similarly, the Mute button should switch to an "Unmute, button" rather than "Mute, checked check box." There are probably a few more items to clean up, too.
<rdar://problem/15384781>
Looks like the captions popup never really worked properly either. (Not a regression, but needs fixing.)
Created attachment 215982 [details] partial patch (may need more)
Now the caption popup is screen reader and keyboard accessible, assuming you can get keyboard focus into the shadow DOM (e.g. with a screen reader or with a mouse click on the Captions popup button). Arrow keys change selection, escape will close the menu, and spacebar/return/enter will select the current menu item, matching OS X HI guidelines for keyboard control of menus.
(In reply to comment #4) > …assuming you can get keyboard focus into the shadow DOM… This is in reference to Bug 117857 - FKA: Default HTML 5 Video controls are not keyboard accessible
Comment on attachment 215982 [details] partial patch (may need more) Attachment 215982 [details] did not pass mac-wk2-ews (mac-wk2): Output: http://webkit-queues.appspot.com/results/19598935 New failing tests: media/controls-styling-strict.html accessibility/media-element.html media/controls-strict.html media/audio-controls-rendering.html media/controls-without-preload.html
Created attachment 215985 [details] Archive of layout-test-results from webkit-ews-15 for mac-mountainlion-wk2 The attached test failures were seen while running run-webkit-tests on the mac-wk2-ews. Bot: webkit-ews-15 Port: mac-mountainlion-wk2 Platform: Mac OS X 10.8.5
Comment on attachment 215982 [details] partial patch (may need more) Attachment 215982 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/20328818 New failing tests: media/controls-strict.html media/controls-styling-strict.html media/audio-controls-rendering.html accessibility/media-element.html media/controls-without-preload.html media/controls-after-reload.html
Created attachment 215989 [details] Archive of layout-test-results from webkit-ews-02 for mac-mountainlion The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: webkit-ews-02 Port: mac-mountainlion Platform: Mac OS X 10.8.5
Created attachment 216004 [details] patch
Comment on attachment 216004 [details] patch r=me. It'd be nice if the change from aria-checked -> aria-label for muteButton, fullscreenButton, and captionButton was mentioned in the ChangeLog.
(In reply to comment #11) > (From update of attachment 216004 [details]) > r=me. It'd be nice if the change from aria-checked -> aria-label for muteButton, fullscreenButton, and captionButton was mentioned in the ChangeLog. That was this comment: "Updated some of the control labels/roles to improve accessibilty." …but I'll update to provide more specifics.
Created attachment 216105 [details] patch with review feedback
Comment on attachment 216105 [details] patch with review feedback Clearing flags on attachment: 216105 Committed r158713: <http://trac.webkit.org/changeset/158713>
All reviewed patches have been landed. Closing bug.