Skip to content

Conversation

ryantrem
Copy link
Member

I noticed that Color and Vector/Quaternion related properties were not displaying correctly in Inspector v2 specifically in Playground. After a little investigation, I realized that imports from core, loaders, etc. were being correctly externalized to BABYLON.* for any code directly in inspector-v2, but not for sharedUiComponents, which means specifically for our shared controls, the Playground app bundle itself was basically getting extra copies of Color*/Vector*/Quaternion classes and the instanceof checks were failing (because they were two different classes with the same name/content). This change updates Playground's webpack config to externalize any core, loaders, etc. imports to use BABYLON.*, which fixes the problem.

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 10, 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.

@ryantrem ryantrem enabled auto-merge (squash) October 10, 2025 22:41
@bjsplat
Copy link
Collaborator

bjsplat commented Oct 10, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 10, 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/17274/merge/

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 10, 2025

Building or testing the playground has failed.

If the tests failed, results can be found here:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/17274/merge/testResults/

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 10, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 10, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 10, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 10, 2025

Building or testing the playground has failed.

If the tests failed, results can be found here:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/PLAYGROUND/refs/pull/17274/merge/testResults/

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 10, 2025

@bjsplat
Copy link
Collaborator

bjsplat commented Oct 10, 2025

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

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

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

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

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

@ryantrem ryantrem merged commit 849914a into BabylonJS:master Oct 12, 2025
22 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.

3 participants