Skip to content

Document Sandbox camera URL parameters - #1609

Merged
RaananW merged 1 commit into
BabylonJS:masterfrom
VicenteCartas:sandbox-camera-url-parameters
Aug 24, 2026
Merged

Document Sandbox camera URL parameters#1609
RaananW merged 1 commit into
BabylonJS:masterfrom
VicenteCartas:sandbox-camera-url-parameters

Conversation

@VicenteCartas

Copy link
Copy Markdown
Contributor

Summary

  • Document the new cameraMinZ and cameraLowerRadiusLimit Sandbox URL parameters
  • Describe validation ranges, scene-unit semantics, camera restrictions, and local-preset interaction
  • Add a working close-up inspection example

Companion to BabylonJS/Babylon.js#18807.

Validation

  • Full page serialized successfully through the production next-mdx-remote pipeline
  • Documented asset URL returns HTTP 200

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@bjsplat

bjsplat commented Aug 17, 2026

Copy link
Copy Markdown

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Documents newly added Babylon.js Sandbox URL parameters for camera tuning, including their validation expectations and how they interact with the Sandbox’s saved (browser-local) Inspector camera preset behavior.

Changes:

  • Added documentation for cameraMinZ and cameraLowerRadiusLimit, including applicability and numeric constraints.
  • Documented how valid URL camera overrides affect the active browser-local camera preset during initial load and subsequent reloads.
  • Added a close-up inspection example URL demonstrating the new parameters.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@RaananW
RaananW merged commit 5cdc704 into BabylonJS:master Aug 24, 2026
2 checks passed
VicenteCartas added a commit that referenced this pull request Aug 24, 2026
> 🤖 *This PR was created by the create-pr skill.*

## Summary
- Revert Documentation PR #1609 while Babylon.js PR #18807 remains under
discussion.
- Remove the unpublished camera preset and `cameraMinZ` /
`cameraLowerRadiusLimit` documentation from the live Sandbox page.

## Motivation
The documentation was merged before the corresponding Babylon.js feature
was accepted and merged. This keeps the published documentation aligned
with the currently released Sandbox.

The removed documentation should be re-landed once the implementation
discussion in BabylonJS/Babylon.js#18807 is
resolved and the feature is merged.

## Validation
- `npm run validate:content` passed with 805 Markdown files and 0
errors.
- `npm run build` passed and generated all 1,402 pages.
- Documentation tests: 79/80 passed; the sole failure is the
pre-existing Windows path-separator assertion in
`content-graph.test.ts`.
- Revert independently reviewed and verified byte-exact against squash
commit `5cdc704f` from
#1609.

Co-authored-by: Vicente Cartas Espinel <vicente.cartas@microsoft.com>
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.

4 participants