Merge pull request #75 from lyleunderwood/strip-nbsp

strip nbsp for comparison

Author: lovasoa

lib/react-contenteditable.js

@@ -22,6 +22,10 @@ function _possibleConstructorReturn(self, call) { if (!self) { throw new Referen
 
 function _inherits(subClass, superClass) { if (typeof superClass !== "function" && superClass !== null) { throw new TypeError("Super expression must either be null or a function, not " + typeof superClass); } subClass.prototype = Object.create(superClass && superClass.prototype, { constructor: { value: subClass, enumerable: false, writable: true, configurable: true } }); if (superClass) Object.setPrototypeOf ? Object.setPrototypeOf(subClass, superClass) : subClass.__proto__ = superClass; }
 
+var stripNbsp = function stripNbsp(str) {
+  return str.replace(/ |\u202F|\u00A0/g, ' ');
+};
+
 var ContentEditable = function (_React$Component) {
   _inherits(ContentEditable, _React$Component);
 
@@ -70,7 +74,7 @@ var ContentEditable = function (_React$Component) {
       }
 
       // ...or if html really changed... (programmatically, not by user edit)
-      if (nextProps.html !== htmlEl.innerHTML && nextProps.html !== props.html) {
+      if (stripNbsp(nextProps.html) !== stripNbsp(htmlEl.innerHTML) && nextProps.html !== props.html) {
         return true;
       }
 

src/react-contenteditable.js

@@ -1,5 +1,7 @@
 import React from 'react';
 
+let stripNbsp = str => str.replace(/ |\u202F|\u00A0/g, ' ');
+
 export default class ContentEditable extends React.Component {
   constructor() {
     super();
@@ -34,7 +36,10 @@ export default class ContentEditable extends React.Component {
     }
 
     // ...or if html really changed... (programmatically, not by user edit)
-    if (nextProps.html !== htmlEl.innerHTML && nextProps.html !== props.html) {
+    if (
+      stripNbsp(nextProps.html) !== stripNbsp(htmlEl.innerHTML) &&
+      nextProps.html !== props.html
+    ) {
       return true;
     }