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

OrbitControls -> MapControls #405

Merged
merged 2 commits into from
Jan 17, 2025
Merged

OrbitControls -> MapControls #405

merged 2 commits into from
Jan 17, 2025

Conversation

khdlr
Copy link
Member

@khdlr khdlr commented Jan 17, 2025

Fixes #404, switches OrbitControls for MapControls, and sets it up so that a single touch finger pans the map, while two finger touches rotate the map

@khdlr khdlr requested a review from FlorianK13 January 17, 2025 08:55
@FlorianK13
Copy link
Member

Can you also delete <React.StrictMode> in index.jsx? As far as I understand this is some debugging tool that somehow got into our website code 🙃
Don't want to create a new PR just for these two line changes.

@khdlr
Copy link
Member Author

khdlr commented Jan 17, 2025

My understanding is that when doing a release build, strict mode will not do anything, i.e. strict mode only has implications for local dev builds. But I'm happy to get rid of it, because it will fix the simulation running twice in parallel when testing the app locally 😄

@khdlr khdlr merged commit b5968a4 into main Jan 17, 2025
2 checks passed
@khdlr khdlr deleted the feature/3d-touch-rotation branch January 17, 2025 10:02
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.

Touch Navigation
2 participants