[Icons] explain svg colors and improve example configuration #2671
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I got mislead by the configuration examples setting the
fill
attribute on the<svg>
element. All icons i looked have<path>
elements that referencecurrentColor
for either fill or outline. andcurrentColor
in the context of CSS is thecolor
attribute of the element. thus, to change the color of an icon, we need to set the CSS color. changing thefill
attribute on the top level<svg>
has no effect.I am not 100% sure if this is always true, maybe the section i add needs to be less assertive or explain in which cases it is correct.