Make debounce specific to the instance

[sslotsky]

Resubmission of #49

Elected to resubmit rather than rebase because

1. Code was extremely stale
2. Easier to undo my changes to the lib folder, which I assume you don't want

Also noticed that #66 proposes the same change as this does, but it also needs rebasing. @JedWatson I'll leave it up to you which one you prefer, but assuming you use this one, let's get it before it goes stale again!

[obrok]

@JedWatson could you post a notification in this PR when/if you release this?

[chungeunkim]

+1

[sslotsky]

@JedWatson this has gone stale again. Are you not watching this repo?

[alpacaaa]

Thanks @sslotsky for the patch. For anyone in need of a quick hack, here's what I'm using:

import debounce from 'lodash.debounce'

class PatchedCodeMirror extends CodeMirror {
  constructor() {
    super()
    this.handleUpdate = debounce(function(nextProps) {
      if (this.codeMirror && nextProps.value !== undefined && this.codeMirror.getValue() !== nextProps.value) {
        this.codeMirror.setValue(nextProps.value)
      }
      if (typeof nextProps.options === 'object') {
        for (let optionName in nextProps.options) {
          if (nextProps.options.hasOwnProperty(optionName)) {
            this.codeMirror.setOption(optionName, nextProps.options[optionName])
          }
        }
      }
    }, 0)
  }

  componentWillReceiveProps(nextProps) {
    this.handleUpdate(nextProps)
  }
}

Hopefully this will get merged soon 😕

[JedWatson]

Hey @sslotsky sorry for the delay - I believe a valid fix for this has already been merged: see https://github.com/JedWatson/react-codemirror/blob/master/src/Codemirror.js#L39

componentWillMount () {
	this.componentWillReceiveProps = debounce(this.componentWillReceiveProps, 0);
},

If that doesn't work for you, I'll replace it with the method in this PR - let me know.

Thanks!

[JedWatson]

by the way, I've just published 0.3.0 including a fix that I think works

[alpacaaa]

@JedWatson can confirm 0.3.0 works as expected, thanks!