Skip to content

Conversation

PButcher
Copy link
Contributor

@PButcher PButcher commented Nov 26, 2017

Suppresses the following warning when using the component with A-Frame v0.7.0:

THREE.MeshLambertMaterial: .shading has been removed. Use the boolean .flatShading instead.

.shading has been removed and is now .flatShading, see: https://threejs.org/docs/index.html#api/materials/Material

Suppresses the following warning: `THREE.MeshLambertMaterial: .shading has been removed. Use the boolean .flatShading instead.`

`.shading` has been removed and is now `.flatShading`, see: https://threejs.org/docs/index.html#api/materials/Material
@feiss
Copy link
Contributor

feiss commented Nov 27, 2017

thank you for your help!
flatShading is available from threejs r87, so environment component would be broken for users using aframe < v0.7. Could you make it so it is retro compatible with aframe < v0.7 too? thanks!

Use .shading for ground MeshLambertMaterial in A-Frame < 0.7.0 and .flatShading for >= 0.7.0
@PButcher
Copy link
Contributor Author

Good point, done.

@feiss feiss merged commit ae3d8cd into supermedium:master Mar 22, 2018
@PButcher PButcher deleted the patch-1 branch March 22, 2018 11:11
PiTiLeZarD pushed a commit to PiTiLeZarD/aframe-environment-component-v2 that referenced this pull request May 11, 2022
PiTiLeZarD pushed a commit to PiTiLeZarD/aframe-environment-component-v2 that referenced this pull request May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants