Fix: Do not set text on canvas element if it didn't change.
authorAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Mon, 2 Dec 2013 13:29:47 +0000 (14:29 +0100)
committerAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Mon, 2 Dec 2013 13:50:54 +0000 (14:50 +0100)
On setting the same text on a text node Chrome does nothing,
but FF triggers ContentObserver handler.

src/editor/modules/documentCanvas/canvas/wlxmlListener.js

index ce9de6d..80da027 100644 (file)
@@ -73,7 +73,9 @@ var handlers = {
         if(toSet === '') {
             toSet = utils.unicode.ZWS;
         }
         if(toSet === '') {
             toSet = utils.unicode.ZWS;
         }
-        canvasElement.setText(toSet);
+        if(toSet !== canvasElement.getText()) {
+            canvasElement.setText(toSet);
+        }
     }
 };
 
     }
 };