Skip to content

fix(RWInteractor): async requestPointerLock - #3205

Merged
finetjul merged 1 commit into
masterfrom
requestpointerlock-async
May 11, 2026
Merged

fix(RWInteractor): async requestPointerLock#3205
finetjul merged 1 commit into
masterfrom
requestpointerlock-async

Conversation

@floryst

@floryst floryst commented Jan 16, 2025

Copy link
Copy Markdown
Contributor

Context

Chrome 131+ now requests permissions for pointer lock. We need to ensure that pointer lock is achieved before continuing with interactions.

Results

  • support async requestPointerLock

Changes

  • all calls to requestPointerLock have been updated
  • examine whether methods calling requestPointerLock should be made async

PR and Code Checklist

  • semantic-release commit messages
  • Run npm run reformat to have correctly formatted code

Testing

  • This change adds or fixes unit tests
  • Tested environment:
    • vtk.js:
    • OS:
    • Browser:

@danyim

danyim commented Mar 13, 2025

Copy link
Copy Markdown

I just verified this change in https://github.com/SironaMedical/services-frontend/pull/6140 - looks good to me! Could we get this merged?

@floryst
floryst marked this pull request as ready for review March 14, 2025 17:33
@floryst
floryst force-pushed the requestpointerlock-async branch from 7240b49 to 73ab82f Compare March 14, 2025 17:34
@floryst
floryst enabled auto-merge May 30, 2025 19:56
@floryst
floryst added this pull request to the merge queue May 30, 2025
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to a conflict with the base branch May 30, 2025
@sankhesh

Copy link
Copy Markdown
Collaborator

@floryst Could you please rebase this change? It seems its ready for merging otherwise.

@danyim

danyim commented May 5, 2026

Copy link
Copy Markdown

Bump - can we get this PR into a mergeable state?

@jadh4v
jadh4v force-pushed the requestpointerlock-async branch from f5ab529 to d0148bf Compare May 8, 2026 18:01
@finetjul
finetjul added this pull request to the merge queue May 11, 2026
Merged via the queue into master with commit e1fb07b May 11, 2026
2 checks passed
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 35.14.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

@github-actions github-actions Bot added the released Automated label label May 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Automated label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants