use innerHTML in updateInput() function only once to improve performance

yabwe · Mar 25, 2016 · cb0cdbf · cb0cdbf
1 parent 5d52829
commit cb0cdbf
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/js/events.js b/src/js/events.js
@@ -391,12 +391,14 @@
             }
             // An event triggered which signifies that the user may have changed someting
             // Look in our cache of input for the contenteditables to see if something changed
-            var index = target.getAttribute('medium-editor-index');
-            if (target.innerHTML !== this.contentCache[index]) {
+            var index = target.getAttribute('medium-editor-index'),
+                html = target.innerHTML;
+
+            if (html !== this.contentCache[index]) {
                 // The content has changed since the last time we checked, fire the event
                 this.triggerCustomEvent('editableInput', eventObj, target);
             }
-            this.contentCache[index] = target.innerHTML;
+            this.contentCache[index] = html;
         },
 
         handleDocumentSelectionChange: function (event) {