-
Notifications
You must be signed in to change notification settings - Fork 81
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
docs: Add Spatial Navigation guide #190
Conversation
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.
Some minor inline suggestions. The player options should be included in options.mdx too.
Co-authored-by: mister-ben <[email protected]>
Co-authored-by: mister-ben <[email protected]>
Co-authored-by: mister-ben <[email protected]>
Co-authored-by: mister-ben <[email protected]>
* `stop()` - Stops the Spatial Navigation by removing the keydown event listener from the player and updating the internal state. | ||
* `pause()` - Temporarily disables the Spatial Navigation functionality without removing the event listeners. | ||
* `resume()` - Re-enables the Spatial Navigation functionality after it has been paused. | ||
* `updateFocusableComponents()` - Updates and retrieves the list of focusable components within the player. |
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.
This may need to be updated according to videojs/video.js#8717
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.
@amtins PR videojs/video.js#8717 was removed
Would be great to get this merged, I was trying to implement this, and had to really dig to find the docs hidden in this PR. |
No description provided.