remove debugging code

[roojs1] / Roo / bootstrap / MenuItem.js

diff --git a/Roo/bootstrap/MenuItem.js b/Roo/bootstrap/MenuItem.js
index 8d954cc..cd2035e 100644 (file)
--- a/Roo/bootstrap/MenuItem.js
+++ b/Roo/bootstrap/MenuItem.js
@@ -12,10 +12,11 @@
  * Bootstrap MenuItem class
  * @cfg {String} html the menu label
  * @cfg {String} href the link
- * @cfg {Boolean} preventDefault do not trigger A href on clicks.
+ * @cfg {Boolean} preventDefault do not trigger A href on clicks (default false).
  * @cfg {Boolean} isContainer is it a container - just returns a drop down item..
  * @cfg {Boolean} active  used on sidebars to highlight active itesm
  * @cfg {String} fa favicon to show on left of menu item.
+ * @cfg {Roo.bootsrap.Menu} menu the child menu.
  * 
  * 
  * @constructor
@@ -42,7 +43,7 @@ Roo.extend(Roo.bootstrap.MenuItem, Roo.bootstrap.Component,  {
     
     href : false,
     html : false,
-    preventDefault: true,
+    preventDefault: false,
     isContainer : false,
     active : false,
     fa: false,
@@ -52,22 +53,26 @@ Roo.extend(Roo.bootstrap.MenuItem, Roo.bootstrap.Component,  {
         if(this.isContainer){
             return {
                 tag: 'li',
-                cls: 'dropdown-menu-item'
+                cls: 'dropdown-menu-item '
             };
         }
         var ctag = {
             tag: 'span',
             html: 'Link'
-        }
+        };
         
         var anc = {
             tag : 'a',
+            cls : 'dropdown-item',
             href : '#',
             cn : [  ]
         };
         
         if (this.fa !== false) {
-            
+            anc.cn.push({
+                tag : 'i',
+                cls : 'fa fa-' + this.fa
+            });
         }
         
         anc.cn.push(ctag);
@@ -92,18 +97,26 @@ Roo.extend(Roo.bootstrap.MenuItem, Roo.bootstrap.Component,  {
         return cfg;
     },
     
-    initEvents: function() {
+    initEvents: function()
+    {
         if (this.parent().type == 'treeview') {
             this.el.select('a').on('click', this.onClick, this);
         }
         
+        if (this.menu) {
+            this.menu.parentType = this.xtype;
+            this.menu.triggerEl = this.el;
+            this.menu = this.addxtype(Roo.apply({}, this.menu));
+        }
+        
     },
     onClick : function(e)
     {
         Roo.log('item on click ');
-        //if(this.preventDefault){
-        //    e.preventDefault();
-        //}
+        
+        if(this.preventDefault){
+            e.preventDefault();
+        }
         //this.parent().hideMenuItems();
         
         this.fireEvent('click', this, e);
@@ -111,7 +124,7 @@ Roo.extend(Roo.bootstrap.MenuItem, Roo.bootstrap.Component,  {
     getEl : function()
     {
         return this.el;
-    }
+    } 
 });