Browse Source

Merge pull request #11857 from donmccurdy/feat-gltf-default-shading

[gltf] Use flat shading if normals are not included.
Mr.doob 8 years ago
parent
commit
d770cef0ba
1 changed files with 15 additions and 0 deletions
  1. 15 0
      examples/js/loaders/GLTF2Loader.js

+ 15 - 0
examples/js/loaders/GLTF2Loader.js

@@ -2204,6 +2204,21 @@ THREE.GLTF2Loader = ( function () {
 
 
 						}
 						}
 
 
+						if ( geometry.attributes.normal === undefined ) {
+
+							if ( material.flatShading !== undefined ) {
+
+								material.flatShading = true;
+
+							} else {
+
+								// TODO: Remove this backwards-compatibility fix after r87 release.
+								material.shading = THREE.FlatShading;
+
+							}
+
+						}
+
 						meshNode = new THREE.Mesh( geometry, material );
 						meshNode = new THREE.Mesh( geometry, material );
 						meshNode.castShadow = true;
 						meshNode.castShadow = true;