New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[JW8-2607] Keep controls visible is activated via keyboard #3259
Conversation
…interaction JW8-2607
Generated by 🚫 dangerJS |
@@ -71,10 +70,20 @@ export default class Controls { | |||
this.activeTimeout = setTimeout(this.userInactiveTimeout, remainingTime); | |||
return; | |||
} | |||
if (this.playerContainer.querySelector('.jw-tab-focus')) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DanFerrer why do we need this ? We already set the activeTimeout to 0 when tabbing inside the player
jwplayer/src/js/view/controls/controls.js
Line 325 in 8bef82f
// keep controls active when navigating inside the player |
This PR will...
Only call
this.userInactive
inside theuserInactiveTimeout
if players controls weren't shown through a keyboard interaction.Why is this Pull Request needed?
We want to keep the controls visible when a user is currently focused on the player for better accessibility.
Are there any points in the code the reviewer needs to double check?
Are there any Pull Requests open in other repos which need to be merged with this?
Addresses Issue(s):
JW8-2607