GLTFLoader: Properly set defines for GLTFMeshStandardSGMaterial #20242
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For non-built-in materials, the app must set the defines, because the renderer does not do it.
(Sorry, there is considerable clean-up here. I typically keep that separate... The significant changes in this PR are to
specularMap.set()
andglossinessMap.set()
.)/ping @donmccurdy