Added RemoveMenuItem

src/command/Menus.js

@@ -186,16 +186,6 @@ define(function (require, exports, module) {
         return binding;
     }
 
-    /** NOT IMPLEMENTED
-     * Removes MenuItem
-     * 
-     * TODO Question: for convenience should API provide a way to remove related
-     * keybindings and Command object?
-     */
-    // function removeMenuItem(id) {
-    //    NOT IMPLEMENTED
-    // }
-
     var _menuDividerIDCount = 1;
     function _getNextMenuItemDividerID() {
         return "brackets-menuDivider-" + _menuDividerIDCount++;
@@ -366,6 +356,39 @@ define(function (require, exports, module) {
 
         return $relativeElement;
     };
+
+    /**
+     * Removes a  menu item with the specified id. 
+     *
+     * @param {!string | Command} command - the command the menu would execute if we weren't deleting it.
+     *
+     * Note, keyBindings are not affected at all by removing a menu item. 
+     * They would have to be removed separately.
+     * 
+     */
+    Menu.prototype.removeMenuItem = function (command) {
+        var menuItemID;
+
+        if (!command) {
+            throw new Error("removeMenuItem(): missing required parameters: command");
+        }
+
+        if (typeof (command) === "string") {
+            var commandObj = CommandManager.get(command);
+
+            if (!commandObj) {
+                throw new Error("removeMenuItem(): command not found: " + command);
+            }
+
+            menuItemID = menuItemMap[this.id + "-" + command].id;
+        } else {
+            menuItemID = this.id + "-" + command.getID();
+        }
+
+        //Targeting parent to get the menu item <a> and the <li> that contains it
+        $(_getHTMLMenuItem(menuItemID)).parent().remove();
+        delete menuItemMap[menuItemID];
+    };
 
     /**
      * Adds a new menu item with the specified id and display text. The insertion position is

test/spec/Menu-test.js

@@ -211,6 +211,89 @@ define(function (require, exports, module) {
             });
         });
 
+        describe("Remove Menu Items", function () {
+            it("should add then remove new menu item to empty menu with a command id", function () {
+                runs(function () {
+                    CommandManager.register("Brackets Test Command Custom", "custom.command", function () {});
+                    var commandString = "custom.command";
+                    var menu = Menus.addMenu("Custom", "menu-custom");
+                    var $listItems = testWindow.$("#menu-custom > ul").children();
+                    expect($listItems.length).toBe(0);
+
+                    // Re-use commands that are already registered
+                    var menuItem = menu.addMenuItem(commandString);
+                    expect(menuItem).not.toBeNull();
+                    expect(menuItem).toBeDefined();
+
+                    expect(typeof (commandString)).toBe("string");
+
+                    $listItems = testWindow.$("#menu-custom > ul").children();
+                    expect($listItems.length).toBe(1);
+                    expect($($listItems[0]).length).toBe(1);
+
+                    menu.removeMenuItem(commandString);
+                    $listItems = testWindow.$("#menu-custom > ul").children();
+                    expect($listItems.length).toBe(0);
+                    expect($($listItems[0]).length).toBe(0);
+                });
+            });
+
+            it("should add then remove new menu item to empty menu with a command", function () {
+                runs(function () {
+                    CommandManager.register("Brackets Test Command Custom", "custom.command", function () {});
+                    var commandString = "custom.command";
+                    var menu = Menus.addMenu("Custom", "menu-custom");
+                    var $listItems = testWindow.$("#menu-custom > ul").children();
+                    expect($listItems.length).toBe(0);
+
+                    // Re-use commands that are already registered
+                    var menuItem = menu.addMenuItem(commandString);
+                    expect(menuItem).not.toBeNull();
+                    expect(menuItem).toBeDefined();
+
+                    $listItems = testWindow.$("#menu-custom > ul").children();
+                    expect($listItems.length).toBe(1);
+                    expect($($listItems[0]).length).toBe(1);
+
+                    var command = CommandManager.get(commandString);
+                    expect(typeof (command)).toBe("object");
+
+                    menu.removeMenuItem(command);
+                    $listItems = testWindow.$("#menu-custom > ul").children();
+                    expect($listItems.length).toBe(0);
+                    expect($($listItems[0]).length).toBe(0);
+                });
+            });
+
+            it("should gracefully handle someone trying to delete a menu item that doesn't exist", function () {
+                runs(function () {
+                    var commandString = "custom.command";
+                    var menu = Menus.addMenu("Custom", "menu-custom");
+
+                    var exceptionThrown = false;
+                    try {
+                        menu.removeMenuItem(commandString);
+                    } catch (e) {
+                        exceptionThrown = true;
+                    }
+                    expect(exceptionThrown).toBeTruthy();
+                });
+            });
+
+            it("should gracefully handle someone trying to delete nothing", function () {
+                runs(function () {
+                    var menu = Menus.addMenu("Custom", "menu-custom");
+
+                    var exceptionThrown = false;
+                    try {
+                        menu.removeMenuItem();
+                    } catch (e) {
+                        exceptionThrown = true;
+                    }
+                    expect(exceptionThrown).toBeTruthy();
+                });
+            });
+        });
 
         describe("Add Menu Items", function () {