Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Clean up GLTFLoader Doc #13413

Merged
merged 1 commit into from
Feb 23, 2018
Merged

Clean up GLTFLoader Doc #13413

merged 1 commit into from
Feb 23, 2018

Conversation

takahirox
Copy link
Collaborator

The image formats glTF supports are jpg and png.

https://github.com/KhronosGroup/glTF/tree/master/specification/2.0#imageuri

The image format must be jpg or png.

I think we can remove '...' from the sentence "External files store textures (.jpg, .png, ...)" in GLTFLoader doc.

@mrdoob mrdoob added this to the r91 milestone Feb 23, 2018
@mrdoob mrdoob merged commit 2760318 into mrdoob:dev Feb 23, 2018
@mrdoob
Copy link
Owner

mrdoob commented Feb 23, 2018

Thanks!

@takahirox takahirox deleted the UpdateGLTFLoaderDoc2 branch February 24, 2018 13:45
@takahirox takahirox mentioned this pull request Feb 27, 2018
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@donmccurdy donmccurdy donmccurdy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
r91
Development

Successfully merging this pull request may close these issues.
3 participants
@takahirox @mrdoob @donmccurdy