Line2NodeMaterial: `lineWidth` has no effect.

[Jenychen1996]

Description

In Line2NodeMaterial, there is a lineWidth property, but it seems to have no effect. The line width is still modified based on the linewidth property used in the Material.

Reproduction steps

1.use lineWidth property
2.use linewidth property

Code

// code goes here
new THREE.Line2NodeMaterial( {

					color: 0xffffff,
					linewidth: 5, // in world units with size attenuation, pixels otherwise
                                        // lineWidth: 10, // ineffective
					vertexColors: true,
					dashed: false,
					alphaToCoverage: true,
					transparent: true

				} );

Live example

None

Screenshots

No response

Version

r171 / r178

Device

No response

Browser

No response

OS

No response

[Mugen87]

The TSL object materialLineWidth refers to material.linewidth. Line2NodeMaterial.lineWidth is not used in the corresponding TSL shader code. The examples also do not refer to Line2NodeMaterial.lineWidth. So I think the property can indeed be removed.

Are you interested in making a PR with the change?

[Jenychen1996]

The TSL object materialLineWidth refers to material.linewidth. Line2NodeMaterial.lineWidth is not used in the corresponding TSL shader code. The examples also do not refer to Line2NodeMaterial.lineWidth. So I think the property can indeed be removed.

Are you interested in making a PR with the change?

https://github.com/mrdoob/three.js/pull/31401

I have submitted a PR to rename lineWidth to linewidth and change the default value of the comment to 1. Please review.