refactoring setting text on a node
authorAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Tue, 6 May 2014 09:58:05 +0000 (11:58 +0200)
committerAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Tue, 6 May 2014 10:04:21 +0000 (12:04 +0200)
src/editor/modules/documentCanvas/canvas/documentElement.js
src/editor/modules/documentCanvas/canvas/wlxmlListener.js

index a08788b..3073e72 100644 (file)
@@ -189,18 +189,14 @@ $.extend(DocumentTextElement.prototype, {
         this.canvas = null;
         return this;
     },
         this.canvas = null;
         return this;
     },
-    handle: function(event) {
-        var toSet = event.meta.node.getText();
-        if(toSet === '') {
-            toSet = utils.unicode.ZWS;
+    setText: function(text) {
+        if(text === '') {
+            text = utils.unicode.ZWS;
         }
         }
-        if(toSet !== this.getText()) {
-            this.setText(toSet);
+        if(text !== this.getText()) {
+            this.dom.contents()[0].data = text;
         }
     },
         }
     },
-    setText: function(text) {
-        this.dom.contents()[0].data = text;
-    },
     getText: function(options) {
         options = _.extend({raw: false}, options || {});
         var toret = this.dom.text();
     getText: function(options) {
         options = _.extend({raw: false}, options || {});
         var toret = this.dom.text();
index 54d6134..760208f 100644 (file)
@@ -79,7 +79,7 @@ var handlers = {
     },
     nodeTextChange: function(event) {
         var element = utils.getElementForNode(event.meta.node);
     },
     nodeTextChange: function(event) {
         var element = utils.getElementForNode(event.meta.node);
-        element.handle(event);
+        element.setText(event.meta.node.getText());
     },
 
     metadataChanged: _metadataEventHandler,
     },
 
     metadataChanged: _metadataEventHandler,