keyboard trap on caption button (MenuButton)

[gdkraus]

Currently, as you tab through the interface, once you get to the caption button you get trapped in there. This can be very difficult, or even impossible, for users to get out of. I think the problem is src/js/menu.js line 185. The event.preventDefault(); should come inside of the if-then structure, only if space, enter, or escape is pressed. Otherwise the tab key is getting prevented too, which keeps users trapped there.

I would recommend this as a hotfix as it is creating fundamental barriers for certain user groups (keyboard-only users). It is preventing them from interacting with the player and also with any other content on the page where Video.JS is implemented. I can submit a patch for it if you would like, but since it's a hotfix I thought I'd check first.

[heff]

Thanks Greg, a patch would be great, and if you can make it against the stable branch that would be best. We'll get this out ASAP.

[heff]

This has been fixed and released in v4.11.3! See #1760