Removing unnecessary element detach logic from Canvas DocumentNodeElement

[fnpeditor.git] / src / editor / modules / nodePane / nodePane.js

diff --git a/src/editor/modules/nodePane/nodePane.js b/src/editor/modules/nodePane/nodePane.js
index f3fb68a..1ecafe1 100644 (file)
--- a/src/editor/modules/nodePane/nodePane.js
+++ b/src/editor/modules/nodePane/nodePane.js
@@ -10,14 +10,21 @@ define([
 
 return function(sandbox) {
     
-    var view = $(_.template(templateSrc)({tagNames: wlxmlUtils.wlxmlTagNames, classNames: wlxmlUtils.wlxmlClassNames}));
+    var view = $(_.template(templateSrc)({tagNames: wlxmlUtils.wlxmlTagNames, classNames: wlxmlUtils.wlxmlClassNames})),
+        currentNode;
     
     view.on('change', 'select', function(e) {
         var target = $(e.target);
-        var attr = target.attr('class').split('-')[3] === 'tagSelect' ? 'tag' : 'class';
-        sandbox.publish('nodeElementChange', attr, target.val().replace(/-/g, '.'));
+        var attr = target.attr('class').split('-')[3] === 'tagSelect' ? 'Tag' : 'Class',
+            value = target.val().replace(/-/g, '.');
+                
+        if(attr === 'Class') {
+            //currentNode.document.transform('setClass', {node: currentNode, klass: value});
+            currentNode.transform('smartxml.setAttr', {name: 'class', value: value});
+        }
+        //currentNode['set' + attr](value);
     });
-    
+   
     return {
         start: function() {
             sandbox.publish('ready');
@@ -25,17 +32,29 @@ return function(sandbox) {
         getView: function() {
             return view;
         },
-        setNodeElement: function(nodeElement) {
-            view.find('.rng-module-nodePane-tagSelect').val(nodeElement.getWlxmlTag());
+        setNodeElement: function(wlxmlNodeElement) {
+            var module = this;
+            if(!currentNode) {
+                wlxmlNodeElement.document.on('change', function(event) {
+                    if(event.type === 'nodeAttrChange' && event.meta.node.sameNode(currentNode)) {
+                        module.setNodeElement(currentNode);
+                    }
+                });
+            }
 
-            var escapedClassName = (nodeElement.getWlxmlClass() || '').replace(/\./g, '-')
+            view.find('.rng-module-nodePane-tagSelect').val(wlxmlNodeElement.getTagName());
+
+            var escapedClassName = (wlxmlNodeElement.getClass() || '').replace(/\./g, '-');
             view.find('.rng-module-nodePane-classSelect').val(escapedClassName);
 
-            var widget = metaWidget.create({attrs:nodeElement.getWlxmlMetaAttrs()});
+            var widget = metaWidget.create({attrs:wlxmlNodeElement.getMetaAttributes()});
             widget.on('valueChanged', function(key, value) {
-                sandbox.publish('nodeElementChange', key, value);
+                wlxmlNodeElement.transform('wlxml.setMetaAttribute', {name: key, value: value});
+                //wlxmlNodeElement.setMetaAttribute(key, value);
             });
             view.find('.metaFields').empty().append(widget.el);
+
+            currentNode = wlxmlNodeElement;
         }
     };