Fix volume menu keyboard access. #3034
Closed
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
OwenEdwards
force-pushed
the
fix/volume-menu-keyboard-access
branch
from
f3345d4
to
2adeae6
Compare
…yboard operation.
OwenEdwards
force-pushed
the
fix/volume-menu-keyboard-access
branch
from
2adeae6
to
8cfab9e
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 ...
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 #3033, this fixes keyboard operation of the 'Volume Menu' (really a 'Volume popup') without changing the DOM. This replaces #2994.