React: fill is not a style property, it's a svg attribute.

[macno]

Added a default to currentColor but it's possibile to overwrite it via 'fill' prop

Updates react test

It should fix also #240

[vercel]

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/primer/octicons/ldeaqst4x
✅ Preview: https://octicons-git-fork-macno-master.primer.now.sh

[colebemis]

Thanks for the PR, @macno! 🎉 This would be a great addition to the library. Could you update the docs to reflect this change? This is the markdown file for the react docs: https://github.com/primer/octicons/blob/master/docs/content/packages/react.mdx

Let me know if you have any questions :)

[macno]

@colebemis no, thank you for the project :)

[macno]

@colebemis Hi, I did all the changes suggested, do they look good to you?

[colebemis]

@macno Sorry for the delay. We're working on releasing a new major version of Octicons that includes all the new icons you see on github.com. I'll probably bundle this change with that release.

[colebemis]

Closing in favor of #551