Disable menu for non configuration items

rk4n3 · Dec 24, 2015 · 6ab558d · 6ab558d
1 parent 7abfcac
commit 6ab558d
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 2 deletions.
diff --git a/src/main/webapp/resources/net/apachegui/ConfigurationTree.js b/src/main/webapp/resources/net/apachegui/ConfigurationTree.js
@@ -119,27 +119,31 @@ define([
             this.menu.bindDomNode(div);
 
             this.menu.addChild(new MenuItem({
+                id: this.id + '-menu-item-expand-all',
                 label : "Expand All",
                 onClick : function() {
                     that.expandAll();
                 }
             }));
 
             this.menu.addChild(new MenuItem({
+                id: this.id + '-menu-item-collapse-all',
                 label : "Collapse All",
                 onClick : function() {
                     that.collapseAll();
                 }
             }));
 
             this.menu.addChild(new MenuItem({
+                id: this.id + '-menu-item-edit',
                 label : "Edit",
                 onClick : function() {
                     that._showEditLineDialog();
                 }
             }));
 
             this.menu.addChild(new MenuItem({
+                id: this.id + '-menu-item-delete',
                 label : "Delete",
                 onClick : function() {
                     that._deleteLine()
@@ -148,18 +152,21 @@ define([
 
             var subMenu = new Menu();
             subMenu.addChild(new MenuItem({
+                id: this.id + '-menu-item-add-enclosure',
                 label : "New Enclosure",
                 onClick: function() {
                     that._showAddLineDialog(that.LineTypes.ENCLOSURE);
                 }
             }));
             subMenu.addChild(new MenuItem({
+                id: this.id + '-menu-item-add-directive',
                 label : "New Directive",
                 onClick: function() {
                     that._showAddLineDialog(that.LineTypes.DIRECTIVE);
                 }
             }));
             this.menu.addChild(new PopupMenuItem({
+                id: this.id + '-menu-item-add-submenu',
                 label : "Add",
                 popup : subMenu
             }));
@@ -535,6 +542,15 @@ define([
             on(this.menu, "focus", function(e) {
                 var tn = registry.getEnclosingWidget(this.currentTarget);
                 that.currentTreeItem = tn.item;
+
+                var lineType = that._getItemProperty(that.currentTreeItem, 'lineType');
+                var isConfigurationItem = (lineType == that.LineTypes.ENCLOSURE || lineType == that.LineTypes.DIRECTIVE);
+
+                registry.byId(that.id + '-menu-item-edit').set('disabled', !isConfigurationItem);
+                registry.byId(that.id + '-menu-item-delete').set('disabled', !isConfigurationItem);
+                registry.byId(that.id + '-menu-item-add-enclosure').set('disabled', !isConfigurationItem);
+                registry.byId(that.id + '-menu-item-add-directive').set('disabled', !isConfigurationItem);
+
                 that.onMenuFocus();
             });
         },

diff --git a/src/main/webapp/resources/net/apachegui/VirtualHosts.js b/src/main/webapp/resources/net/apachegui/VirtualHosts.js
@@ -22,8 +22,7 @@ define([ "dojo/_base/declare",
          "dijit/Tooltip",
          "dojo/dom-attr",
          "net/apachegui/ConfigurationTree",
-         "net/apachegui/InputAutoSuggest"
-], function(declare, dom, request, registry, on, ItemFileWriteStore, DataGrid, TitlePane, RefreshableTree, Tree, ForestStoreModel, Observable, Menu, MenuItem, PopupMenuItem, Select, scroll, query, array, lang, domConstruct, Tooltip, domAttr, ConfigurationTree, InputAutoSuggest) {
+], function(declare, dom, request, registry, on, ItemFileWriteStore, DataGrid, TitlePane, RefreshableTree, Tree, ForestStoreModel, Observable, Menu, MenuItem, PopupMenuItem, Select, scroll, query, array, lang, domConstruct, Tooltip, domAttr, ConfigurationTree) {
 
     declare("net.apachegui.VirtualHosts", null, {