Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Keyboard control of menus doesn't work for the Chapters menu #3164

Closed
OwenEdwards opened this issue Mar 9, 2016 · 4 comments
Closed

Keyboard control of menus doesn't work for the Chapters menu #3164

OwenEdwards opened this issue Mar 9, 2016 · 4 comments
Labels
a11y This item might affect the accessibility of the player

Comments

@OwenEdwards
Copy link
Member

Description

Keyboard acces to menus was fixed in #3033, but #3163 exposes that keyboard access doesn't work for a menu with a title (such as the Chapters menu).

Steps to reproduce

Expected behavior

Use Tab to move focus to the Captions menu button
Use Enter to open the menu
Observe that the arrow keys move focus up and down the menu

Observed behavior

Then use Tab to move focus to the Chapters Menu button
Use Enter to open the menu
Arrow keys do not work

Error output

If there are any errors at all, please include them here.

TypeError: children[item].el_ is undefined

Additional Information

Please include any additional information necessary here. Including the following:

versions

videojs

what version of videojs does this occur with?

Any versions after #3163 is merged.

browsers

what browser are affected?

Checked on Firefox, v44.

OSes

what platforms (operating systems and devices) are affected?

Checked on OS X.

plugins

are any videojs plugins being used on the page? If so, please list them below.

No.

@OwenEdwards OwenEdwards added the a11y This item might affect the accessibility of the player label Mar 9, 2016
@gkatsev
Copy link
Member

gkatsev commented Mar 9, 2016

One way to work around it is to insert an object with the el_ set to the element instead of just inserting the element itself.

@gkatsev
Copy link
Member

gkatsev commented Mar 9, 2016

Or maybe we just want to make a menu-title component?

@gkatsev
Copy link
Member

gkatsev commented Mar 9, 2016

I guess we don't even care to make the title focusable? Anyway, I have a PR to potentially fix it.

@OwenEdwards
Copy link
Member Author

Yeah, the title element shouldn't be focusable, because it has no action associated with it. I thought about the idea of a menu-title component, but I'm really not convinced that there's enough functionality for that to make sense - ss we discussed, it ought to move out of the LI in the future, but that'll have to wait until v6.x.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 27, 2022
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
a11y This item might affect the accessibility of the player
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants