Refactor item command class to use item widget

src/ItemCommand.h

@@ -1,66 +1,41 @@
-#ifndef ItemCommand_H
-#define ItemCommand_H
+#pragma once
 
-#include "MenuItem.h"
+#include "BaseItemZeroWidget.h"
 
 /**
- * @class ItemCommand
- * @brief Represents a menu item that executes a command when selected.
+ * @brief A menu item that executes a callback function when selected.
  *
- * The ItemCommand class inherits from MenuItem and allows for the execution
- * of a callback function when the item is entered. This is useful for creating
- * interactive menu items in an LCD menu system.
+ * This class extends the BaseItemZeroWidget class and provides a menu item
+ * that executes a callback function when selected. The callback function is
+ * provided as a function pointer during construction.
  *
- * @note The callback function should match the signature defined by the fptr type.
+ * As a BaseItemZeroWidget, this item responds to selection events in the menu
+ * system. When the user confirms the selection, handleCommit is triggered,
+ * which executes the provided callback.
  */
-class ItemCommand : public MenuItem {
+class ItemCommand : public BaseItemZeroWidget {
   private:
-    // Declare a function pointer for the command callback.
-    fptr callback;
+    void (*callback)();
 
   public:
-    /**
-     * Construct a new ItemCommand object.
-     *
-     * @param text The text of the item.
-     * @param callback A reference to the callback function to be invoked when
-     * the item is entered.
-     */
-    ItemCommand(const char* text, fptr callback) : MenuItem(text), callback(callback) {}
-
-    /**
-     * Get the callback function for this item.
-     *
-     * @return The function pointer to the callback function.
-     */
-    fptr getCallback() { return callback; }
-
-    /**
-     * Set the callback function for this item.
-     *
-     * @param callback A reference to the new callback function to be invoked
-     * when the item is entered.
-     */
-    void setCallBack(fptr callback) { this->callback = callback; };
+    ItemCommand(const char* text, void (*callback)()) : BaseItemZeroWidget(text), callback(callback) {}
 
   protected:
-    bool process(LcdMenu* menu, const unsigned char command) override {
-        switch (command) {
-            case ENTER:
-                executeCommand();
-                return true;
-            default:
-                return false;
-        }
-    }
-    void executeCommand() {
-        if (callback != NULL) {
-            callback();
-        }
-        LOG(F("ItemCommand::enter"), text);
+    void handleCommit(LcdMenu* menu) override {
+        if (callback) callback();
     }
 };
 
-#define ITEM_COMMAND(...) (new ItemCommand(__VA_ARGS__))
-
-#endif  // ITEM_COMMAND_H
+/**
+ * @brief Create a new command item.
+ *
+ * @param text The text to display for the item.
+ * @param callback The function to call when the item is selected.
+ * @return MenuItem* The created item. Caller takes ownership of the returned pointer.
+ *
+ * @example
+ *   auto item = ITEM_COMMAND("Save", []() { save_data(); });
+ */
+inline MenuItem* ITEM_COMMAND(const char* text, void (*callback)()) {
+    return new ItemCommand(text, callback);
+}