Skip to content

Check next props value against code mirror value#122

Open
durran wants to merge 1 commit intoJedWatson:masterfrom
durran:fix-next-props
Open

Check next props value against code mirror value#122
durran wants to merge 1 commit intoJedWatson:masterfrom
durran:fix-next-props

Conversation

@durran
Copy link
Copy Markdown

@durran durran commented Jun 28, 2017

We we having the same issues as described in #121 and #106. In our case, simply checking the next prop's value against the current code mirror value resolved the issue.

@OsamaShabrez
Copy link
Copy Markdown

Waiting for this PR to be merged. This solves our problem as well.

@marcofugaro
Copy link
Copy Markdown

please @JedWatson

@marcofugaro
Copy link
Copy Markdown

@durran I think also the file lib/Codemirror.js needs to be updated, that is the one being used when you require the library. I don't understand the workflow but you probably need to launch just npm run build

@inoas
Copy link
Copy Markdown

inoas commented Jul 3, 2017

We are waiting for this (or #107) to merge and considering moving to https://github.com/securingsincity/react-ace if there is nothing happening anytime soon cause we do not want to create a real fork.

@durran
Copy link
Copy Markdown
Author

durran commented Jul 4, 2017

@marcofugaro I wasn't sure of the workflow as well so figure the build would be handled by @JedWatson before the publish.

@besh
Copy link
Copy Markdown

besh commented Jul 7, 2017

@JedWatson
Just pinging because it looks like others were pinging with lowercase jedwatson which didn't appear to be performing the @mention.

Many have been needing the changes either in this PR or in #107. Are you open to having others help maintain this repo? We love everything you've done with it so far and I personally wouldn't want to have to rely on a random branch or switch to a different lib.

@christopherdro
Copy link
Copy Markdown

@JedWatson

@christopherdro
Copy link
Copy Markdown

Unfortunately this did not solve #106 for me.

@inoas
Copy link
Copy Markdown

inoas commented Jul 13, 2017

Are you sure?
Does #107 fix it for your @christopherdro

@christopherdro
Copy link
Copy Markdown

Yes #107 works

@inoas
Copy link
Copy Markdown

inoas commented Oct 2, 2017

Edit: Please try https://github.com/scniro/react-codemirror2 instead.
That fork is maintained, available via npm and gathering stars.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants