Commit 6a1c58e
committed
minor #2671 [Icons] explain svg colors and improve example configuration (dbu)
This PR was merged into the 2.x branch.
Discussion
----------
[Icons] explain svg colors and improve example configuration
| Q | A
| ------------- | ---
| Bug fix? | no
| New feature? | no
| Docs? | yes
| Issues | -
| License | MIT
I got mislead by the configuration examples setting the `fill` attribute on the `<svg>` element. All icons i looked have `<path>` elements that reference `currentColor` for either fill or outline. and `currentColor` in the context of CSS is the `color` attribute of the element. thus, to change the color of an icon, we need to set the CSS color. changing the `fill` 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.
Commits
-------
b9347f6 [Icons] explain svg colors and improve example configuration1 file changed
+7
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
109 | 109 | | |
110 | 110 | | |
111 | 111 | | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
| 118 | + | |
112 | 119 | | |
113 | 120 | | |
114 | 121 | | |
| |||
0 commit comments