Fix: Resolve incorrect rendering of some colored icons

[kimtaejin3]

I solved the issue I mentioned in #827

@kamijin-fanta @gorangajic Could you please review this?

• Before I fix it

• After I fix it

[soseonghyeon]

@kamijin-fanta When is this going to be merged? I want to use GrChrome in Grommet Icons but now I can't use this

[kamijin-fanta]

@kimtaejin3 Thanks for your contribution.

If multiColor is enabled, changing the color of icons with color props will not work. grommet-Icons include some colored icons, but most are single-color icons.

Twotone in Ant Design Icons may be good with mutiColor enabled.

[kimtaejin3]

@kamijin-fanta Thanks for replying! Then why don't you enable multicolor only for Twotone in Ant Design Icons? I think we need to think more about Grommet Icons

[kamijin-fanta]

I have looked into what happened with grommet-Icons.

✅OK / ⚠OK, but color is overrided / 🚫NG

• ✅ use fill="none" stroke="#000" https://github.com/grommet/grommet-icons/blob/v4.11.0/public/img/anchor.svg
• ⚠ toplevel not specific / path fill="#A4C639" https://github.com/grommet/grommet-icons/blob/v4.11.0/public/img/android.svg
• 🚫 toplevel fill="none" / path fill="#000" https://github.com/grommet/grommet-icons/blob/v4.11.0/public/img/clock.svg
• 🚫 toplevel fill="none" / path fill="black" https://github.com/grommet/grommet-icons/blob/v4.11.0/public/img/caret-right-fill.svg

The library adds stroke="currentColor" fill="currentColor" when drawing SVG. However, this seems to be overridden if fill="none" is specified in the SVG tag. I think the top-level fill can be removed. However, it needs to be investigated if it does not break compatibility.

Twotone in Ant Design Icons should be included in the library update.

[kamijin-fanta]

This change will be released as v4.12.0.

ccfbc00