Skip to content

Remove references to mozRequestPointerLock and friends. NFC #24603

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Jun 18, 2025

The official requestPointerLock API is supported in all the browsers that we support.

See https://developer.mozilla.org/en-US/docs/Web/API/Element/requestPointerLock

@sbc100 sbc100 force-pushed the mozRequestPointerLock branch 2 times, most recently from 930165c to a26aed9 Compare June 18, 2025 16:19
@sbc100 sbc100 requested review from kripken and RReverser June 18, 2025 16:19
@sbc100 sbc100 force-pushed the mozRequestPointerLock branch from a26aed9 to 1600f8c Compare June 18, 2025 16:23
Copy link
Member

@kripken kripken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm but worth running the relevant interactive tests for this to verify.

@sbc100 sbc100 force-pushed the mozRequestPointerLock branch from 1600f8c to c1a1a65 Compare June 18, 2025 18:40
The official `requestPointerLock` API is supported in all the browsers
that we support.

See https://developer.mozilla.org/en-US/docs/Web/API/Element/requestPointerLock
@sbc100 sbc100 force-pushed the mozRequestPointerLock branch from c1a1a65 to d5cb41d Compare June 18, 2025 22:19
@RReverser
Copy link
Collaborator

RReverser commented Jun 18, 2025

In case you decide to go on a rampage, this regex should help find other potentially outdated fallbacks: https://github.com/search?q=repo%3Aemscripten-core%2Femscripten+%2F%28%3F-i%29%28%5Cb%28moz%7Cwebkit%7Cms%29%5BA-Z%5D%7C%5C-%28moz%7Cwebkit%7Cms%7Co%29-%29%2F+&type=code

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants