Sfoglia il codice sorgente

addressing feedback

Emmett Lalish 5 anni fa
parent
commit
91bb0148fe
2 ha cambiato i file con 14 aggiunte e 14 eliminazioni
  1. 7 7
      examples/js/loaders/GLTFLoader.js
  2. 7 7
      examples/jsm/loaders/GLTFLoader.js

+ 7 - 7
examples/js/loaders/GLTFLoader.js

@@ -2111,6 +2111,13 @@ THREE.GLTFLoader = ( function () {
 
 
 		}
 		}
 
 
+		// https://github.com/mrdoob/three.js/issues/11438#issuecomment-507003995
+		if ( material.normalScale && ! useVertexTangents ) {
+
+			material.normalScale.y = - material.normalScale.y;
+
+		}
+
 		mesh.material = material;
 		mesh.material = material;
 
 
 	};
 	};
@@ -2262,13 +2269,6 @@ THREE.GLTFLoader = ( function () {
 
 
 			if ( materialDef.name !== undefined ) material.name = materialDef.name;
 			if ( materialDef.name !== undefined ) material.name = materialDef.name;
 
 
-			// https://github.com/mrdoob/three.js/issues/11438#issuecomment-507003995
-			if ( material.normalScale && ! material.vertexTangents ) {
-
-				material.normalScale.y = - material.normalScale.y;
-
-			}
-
 			// baseColorTexture, emissiveTexture, and specularGlossinessTexture use sRGB encoding.
 			// baseColorTexture, emissiveTexture, and specularGlossinessTexture use sRGB encoding.
 			if ( material.map ) material.map.encoding = THREE.sRGBEncoding;
 			if ( material.map ) material.map.encoding = THREE.sRGBEncoding;
 			if ( material.emissiveMap ) material.emissiveMap.encoding = THREE.sRGBEncoding;
 			if ( material.emissiveMap ) material.emissiveMap.encoding = THREE.sRGBEncoding;

+ 7 - 7
examples/jsm/loaders/GLTFLoader.js

@@ -2178,6 +2178,13 @@ var GLTFLoader = ( function () {
 
 
 		}
 		}
 
 
+		// https://github.com/mrdoob/three.js/issues/11438#issuecomment-507003995
+		if ( material.normalScale && ! useVertexTangents ) {
+
+			material.normalScale.y = - material.normalScale.y;
+
+		}
+
 		mesh.material = material;
 		mesh.material = material;
 
 
 	};
 	};
@@ -2329,13 +2336,6 @@ var GLTFLoader = ( function () {
 
 
 			if ( materialDef.name !== undefined ) material.name = materialDef.name;
 			if ( materialDef.name !== undefined ) material.name = materialDef.name;
 
 
-			// https://github.com/mrdoob/three.js/issues/11438#issuecomment-507003995
-			if ( material.normalScale && ! material.vertexTangents ) {
-
-				material.normalScale.y = - material.normalScale.y;
-
-			}
-
 			// baseColorTexture, emissiveTexture, and specularGlossinessTexture use sRGB encoding.
 			// baseColorTexture, emissiveTexture, and specularGlossinessTexture use sRGB encoding.
 			if ( material.map ) material.map.encoding = sRGBEncoding;
 			if ( material.map ) material.map.encoding = sRGBEncoding;
 			if ( material.emissiveMap ) material.emissiveMap.encoding = sRGBEncoding;
 			if ( material.emissiveMap ) material.emissiveMap.encoding = sRGBEncoding;