Skip to content

Conversation

@spsDrop
Copy link
Contributor

@spsDrop spsDrop commented Nov 21, 2025

This PR adds support for reading the up_axis, chirality, safe_orbit_camera_elevation_min_max_radians, and safe_orbit_camera_radius_min header values and applies them if the active camera is a ArcRotateCamera.

image

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 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 Nov 21, 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 Nov 21, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Nov 21, 2025

@spsDrop spsDrop marked this pull request as ready for review November 21, 2025 23:18
@spsDrop
Copy link
Contributor Author

spsDrop commented Nov 21, 2025

I can't seem to add a label to this PR now that it's created. But it's a new feature

@deltakosh deltakosh enabled auto-merge (squash) November 26, 2025 15:39
auto-merge was automatically disabled December 1, 2025 22:04

Head branch was pushed to by a user without write access

@bjsplat
Copy link
Collaborator

bjsplat commented Dec 1, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Dec 1, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Dec 1, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Dec 1, 2025

@sebavan
Copy link
Member

sebavan commented Dec 2, 2025

@CedricGuillemet can you review the latest commit ?

@sebavan sebavan merged commit 4fabee8 into BabylonJS:master Dec 2, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants