Fix: Do not set text on canvas element if it didn't change.

[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 57a245e..16f2f68 100644 (file)
--- a/src/editor/modules/documentCanvas/canvas/keyboard.js
+++ b/src/editor/modules/documentCanvas/canvas/keyboard.js
@@ -47,18 +47,26 @@ handlers.push({key: KEYS.ENTER,
         if(Object.keys(cursor.getPosition()).length === 0) {
             var currentElement = canvas.getCurrentNodeElement();
             if(currentElement) {
         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:''});
                 added.append({text:''});

-                canvas.setCurrentElement(added, {caretTo: 'start'});
+                canvas.setCurrentElement(utils.findCanvasElement(added), {caretTo: 'start'});

             }
             return;
         }
 
         if(!cursor.isSelecting()) {
             if(event.ctrlKey) {
             }
             return;
         }
 
         if(!cursor.isSelecting()) {
             if(event.ctrlKey) {

-                var added = element.after({tag: element.getWlxmlTag() || 'div', klass: element.getWlxmlClass() || 'p'});
-                added.append({text:''});
-                canvas.setCurrentElement(added, {caretTo: 'start'});
+                if(element instanceof documentElement.DocumentTextElement) {
+                    element = element.parent();
+                }
+                var added = element.data('wlxmlNode').after(
+                    {tagName: element.getWlxmlTag() || 'div', attrs: {'class': element.getWlxmlClass() || 'p'}}
+                );
+                added.append({text: ''});
+                canvas.setCurrentElement(utils.findCanvasElement(added), {caretTo: 'start'});

 
             } else {
 
 
             } else {
 


@@ -66,25 +74,25 @@ handlers.push({key: KEYS.ENTER,
                     return false; // top level element is unsplittable
                 }
 
                     return false; // top level element is unsplittable
                 }
 

-                var elements = position.element.split({offset: position.offset}),
+                var nodes = position.element.data('wlxmlNode').split({offset: position.offset}),

                     newEmpty,
                     goto,
                     gotoOptions;
 
                 if(position.offsetAtBeginning)
                     newEmpty,
                     goto,
                     gotoOptions;
 
                 if(position.offsetAtBeginning)

-                    newEmpty = elements.first;
+                    newEmpty = nodes.first;

                 else if(position.offsetAtEnd)
                 else if(position.offsetAtEnd)

-                    newEmpty = elements.second;
+                    newEmpty = nodes.second;

                 
                 if(newEmpty) {
                 
                 if(newEmpty) {

-                    goto = newEmpty.append(documentElement.DocumentTextElement.create({text: ''}, this));
+                    goto = newEmpty.append({text: ''});

                     gotoOptions = {};
                 } else {
                     gotoOptions = {};
                 } else {

-                    goto = elements.second;
+                    goto = nodes.second;

                     gotoOptions = {caretTo: 'start'};
                 }
 
                     gotoOptions = {caretTo: 'start'};
                 }
 

-                canvas.setCurrentElement(goto, gotoOptions);
+                canvas.setCurrentElement(utils.findCanvasElement(goto), gotoOptions);

             }
         }
     }
             }
         }
     }


@@ -181,14 +189,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;
             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);
                 }
                 } else {
                     goto = element.getNearestTextElement(direction);
                 }

-                parent.detach();
+                parent.data('wlxmlNode').detach();

             } else {
                 goto = element.getNearestTextElement(direction);
             } else {
                 goto = element.getNearestTextElement(direction);

-                element.detach();
+                element.data('wlxmlNode').detach();

             }
             canvas.setCurrentElement(goto, {caretTo: caretTo});
             canvas.publisher('contentChanged');
             }
             canvas.setCurrentElement(goto, {caretTo: caretTo});
             canvas.publisher('contentChanged');