Only set options if they have changed; fixes JedWatson#53, closes JedWatson#93

Author: JedWatson

package.json

@@ -18,14 +18,15 @@
     "codemirror": "^5.18.2",
     "create-react-class": "^15.5.1",
     "lodash.debounce": "^4.0.8",
+    "lodash.isequal": "^4.5.0",
     "prop-types": "^15.5.4"
   },
   "devDependencies": {
     "gulp": "^3.9.1",
     "happiness": "^7.1.2",
     "react": "^15.5.3",
-    "react-dom": "^15.5.3",
-    "react-component-gulp-tasks": "^0.7.7"
+    "react-component-gulp-tasks": "^0.7.7",
+    "react-dom": "^15.5.3"
   },
   "peerDependencies": {
     "react": ">=15.5 <16",

src/Codemirror.js

@@ -4,6 +4,7 @@ const findDOMNode = ReactDOM.findDOMNode;
 const PropTypes = require('prop-types');
 const className = require('classnames');
 const debounce = require('lodash.debounce');
+const isEqual = require('lodash.isequal');
 const createReactClass = require('create-react-class');
 
 function normalizeLineEndings (str) {
@@ -69,11 +70,17 @@ const CodeMirror = createReactClass({
 		if (typeof nextProps.options === 'object') {
 			for (let optionName in nextProps.options) {
 				if (nextProps.options.hasOwnProperty(optionName)) {
-					this.codeMirror.setOption(optionName, nextProps.options[optionName]);
+					this.setOptionIfChanged(optionName, nextProps.options[optionName]);
 				}
 			}
 		}
 	},
+	setOptionIfChanged (optionName, newValue) {
+ 		const oldValue = this.codeMirror.getOption(optionName);
+ 		if (!isEqual(oldValue, newValue)) {
+ 			this.codeMirror.setOption(optionName, newValue);
+ 		}
+ 	},
 	getCodeMirror () {
 		return this.codeMirror;
 	},