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

Published 20 hours ago • verified publisher icon JedWatson

Deprecate react-codemirror

Open scniro opened this issue 8 years ago • 14 comments

There are currently 36 open issues, 7 open pull requests . A lot of users are seemingly holding out for all of this to be fixed.

I created https://github.com/scniro/react-codemirror2 and plan to actively maintain it. Please consider deprecating this package if you are unable to maintain it moving forward.

scniro avatar Aug 10 '17 15:08 scniro

I went and closed my PR since we've switched to react-codemirror2. Highly recommend it if anyone is considering using or currently using this react-codemirror module. Thanks @scniro!

rickbergfalk avatar Sep 23 '17 16:09 rickbergfalk

@rickbergfalk , where can I find the react-codemirror2?

ashking avatar Sep 24 '17 02:09 ashking

Its on npm as react-codemirror2. https://github.com/scniro/react-codemirror2

rickbergfalk avatar Sep 24 '17 03:09 rickbergfalk

Great, thanks!

ashking avatar Sep 24 '17 03:09 ashking

@rickbergfalk You are welcome and thank you as well for the recommendation! Please feel free to open up an issue over there should you experience any troubles or have any suggestions to help improve.

scniro avatar Sep 24 '17 13:09 scniro

@scniro Appreciate you pointing this out!

Were upgrading to React16 and came across this issue.

Looks like your peer dependencies include React 16.x, so we'll move over to this as long as the core api is the same (looks like it is).

Thanks!

jamesopti avatar Oct 03 '17 22:10 jamesopti

@JedWatson Whats you opinion? Let this nice package die or get some helping hands to polish it. I know that maintaining FOSS is not the easiest job :)

markusguenther avatar Apr 23 '18 23:04 markusguenther

@markusguenther It's been ~9 months since this was opened. What are you expectations for a response? Why not help make react-codemirror2 better instead of kicking dirt here?

scniro avatar Apr 24 '18 22:04 scniro

@scniro Is there a migration guide from this lib to yours? API seems to have changed quite a bit.

Kudos for taking the initiative BTW :)

itaysabato avatar Jun 28 '18 23:06 itaysabato

@itaysabato We did the move some month ago and it was pretty simple.

https://github.com/neos/neos-ui/commit/eeb9a9eb6ad260867348f60f908ea9abe0e8c6a4

markusguenther avatar Jul 02 '18 06:07 markusguenther

Thanks for the example @markusguenther :)

I switched to the Controlled component which has different behaviors, e.g. I had to use onBeforeChange because onChange was never called. Not sure why this is or why there is an UnControlled component at all...

Anyway, I managed to migrate by myself but still think a migration guide would be very helpful to others.

itaysabato avatar Jul 02 '18 06:07 itaysabato

@itaysabato a migration guide for a small component? Yikes. Doesn't the readme cover it? We're not talking frameworks here...

scniro avatar Jul 02 '18 11:07 scniro

@scniro actually no, the readme doesn't cover it. The "migration guide" can be one or two paragraphs in the readme, it doesn't have to be a big deal.

AFAIK react-codemirror2 started as a fork for this npm module so it would make sense IMO to have a section explaining how to migrate from it. Maybe it's just me.

itaysabato avatar Jul 02 '18 16:07 itaysabato

@itaysabato It's not a fork, it's a rewrite. There is no overlap or shared history at all with this repo. How does the readme not cover it? As a user who's never used either repo, I'm willing to bet it's all there. I put a bit of effort into making it pretty comprehensive. Anyways, I don't care to waste time explaining something, that IMO, explains itself (the whole point of a readme? 🤔)

scniro avatar Jul 02 '18 19:07 scniro