feat: add Menu/Submenu::remove_at (#93)

.changes/remove_at.md

@@ -0,0 +1,5 @@
+---
+"muda": "patch"
+---
+
+Add `Menu/Submenu::remove_at` to remove an item at specified index.

src/items/submenu.rs

@@ -132,6 +132,18 @@ impl Submenu {
         self.inner.borrow_mut().remove(item)
     }
 
+    /// Remove the menu item at the specified position from this submenu.
+    pub fn remove_at(&self, position: usize) -> Option<MenuItemKind> {
+        let mut items = self.items();
+        if items.len() > position {
+            let item = items.remove(position);
+            let _ = self.remove(item.as_ref());
+            Some(item)
+        } else {
+            None
+        }
+    }
+
     /// Returns a list of menu items that has been added to this submenu.
     pub fn items(&self) -> Vec<MenuItemKind> {
         self.inner.borrow().items()
@@ -159,8 +171,6 @@ impl Submenu {
         self.inner.borrow_mut().set_enabled(enabled)
     }
 
-    // TODO: in a minor release, rename the following two functions to be `set_as_*`
-
     /// Set this submenu as the Window menu for the application on macOS.
     ///
     /// This will cause macOS to automatically add window-switching items and

src/menu.rs

@@ -144,6 +144,18 @@ impl Menu {
         self.inner.borrow_mut().remove(item)
     }
 
+    /// Remove the menu item at the specified position from this menu.
+    pub fn remove_at(&self, position: usize) -> Option<MenuItemKind> {
+        let mut items = self.items();
+        if items.len() > position {
+            let item = items.remove(position);
+            let _ = self.remove(item.as_ref());
+            Some(item)
+        } else {
+            None
+        }
+    }
+
     /// Returns a list of menu items that has been added to this menu.
     pub fn items(&self) -> Vec<MenuItemKind> {
         self.inner.borrow().items()