d2l-button-icon doesn't pass the color property down to d2l-icon about core HOT 5 CLOSED

Interesting! So I think we were being a bit intentional about only having the icon be ferrite, since the design kind of specs it out that way. What's your use case for coloured icons?

from core.

Just glitter. I'm switching between tier1:role-switch and tier1:close-circle to cancel it:

It's a bit easier to tell the two states apart with some color (but not required). If we don't want that because of design I'm fine with that!

from core.

Here it is, hacked with color, in case this screams "obviously wrong" :)

from core.

Yeah I'm pretty sure design intentionally wanted the grey icons to avoid having particular ones popping. There's also the accessibility angle, where enough contrast needs to exist between the icon colour and the hover background colour, which we can guarantee by controlling it.

I'd also worry that if we supported this, then suddenly colours set on parent elements would start bleeding into our icons everywhere, without us actually wanting it to happen. i.e.

<div style="color:pink">
    Pink text
    <d2l-icon></d2l-icon>
</div>

from core.

Ok, cool! Thanks for the explanation.

Ah that cascading thing is definitely not pretty.

from core.