Skip to content

Commit

Permalink
fix(FEC-12719): [WEB][UI]- Player v7 | playlist v3.0.1-canary.1-51e0b…
Browse files Browse the repository at this point in the history
…54 | Accessibility | After entry reaches the end, when tab navigate to the square next video in the middle of the player (autoContinue: false) "Space" doesn't work. (#701)

### Description of the Changes
add the ability to move the next video by space key.

solves FEC-12719

### CheckLists

- [ ] changes have been done against master branch, and PR does not
conflict
- [ ] new unit / functional tests have been added (whenever applicable)
- [ ] test are passing in local environment
- [ ] Travis tests are passing (or test results are not worse than on
master branch :))
- [ ] Docs have been updated
  • Loading branch information
Tzipi-kaltura committed Dec 12, 2022
1 parent 5ce86c1 commit fdb92df
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/components/playlist-next-screen/playlist-next-screen.js
Original file line number Diff line number Diff line change
Expand Up @@ -78,7 +78,8 @@ class PlaylistNextScreen extends Component {
* @memberof PlaylistNextScreen
*/
onKeyDown = (e: KeyboardEvent): void => {
if (e.keyCode === KeyMap.ENTER) {
if (e.keyCode === KeyMap.ENTER || e.keyCode === KeyMap.SPACE) {
e.preventDefault();
this.onPosterClick();
}
};
Expand Down

0 comments on commit fdb92df

Please sign in to comment.