Browse Source

Merge pull request #17992 from WestLangley/dev_svgrenderer

SVGRenderer: normalize after applying normalMatrix
Mr.doob 5 years ago
parent
commit
447e8e7297
2 changed files with 2 additions and 2 deletions
  1. 1 1
      examples/js/renderers/SVGRenderer.js
  2. 1 1
      examples/jsm/renderers/SVGRenderer.js

+ 1 - 1
examples/js/renderers/SVGRenderer.js

@@ -439,7 +439,7 @@ THREE.SVGRenderer = function () {
 
 
 		} else if ( material.isMeshNormalMaterial ) {
 		} else if ( material.isMeshNormalMaterial ) {
 
 
-			_normal.copy( element.normalModel ).applyMatrix3( _normalViewMatrix );
+			_normal.copy( element.normalModel ).applyMatrix3( _normalViewMatrix ).normalize();
 
 
 			_color.setRGB( _normal.x, _normal.y, _normal.z ).multiplyScalar( 0.5 ).addScalar( 0.5 );
 			_color.setRGB( _normal.x, _normal.y, _normal.z ).multiplyScalar( 0.5 ).addScalar( 0.5 );
 
 

+ 1 - 1
examples/jsm/renderers/SVGRenderer.js

@@ -455,7 +455,7 @@ var SVGRenderer = function () {
 
 
 		} else if ( material.isMeshNormalMaterial ) {
 		} else if ( material.isMeshNormalMaterial ) {
 
 
-			_normal.copy( element.normalModel ).applyMatrix3( _normalViewMatrix );
+			_normal.copy( element.normalModel ).applyMatrix3( _normalViewMatrix ).normalize();
 
 
 			_color.setRGB( _normal.x, _normal.y, _normal.z ).multiplyScalar( 0.5 ).addScalar( 0.5 );
 			_color.setRGB( _normal.x, _normal.y, _normal.z ).multiplyScalar( 0.5 ).addScalar( 0.5 );