Skip to content

default fill color for SVGs

Simon Knox requested to merge svg-color into master

What does this MR do?

Most of the time we want icons to match text color, so do that by default.

set default SVG fill to currentColor, which will inherit the current color value. In any places where we want custom fill values, they can easily use new CSS (or alternate classes).

Are there points in the code the reviewer needs to double check?

I haven't tested all the places I have removed styles, for most of them was checking if there was color on the parent.

Why was this MR needed?

we need to fix our CSS

Screenshots (if relevant)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Merge request reports