Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Sandbox: adapt panning sensitivity based on radius (same as viewer) #16490

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

Conversation

AmoebaChant
Copy link
Contributor

The viewer adapts the camera's panning sensitivity based on the radius of the object being viewed, and this change duplicates that logic for the sandbox.

@AmoebaChant AmoebaChant requested a review from ryantrem April 14, 2025 22:07
@bjsplat
Copy link
Collaborator

bjsplat commented Apr 14, 2025

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 14, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 14, 2025

You have made possible changes to the playground.
You can test the snapshot here:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16490/merge/

The snapshot playground with the CDN snapshot (only when available):

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/16490/merge/?snapshot=refs/pull/16490/merge

Note that neither Babylon scenes nor textures are uploaded to the snapshot directory, so some playgrounds won't work correctly.

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 14, 2025

You have changed file(s) that made possible changes to the sandbox.
You can test the sandbox snapshot here:

https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/SANDBOX/refs/pull/16490/merge/

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 14, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Apr 14, 2025

@AmoebaChant AmoebaChant merged commit 2ee29e8 into BabylonJS:master Apr 14, 2025
19 checks passed
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