mdi-react icon indicating copy to clipboard operation
mdi-react copied to clipboard
verified publisher icon levrik

mdi-react-native

Open d4rky-pl opened this issue 5 years ago • 6 comments

Hey! I love your library!

It would be amazing if other than mdi-react and mdi-preact there was a third option, mdi-react-native. The differences would be tiny:

  • do not apply default className
  • replace svg and path with import Svg, { Path } from 'react-native-svg'
  • I'm not 100% sure right now but I think fill needs to be applied on Path instead of Svg

I'd love to send a PR but I'm not a TypeScript person so I gave up after trying to fix the typings (sorry!)

Would you be interested in adding this and releasing it as your own library or would you rather not maintain any extra baggage and I should just fork? :)

d4rky-pl avatar Jan 13 '20 14:01 d4rky-pl

Hey there. Just leave the TypeScript typings out and open a PR. I'll add them to your PR.

levrik avatar Jan 13 '20 15:01 levrik

Any updates on this from your side?

levrik avatar Feb 05 '20 08:02 levrik

Sorry! We ultimately went with a different approach and I totally forgot about this 🤦‍♂ I'll take a look later today and will send a PR 👍

d4rky-pl avatar Feb 05 '20 11:02 d4rky-pl

Just to let you know, I remember about this issue but couldn't find time because of traveling 😅 Please keep it open!

d4rky-pl avatar Feb 12 '20 17:02 d4rky-pl

Any updates on this?

jjarrett21 avatar May 25 '20 22:05 jjarrett21

To be honest due to the global pandemic and everything related to it, I completely forgot about this issue and the code I already had prepared.

I sent a PR #79, it still requires TS fixes though.

d4rky-pl avatar May 27 '20 10:05 d4rky-pl