-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
[GLTF Serializer] Allow GLTF Exporter to export WebP images #13062
[GLTF Serializer] Allow GLTF Exporter to export WebP images #13062
Conversation
Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s). |
db26b8a
to
1b26914
Compare
Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s). |
bfda702
to
6b57bf0
Compare
Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s). |
Snapshot stored with reference name: Test environment: To test a playground add it to the URL, for example: https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13062/merge/index.html#WGZLGJ#4600 Links to test babylon tools with this snapshot: https://playground.babylonjs.com/?snapshot=refs/pull/13062/merge To test the snapshot in the playground with a playground ID add it after the snapshot query string: https://playground.babylonjs.com/?snapshot=refs/pull/13062/merge#BCU1XR#0 |
just a tiny lint issue you can fix with |
6b57bf0
to
02f3a07
Compare
Former-commit-id: cd1c62df09be4c24be4cf853c6f64b6a018e8a5f
Adds JPEG and WEBP as valid optional mimetype options when serializing to a GLB.
Usage:
Add this to the dev host scene's button:
Output as per
gltf-transform inspect
PNG
JPEG*
WEBP
My first PR for this project, so let me know if I'm missing anything.