react-mapkit icon indicating copy to clipboard operation
react-mapkit copied to clipboard

Some updates that makes this library to work with the newest React versions + some bug fixes

Open MohaElder opened this issue 3 years ago • 2 comments

  1. It seems that in newer versions of React(e.g: 18.x.x), you need to use PropsWithChildren to wrap your function or else there will be Property 'children' does not exist on type 'IntrinsicAttributes & Props' error.

reference: Answer from StackOverflow

  1. I noticed that there are missing prop type definitions in Marker (title & subtitle) and Map (tintColor). I have added them.

MohaElder avatar Aug 23 '22 23:08 MohaElder

Hi! I saw your comment on this closed PR that explained why you don't want to upgrade library's React related specifications to ensure its compatibility with older React versions. But I was wondering if it is possible to have a seperate version of this library that supports newer react versions since the current library crashes on newer react versions. If possible, I could help with that! Thanks for the time reading this~

MohaElder avatar Aug 23 '22 23:08 MohaElder

I think this library could probably be updates to both not crash and work on older react versions :D

I plan on circling back to this library soon as I need it for an upcoming project. In the meantime I'm happy to accept PRs.

chrisdrackett avatar Aug 24 '22 00:08 chrisdrackett