Fix menu keyboard access and ARIA labeling for screen readers (Fixes #2155 and #2769), supercedes #2897 #3033
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| @@ -25,6 +25,9 @@ class OffTextTrackMenuItem extends TextTrackMenuItem { | |||
| 'mode': 'disabled' | |||
| }; | |||
|
|
|||
| // MenuItem is selectable | |||
| options['selectable'] = true; | |||
There was a problem hiding this comment.
options.selectable should work fine here.
d20d883 to
060b522
Compare
Handle correct keyboard operation of menu, including single tab stop for menu, arrow keys within the menu, focus movement within the menu, and ARIA roles for parts of menus.
060b522 to
b9c6460
Compare
jgubman
added a commit
to jgubman/video.js
that referenced
this pull request
Jan 27, 2016
* 'master' of https://github.com/videojs/video.js: (38 commits) v5.6.0 @gkatsev updated to latest videojs-ie8 shim. closes videojs#3042 @hubdotcom changed URLs in README to be protocol-relative. closes videojs#3040 @CoWinkKeyDinkInc fixed table in Tracks guide. Replaced some single quotes with double quotes. closes videojs#2946 @aril-spetalen added language support for Norwegian (nb and nn). closes videojs#3021 @vitor-faiante updated the guides. closes videojs#2781 @gkatsev checked muted status when updating volume bar level. closes videojs#3037 @OwenEdwards fixed double-localization of mute toggle control text. closes videojs#3017 @mister-ben made $primary-foreground-color a !default sass var. closes videojs#3003 @OwenEdwards Fixed volume menu keyboard access. closes videojs#3034 @OwenEdwards Fixed menu keyboard access and ARIA labeling for screen readers. closes videojs#3033 @OwenEdwards added ClickableComponent. Fixed keyboard operation of buttons. closes videojs#3032 v5.5.3 @rcrooks fixed a couple of docs link and a jsdoc comment. closes videojs#2987 @mister-ben updated CDN urls in setup guide. closes videojs#2984 @gkasev updated vjs to correctly return already created player when given an element. closes videojs#3006 v5.5.2 make sure that styleEl_ is in DOM before removing on dispose. closes videojs#3004 v5.5.1 @gkatsev fixed sass if else for icons. closes videojs#2988 ...
chemoish
pushed a commit
to chemoish/videojs-chapter-thumbnails
that referenced
this pull request
Sep 1, 2016
NOTE: videojs/video.js#3033 introduced a breaking change.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Building on #3032, this fixes keyboard operation of Menus without changing the DOM.