Remove programmatic menu item triggering API

Eliminates the ability to programmatically trigger menu item events via the Trigger API in both C and C++ interfaces. Updates documentation, examples, and platform-specific implementations to reflect that events can now only be triggered through actual user interaction. Refactors Windows event handling to emit events directly from menu item click callbacks.

Author: lijy91

examples/menu_c_example/main.c

@@ -151,23 +151,11 @@ int main() {
 
   printf("Added submenu with %zu items\n", native_menu_get_item_count(submenu));
 
-  // Demonstrate programmatic triggering
-  printf("\n=== Testing Programmatic Event Triggering ===\n");
-
-  printf("Triggering file item...\n");
-  native_menu_item_trigger(file_item);
-
-  printf("Triggering checkbox item...\n");
-  native_menu_item_trigger(checkbox_item);
-
-  printf("Triggering checkbox item again...\n");
-  native_menu_item_trigger(checkbox_item);
-
-  printf("Switching radio button...\n");
-  native_menu_item_trigger(radio_item2);
-
-  printf("Triggering exit item...\n");
-  native_menu_item_trigger(exit_item);
+  // Note: Programmatic event triggering is no longer available via trigger API.
+  // Events can only be triggered through actual user interaction.
+  printf("\n=== Programmatic Event Triggering Removed ===\n");
+  printf("Note: The trigger API has been removed. Events are now only "
+         "triggered through user interaction.\n");
 
   // Demonstrate listener removal
   printf("\n=== Testing Listener Removal ===\n");
@@ -179,8 +167,8 @@ int main() {
     printf("Failed to remove checkbox click listener\n");
   }
 
-  printf("Triggering checkbox item after removing click listener...\n");
-  native_menu_item_trigger(checkbox_item);
+  printf("Checkbox item listener removed. Events will now only be triggered "
+         "through user interaction.\n");
 
   // Open menu as context menu (this may not work in console applications)
   printf("\n=== Attempting to Open Context Menu ===\n");
@@ -207,13 +195,11 @@ int main() {
                                     on_menu_item_clicked, (void*)"Additional Test 2");
 
   printf("Added multiple listeners for the same event\n");
-  printf("Triggering item with multiple listeners...\n");
-  native_menu_item_trigger(additional_item);
+  printf("Multiple listeners can be registered for the same event type.\n");
 
   // Remove one listener
   native_menu_item_remove_listener(additional_item, additional_listener1);
-  printf("Removed first listener, triggering again...\n");
-  native_menu_item_trigger(additional_item);
+  printf("Removed first listener. Remaining listener will receive events.\n");
 
   native_menu_item_destroy(additional_item);
 
@@ -228,11 +214,10 @@ int main() {
   printf(
       "5. Handling NATIVE_MENU_ITEM_EVENT_SUBMENU_OPENED and "
       "NATIVE_MENU_ITEM_EVENT_SUBMENU_CLOSED\n");
-  printf("6. Programmatic event triggering\n");
-  printf("7. Event listener removal with native_menu_item_remove_listener()\n");
-  printf("8. Multiple listeners for the same event type\n");
-  printf("9. Manual state management for checkbox and radio items\n");
-  printf("10. Submenu support with event handling\n");
+  printf("6. Event listener removal with native_menu_item_remove_listener()\n");
+  printf("7. Multiple listeners for the same event type\n");
+  printf("8. Manual state management for checkbox and radio items\n");
+  printf("9. Submenu support with event handling\n");
 
   // Cleanup
   native_menu_item_destroy(file_item);

examples/menu_example/main.cpp

@@ -83,23 +83,23 @@ int main() {
 
     std::cout << "Menu created with " << menu->GetItemCount() << " items" << std::endl;
 
-    // Demonstrate programmatic triggering
+    // Demonstrate programmatic triggering by emitting events directly
     std::cout << "\n=== Testing Programmatic Event Triggering ===" << std::endl;
 
     std::cout << "Triggering file item..." << std::endl;
-    file_item->Trigger();
+    file_item->Emit(MenuItemClickedEvent(file_item->GetId()));
 
     std::cout << "Triggering checkbox item..." << std::endl;
-    checkbox_item->Trigger();
+    checkbox_item->Emit(MenuItemClickedEvent(checkbox_item->GetId()));
 
     std::cout << "Triggering checkbox item again..." << std::endl;
-    checkbox_item->Trigger();
+    checkbox_item->Emit(MenuItemClickedEvent(checkbox_item->GetId()));
 
     std::cout << "Switching radio button..." << std::endl;
-    radio_item2->Trigger();
+    radio_item2->Emit(MenuItemClickedEvent(radio_item2->GetId()));
 
     std::cout << "Triggering exit item..." << std::endl;
-    exit_item->Trigger();
+    exit_item->Emit(MenuItemClickedEvent(exit_item->GetId()));
 
     // Open menu as context menu (this may not work in console applications)
     std::cout << "\n=== Attempting to Open Context Menu ===" << std::endl;
@@ -149,8 +149,8 @@ int main() {
 
     // Test submenu items
     std::cout << "Triggering submenu items..." << std::endl;
-    submenu_item1->Trigger();
-    submenu_item2->Trigger();
+    submenu_item1->Emit(MenuItemClickedEvent(submenu_item1->GetId()));
+    submenu_item2->Emit(MenuItemClickedEvent(submenu_item2->GetId()));
 
     std::cout << "\n=== Event System Demo Complete ===" << std::endl;
     std::cout << "This example demonstrates:" << std::endl;
@@ -161,7 +161,7 @@ int main() {
     std::cout << "5. Handling MenuItemSubmenuOpenedEvent and "
                  "MenuItemSubmenuClosedEvent"
               << std::endl;
-    std::cout << "6. Programmatic event triggering" << std::endl;
+    std::cout << "6. Programmatic event emission using Emit()" << std::endl;
     std::cout << "7. Submenu support with event propagation" << std::endl;
 
   } catch (const std::exception& e) {

src/capi/menu_c.cpp

@@ -596,18 +596,6 @@ bool native_menu_item_remove_listener(native_menu_item_t item, int listener_id)
   }
 }
 
-bool native_menu_item_trigger(native_menu_item_t item) {
-  if (!item)
-    return false;
-
-  try {
-    auto menu_item = static_cast<MenuItem*>(item);
-    return menu_item->Trigger();
-  } catch (...) {
-    return false;
-  }
-}
-
 // Menu C API Implementation
 
 native_menu_t native_menu_create(void) {

src/capi/menu_c.h

@@ -346,14 +346,6 @@ int native_menu_item_add_listener(native_menu_item_t item,
 FFI_PLUGIN_EXPORT
 bool native_menu_item_remove_listener(native_menu_item_t item, int listener_id);
 
-/**
- * Programmatically trigger a menu item
- * @param item The menu item
- * @return true if triggered successfully, false otherwise
- */
-FFI_PLUGIN_EXPORT
-bool native_menu_item_trigger(native_menu_item_t item);
-
 /**
  * Menu operations
  */

src/menu.h

@@ -416,16 +416,6 @@ class MenuItem : public EventEmitter<MenuEvent>, public NativeObjectProvider {
    */
   void RemoveSubmenu();
 
-  /**
-   * @brief Programmatically trigger this menu item.
-   *
-   * Simulates a user click on the menu item, firing the appropriate
-   * events and callbacks.
-   *
-   * @return true if the item was successfully triggered, false otherwise
-   */
-  bool Trigger();
-
  protected:
   /**
    * @brief Internal method to get the platform-specific native menu item object.

src/platform/linux/menu_linux.cpp

@@ -260,16 +260,6 @@ void MenuItem::RemoveSubmenu() {
   }
 }
 
-bool MenuItem::Trigger() {
-  if (!IsEnabled())
-    return false;
-  if (pimpl_->gtk_menu_item_) {
-    g_signal_emit_by_name(pimpl_->gtk_menu_item_, "activate");
-    return true;
-  }
-  return false;
-}
-
 void* MenuItem::GetNativeObjectInternal() const {
   return (void*)pimpl_->gtk_menu_item_;
 }

src/platform/macos/menu_macos.mm

@@ -524,17 +524,6 @@ - (void)menuDidClose:(NSMenu*)menu {
   [pimpl_->ns_menu_item_ setSubmenu:nil];
 }
 
-bool MenuItem::Trigger() {
-  if (!IsEnabled())
-    return false;
-
-  // Call the block directly instead of going through target-action
-  if (pimpl_->ns_menu_item_target_.clickedBlock) {
-    pimpl_->ns_menu_item_target_.clickedBlock(pimpl_->id_);
-  }
-  return true;
-}
-
 void* MenuItem::GetNativeObjectInternal() const {
   return (__bridge void*)pimpl_->ns_menu_item_;
 }