X-Git-Url: http://git.roojs.org/?p=app.Builder.js;a=blobdiff_plain;f=resources%2Froo.builder.js;h=a53f114afc92243b84185d2071d73cab282743a8;hp=711499507085d0b31e9bc8c8de2a46ba60a72f22;hb=HEAD;hpb=7ed95495234bb7decb81b254c959edbcebb7a6a5

diff --git a/resources/roo.builder.js b/resources/roo.builder.js
index 711499507..a53f114af 100644
--- a/resources/roo.builder.js
+++ b/resources/roo.builder.js
@@ -54,6 +54,9 @@ Builder  = {
                 return;
             
             }
+            if (nodeName.match(/^#/)) { // comments?
+                return;
+            }
              
             
             
@@ -62,7 +65,7 @@ Builder  = {
                 return;
             }
             if (nodeName != 'BODY') {
-                
+                //Roo.log(currentElement);
                 if (currentElement.hasAttribute('flexy:include')) {
 
 
@@ -75,15 +78,17 @@ Builder  = {
                 if (tagName) {
                     var attr = [];
                     for(i = 0; i < currentElement.attributes.length;i++) {
-                        var aname = currentElement.attributes.item(i).name;
-                        if (aname=='id') {
+                        var attre = currentElement.attributes.item(i);
+                        var aname = attre.name;
+                        var aval = attre.value || '';
+                        if (aname=='id' && aval.match(/^roo\-/)) { // only roo -elements hide..
                             aname= 'xbuilderid';
                         }
-                        // skip
-                        if (currentElement.attributes.item(i).value == 'builderel') {
+                        // skip -- component created by the builder?
+                        if (aval == 'builderel') { 
                             return;
                         }
-                        attr.push(aname + '="' + currentElement.attributes.item(i).value + '"' );
+                        attr.push(aname + '="' + aval + '"' );  //?? escaping attributes?
                     }
                     
                     
@@ -311,6 +316,9 @@ Builder  = {
     
     applyFlexy: function(tree)
     {
+        if (!tree.el) {
+            return;
+        }
         if (typeof(tree['flexy:foreach']) != 'undefined') {
             //Roo.log("add flexy:foreach");
             tree.el.attr('flexy:foreach', tree['flexy:foreach']);
@@ -328,13 +336,28 @@ Builder  = {
             //Roo.log("add flexy:if");
             tree.el.attr('flexy:include', tree['flexy:include']);
         }
-        Roo.log("Add xtype")
+        
+        if (typeof(tree['flexy:ignore']) != 'undefined') {
+            //Roo.log("add flexy:if");
+            tree.el.attr('flexy:ignore', tree['flexy:ignore']);
+        }
+        
         tree.el.attr('xtype', tree['|xns'] + '.' +  tree['xtype']);
-        if (!tree.items || !tree.items.length) { return; }
+        //Roo.log(tree);
+        //Roo.log("Add xtype")
+        
+        
         
-        for (var i = 0; i < tree.items.length; i++){
-            this.applyFlexy(tree.items[i]);
+        if (tree.items && tree.items.length > 0) { 
+        //Roo.log(tree);
+            for (var i = 0; i < tree.items.length; i++){
+                this.applyFlexy(tree.items[i]);
+            }
+        }
+        if (typeof(tree.menu) == 'object') {
+            this.applyFlexy(tree.menu);
         }
+        
     }