findCanvasElementInParent
authorAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Mon, 18 Nov 2013 08:40:41 +0000 (09:40 +0100)
committerAleksander Łukasz <aleksander.lukasz@nowoczesnapolska.org.pl>
Mon, 2 Dec 2013 13:50:52 +0000 (14:50 +0100)
src/editor/modules/documentCanvas/canvas/utils.js

index 87911cf..41a2a0a 100644 (file)
@@ -24,21 +24,29 @@ var findCanvasElement = function(node) {
         return node.getData('canvasElement');
     }
     if(node.nodeType === Node.TEXT_NODE) {
         return node.getData('canvasElement');
     }
     if(node.nodeType === Node.TEXT_NODE) {
-        var parent = node.parent(),
-            toret;
-
-        parent.children().forEach(function(child) {
-            if(child.data('wlxmlNode').sameNode(node))
-                toret = child;
-        });
-        if(toret)
-            return toret;
+        return findCanvasElementInParent(node, node.parent())
     }
 };
 
     }
 };
 
+/**
+ * We take child and its parent as arguments separatly to
+ * handle situation where child was removed from WLXMLDocument
+ * and it lost reference to its parent (but we may still have it on canvas).
+*/
+var findCanvasElementInParent = function(wlxmlChildNode, wlxmlParentNode) {
+    var parentElement = findCanvasElement(wlxmlParentNode),
+        toret;
+    parentElement.children().forEach(function(child) {
+        if(child.data('wlxmlNode').sameNode(wlxmlChildNode))
+            toret = child;
+    });
+    return toret;
+}
+
 return {
     nearestInDocumentOrder: nearestInDocumentOrder,
     findCanvasElement: findCanvasElement,
 return {
     nearestInDocumentOrder: nearestInDocumentOrder,
     findCanvasElement: findCanvasElement,
+    findTextElementInParent: findTextElementInParent,
     unicode: {
         ZWS: '\u200B'
     }
     unicode: {
         ZWS: '\u200B'
     }