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
fix: navigator.keyboard.lock()
fullscreen exit handling
#40365
Conversation
dd045dd
to
391946a
Compare
391946a
to
ceae6fd
Compare
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.
👍
Release Notes Persisted
|
I have automatically backported this PR to "26-x-y", please check out #40387 |
I have automatically backported this PR to "27-x-y", please check out #40388 |
I have automatically backported this PR to "28-x-y", please check out #40389 |
…0365) fix: navigator.keyboard.lock() fullscreen exit handling
Description of Change
Closes #40342.
Fixes an issue where
navigator.keyboard.lock()
did not work per latest expected behavior - pressing Esc exited fullscreen immediately despite a keyboard lock being present to capture the key. We added this behavior historically when we didn't use theExclusiveAccessManager
- it now handles fullscreen capture and exit when necessary.This can be tested with https://fullscreen-keyboard-lock.glitch.me - now, pressing and holding escape correctly triggers
RequiresPressAndHoldEscToExit()
and associated logic.Checklist
npm test
passesRelease Notes
Notes: Fixes an issue where
navigator.keyboard.lock()
did not work per latest expected behavior