fix: breaking from loop properly

[fnpeditor.git] / modules / documentCanvas / canvas.js

diff --git a/modules/documentCanvas/canvas.js b/modules/documentCanvas/canvas.js
index dc64cac..5de60f4 100644 (file)
--- a/modules/documentCanvas/canvas.js
+++ b/modules/documentCanvas/canvas.js
@@ -9,13 +9,14 @@ define([
 'use strict';\r
 \r
 var Canvas = function(xml) {\r
-    this.xml = xml;\r
     this.dom = $(template);\r
-    \r
-    this.content = this.dom.find('#rng-module-documentCanvas-content')\r
-    \r
-    \r
-    this.content.html(transformations.fromXML.getHTMLTree(xml));\r
+    this.content = this.dom.find('#rng-module-documentCanvas-content');\r
+    this.setXML(xml);\r
+}\r
+\r
+Canvas.prototype.setXML = function(xml) {\r
+    this.xml = xml;\r
+    this.content.html(transformations.fromXML.getHTMLTree(xml));  \r
 }\r
 \r
 Canvas.prototype.toXML = function() {\r
@@ -35,6 +36,14 @@ Canvas.prototype.getNode = function(desc) {
     return toret;\r
 }\r
 \r
+Canvas.prototype.getPreviousNode = function(options) {\r
+    var element = $(this.content.find('#' + options.node.id).get(0));\r
+    var prev = element.prev()\r
+    if(prev.length === 0)\r
+        prev = element.parent();\r
+    return new wlxmlNode.Node(prev);\r
+}\r
+\r
 Canvas.prototype._createNode = function(wlxmlTag, wlxmlClass) {\r
             var toBlock = ['div', 'document', 'section', 'header'];\r
             var htmlTag = _.contains(toBlock, wlxmlTag) ? 'div' : 'span';\r
@@ -48,8 +57,11 @@ Canvas.prototype._createNode = function(wlxmlTag, wlxmlClass) {
 \r
 Canvas.prototype.insertNode = function(options) {\r
     var element = $(this.content.find('#' + options.context.id).get(0));\r
-    if(options.place == 'after')\r
-        element[options.place](this._createNode(options.tag, options.klass));\r
+    if(options.place == 'after') {\r
+        var node = this._createNode(options.tag, options.klass);\r
+        element[options.place](node);\r
+        return node;\r
+    }\r
     else if(options.place == 'wrapText') {\r
         var elementContents = element.contents();\r
         if(elementContents.length !== 1 || elementContents.get(0).nodeType != 3)\r
@@ -102,6 +114,13 @@ Canvas.prototype.splitNode = function(options) {
         newNode.append(node)\r
     });\r
     nodeToSplit.after(newNode);\r
+    return newNode;\r
+}\r
+\r
+Canvas.prototype.removeNode = function(options) {\r
+    var toRemove = $(this.content.find('#' + options.node.id).get(0));\r
+    toRemove.remove();\r
+\r
 }\r
 \r
 Canvas.prototype.createList = function(options) {\r
@@ -128,14 +147,14 @@ Canvas.prototype.createList = function(options) {
             else {\r
                 $node = $(node);\r
             }\r
-            $node.attr('wlxml-class', 'list.item');\r
+            $node.attr('wlxml-class', 'item');\r
             nodesToWrap.push($node);\r
         }\r
         if(node.isSameNode(element2.get(0)))\r
-            return;\r
+            return false;\r
     });\r
     \r
-    var list = this._createNode('div', 'list');\r
+    var list = this._createNode('div', 'list-items');\r
     element1.before(list);\r
     \r
     nodesToWrap.forEach(function(node) {\r