Removing unnecessary element detach logic from Canvas DocumentNodeElement

[fnpeditor.git] / src / editor / modules / documentCanvas / canvas / keyboard.js

diff --git a/src/editor/modules/documentCanvas/canvas/keyboard.js b/src/editor/modules/documentCanvas/canvas/keyboard.js
index ee36332..e48917c 100644 (file)
--- a/src/editor/modules/documentCanvas/canvas/keyboard.js
+++ b/src/editor/modules/documentCanvas/canvas/keyboard.js
@@ -47,9 +47,12 @@ handlers.push({key: KEYS.ENTER,
         if(Object.keys(cursor.getPosition()).length === 0) {
             var currentElement = canvas.getCurrentNodeElement();
             if(currentElement) {
-                var added = currentElement.after({tag: currentElement.getWlxmlTag() || 'div', klass: currentElement.getWlxmlClass() || 'p'});
+                var added = currentElement.data('wlxmlNode').after({
+                    tag: currentElement.getWlxmlTag() || 'div',
+                    attrs: {'class': currentElement.getWlxmlClass() || 'p'}
+                });
                 added.append({text:''});
-                canvas.setCurrentElement(added, {caretTo: 'start'});
+                canvas.setCurrentElement(utils.findCanvasElement(added), {caretTo: 'start'});
             }
             return;
         }
@@ -71,25 +74,36 @@ handlers.push({key: KEYS.ENTER,
                     return false; // top level element is unsplittable
                 }
 
-                var elements = position.element.split({offset: position.offset}),
-                    newEmpty,
-                    goto,
-                    gotoOptions;
+                //var nodes = position.element.data('wlxmlNode').split({offset: position.offset}),
+                // var nodes = position.element.data('wlxmlNode').transform('split', {offset: position.offset}),
+                //     newEmpty,
+                //     goto,
+                //     gotoOptions;
 
-                if(position.offsetAtBeginning)
-                    newEmpty = elements.first;
-                else if(position.offsetAtEnd)
-                    newEmpty = elements.second;
+                // if(position.offsetAtBeginning)
+                //     newEmpty = nodes.first;
+                // else if(position.offsetAtEnd)
+                //     newEmpty = nodes.second;
                 
-                if(newEmpty) {
-                    goto = newEmpty.append(documentElement.DocumentTextElement.create({text: ''}, this));
+                // if(newEmpty) {
+                //     //goto = newEmpty.append({text: ''});
+                //     gotoOptions = {};
+                // } else {
+                //     goto = nodes.second;
+                //     gotoOptions = {caretTo: 'start'};
+                // }
+
+                var result = position.element.data('wlxmlNode').breakContent({offset: position.offset}),
+                    goto, gotoOptions;
+                if(result.emptyText) {
+                    goto = result.createdEmpty;
                     gotoOptions = {};
                 } else {
-                    goto = elements.second;
-                    gotoOptions = {caretTo: 'start'};
+                    goto = result.second;
+                    gotoOptions = {caretTo: 'start'};   
                 }
 
-                canvas.setCurrentElement(goto, gotoOptions);
+                canvas.setCurrentElement(utils.findCanvasElement(goto), gotoOptions);
             }
         }
     }
@@ -186,14 +200,14 @@ handlers.push({keys: [KEYS.BACKSPACE, KEYS.DELETE],
                 goto;
             if(parent.children().length === 1 && parent.children()[0].sameNode(element)) {
                 if(grandParent && grandParent.children().length === 1) {
-                    goto = grandParent.append({text: ''});
+                    goto = grandParent.data('wlxmlNode').append({text: ''});
                 } else {
                     goto = element.getNearestTextElement(direction);
                 }
-                parent.detach();
+                parent.data('wlxmlNode').detach();
             } else {
                 goto = element.getNearestTextElement(direction);
-                element.detach();
+                element.data('wlxmlNode').detach();
             }
             canvas.setCurrentElement(goto, {caretTo: caretTo});
             canvas.publisher('contentChanged');