Skip to content
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

Cursor lock when dragging camera #251

Closed
robertlong opened this issue Apr 23, 2018 · 4 comments
Closed

Cursor lock when dragging camera #251

robertlong opened this issue Apr 23, 2018 · 4 comments
Assignees
Labels

Comments

@robertlong
Copy link
Contributor

When you currently drag to look we do not lock the cursor. This causes your cursor to move outside the window and the user can become confused as to where their cursor went.

@gfodor
Copy link
Contributor

gfodor commented May 2, 2018

Did we decide on the exact behavior we want?

@cvan
Copy link
Contributor

cvan commented May 8, 2018

(let me know if you would like me to file a separate issue as a feature request.) has the Pointer Lock API been considered? it is popular for FPS games on the Web.

@johnshaughnessy
Copy link
Contributor

johnshaughnessy commented May 22, 2018

#391 Doesn't solve this by locking the cursor on drag, but it at least allows people to toggle cursor lock on right click. If we were to actually lock the cursor on each drag, then the browser would show a notification each time informing the user how to regain control. If we were to keep the app in cursor locked mode, we wouldn't be able to click on the UI.

@johnshaughnessy
Copy link
Contributor

Marking this as closed because we've explored the solutions and chosen one that works OK. Cursor lock when dragging camera is better than the solution we chose, except that a notification about cursor lock would appear every single time you click, and thus we only lock on right click.

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

No branches or pull requests

4 participants