svg fill-opacity [Bug] about two.js HOT 12 CLOSED

I found following Link: https://stackoverflow.com/questions/6042550/svg-fill-color-transparency-alpha
Is there a way to specify fill-opacity directly in code via a member function for two.Path?
Part of the link:

from two.js.

Have you tried curve.fill = "none";?

from two.js.

from two.js.

Unfortunately different browsers have different implementations of "none", so it's not a great option for Two.js.

Have you tried curve.fill = "rgba(0, 0, 0, 0);"?

from two.js.

from two.js.

Got it. That makes sense. If this is a static image (not animating) and you are using the SVG Renderer then you can always access the underlying SVG element in code like this. But, be sure to call two.update(); or have the renderer render at least once beforehand so that the elements are created:

// Given your curve code
two.add(curve);
two.update();

const elem = curve.renderer.domElement;
elem.setAttribute('fill-opacity', 0);

from two.js.

It is a static image. The image is created after a finite element calculation, and two.update() is always called. Your suggestion with curve.fill="none" serves me fine. I don't have to care about old browsers, because my students have current hardware, usually newer than mine.
But two.makeArrow still has black background, see my first comment, (picture in the middle). Other users of your lib might have that problem too and .nofill() will not work as expected . Or am I wrong?

from two.js.

Got it. I'll have to re-run tests to see if all the browsers support curve.fill = "none". At the time of writing, which was a long time ago, both Firefox and Safari did not support "none". This is why noFill yields transparent. They may now and if so then I'll replace it.

Easy fix.

from two.js.

No, because fill-opacity is not available in all renderers. That is why it's recommended to use:

curve.renderer.domElement.setAttribute('fill-opacity', 0);

from two.js.

ok

from two.js.

v8.12 will have noFill and noStroke execute CSS commands "none" instead of "transparent"

from two.js.

from two.js.