Nodes: Document more modules. #30163
Conversation
| this.isLineBasicNodeMaterial = true; | ||
|
|
||
| this.lights = false; |
There was a problem hiding this comment.
I have removed these statements since the default of lights is already false.
| this.lights = false; | ||
| this.transparent = true; | ||
|
|
||
| this.sizeNode = null; |
There was a problem hiding this comment.
I have removed this property since I think it's from a time where InstancedPointsNodeMaterial was not available yet and we assumed we have to implement size support in PointsNodeMaterial.
There was a problem hiding this comment.
I was doing a review on InstancedPointsNodeMaterial, with Nodes we wouldn't need variations for all this kind of case, and once we have Raycaster and BoundBox through GPU we will have an optimized support for procerural things. I'll try to show something in the next few weeks.
📦 Bundle sizeFull ESM build, minified and gzipped.
🌳 Bundle size after tree-shakingMinimal build including a renderer, camera, empty scene, and dependencies.
|
| this.isPointsNodeMaterial = true; | ||
|
|
||
| this.lights = false; | ||
| this.transparent = true; |
There was a problem hiding this comment.
I don't remember, but removing it seems correct in the current context for me.
Related issue: #29984
Description
Documenting more node material classes.