Fix

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

diff --git a/modules/documentCanvas/canvas/documentElement.js b/modules/documentCanvas/canvas/documentElement.js
index d923787..0cf2e2c 100644 (file)
--- a/modules/documentCanvas/canvas/documentElement.js
+++ b/modules/documentCanvas/canvas/documentElement.js
@@ -3,8 +3,9 @@ define([
 'libs/underscore-min',
 'modules/documentCanvas/classAttributes',
 'modules/documentCanvas/canvas/utils',
-'modules/documentCanvas/canvas/widgets'
-], function($, _, classAttributes, utils, widgets) {
+'modules/documentCanvas/canvas/widgets',
+'modules/documentCanvas/canvas/wlxmlManagers'
+], function($, _, classAttributes, utils, widgets, wlxmlManagers) {
     
 'use strict';
 
@@ -113,19 +114,15 @@ var DocumentNodeElement = function(htmlElement, canvas) {
     DocumentElement.call(this, htmlElement, canvas);
 };
 
-var getDisplayStyle = function(tag, klass) {
-    if(tag === 'metadata')
-        return 'none';
-    if(tag === 'span')
-        return 'inline';
-    return 'block';
-}
-
 $.extend(DocumentNodeElement, {
     createDOM: function(params) {
-        var dom = $('<div document-node-element>'),
-            widgetsContainer = $('<div class="canvas-widgets" contenteditable="false">'),
-            container = $('<div document-element-content>');
+        var dom = $('<div>')
+                .attr('document-node-element', ''),
+            widgetsContainer = $('<div>')
+                .addClass('canvas-widgets')
+                .attr('contenteditable', false),
+            container = $('<div>')
+                .attr('document-element-content', '');
         
         dom.append(widgetsContainer, container);
         // Make sure widgets aren't navigable with arrow keys
@@ -142,7 +139,7 @@ $.extend(DocumentNodeElement, {
             });
         }
         element.data('other-attrs', params.others);
-        widgetsContainer.append(widgets.labelWidget(params.tag, params.klass));
+
         if(params.rawChildren) {
             container.append(params.rawChildren);
         }
@@ -301,8 +298,11 @@ $.extend(DocumentNodeElement.prototype, {
         return this._container().attr('wlxml-tag');
     },
     setWlxmlTag: function(tag) {
+        if(tag === this.getWlxmlTag())
+            return;
+
         this._container().attr('wlxml-tag', tag);
-        this._updateDisplayStyle();
+        this._updateWlxmlManager();
     },
     getWlxmlClass: function() {
         var klass = this._container().attr('wlxml-class');
@@ -311,6 +311,9 @@ $.extend(DocumentNodeElement.prototype, {
         return undefined;
     },
     setWlxmlClass: function(klass) {
+        if(klass === this.getWlxmlClass())
+            return;
+
         this.getWlxmlMetaAttrs().forEach(function(attr) {
             if(!classAttributes.hasMetaAttr(klass, attr.name))
                 this.dom().removeAttr('wlxml-meta-' + attr.name);
@@ -320,12 +323,12 @@ $.extend(DocumentNodeElement.prototype, {
             this._container().attr('wlxml-class', klass.replace(/\./g, '-'));
         else
             this._container().removeAttr('wlxml-class');
-        this._updateDisplayStyle();
+        this._updateWlxmlManager();
     },
-    _updateDisplayStyle: function() {
-        var displayStyle = getDisplayStyle(this.getWlxmlTag(), this.getWlxmlClass());
-        this.dom().css('display', displayStyle);
-        this._container().css('display', displayStyle);
+    _updateWlxmlManager: function() {
+        var manager = wlxmlManagers.getFor(this);
+        this.data('_wlxmlManager', manager);
+        manager.setup();
     },
     is: function(what) {
         if(what === 'list' && _.contains(['list.items', 'list.items.enum'], this.getWlxmlClass()))