From 80f3cbe759008712b6bc3809acd4bc36371dee78 Mon Sep 17 00:00:00 2001
From: philmoz <philmoz@users.noreply.github.com>
Date: Tue, 24 Jan 2023 12:05:45 +1100
Subject: [PATCH] fix(color): Prevent multiple instances of the main menu popup
 (#3067)

---
 radio/src/gui/colorlcd/view_main_menu.cpp |  5 +++++
 radio/src/gui/colorlcd/view_main_menu.h   | 16 +---------------
 2 files changed, 6 insertions(+), 15 deletions(-)

diff --git a/radio/src/gui/colorlcd/view_main_menu.cpp b/radio/src/gui/colorlcd/view_main_menu.cpp
index bf71f11e3..97cce66fd 100644
--- a/radio/src/gui/colorlcd/view_main_menu.cpp
+++ b/radio/src/gui/colorlcd/view_main_menu.cpp
@@ -135,6 +135,11 @@ void ViewMainMenu::deleteLater(bool detach, bool trash)
   Window::deleteLater(detach, trash);
 }
 
+void ViewMainMenu::onClicked()
+{
+  deleteLater();
+}
+
 void ViewMainMenu::onCancel()
 {
   deleteLater();
diff --git a/radio/src/gui/colorlcd/view_main_menu.h b/radio/src/gui/colorlcd/view_main_menu.h
index a6161379c..560d98eed 100644
--- a/radio/src/gui/colorlcd/view_main_menu.h
+++ b/radio/src/gui/colorlcd/view_main_menu.h
@@ -29,24 +29,10 @@ class ViewMainMenu : public Window
   ViewMainMenu(Window* parent);
 
   void onCancel() override;
+  void onClicked() override;
   void paint(BitmapBuffer* dc) override;
   void deleteLater(bool detach = true, bool trash = true) override;
 
-#if defined(HARDWARE_TOUCH)
-  bool onTouchStart(coord_t /*x*/, coord_t /*y*/) override { return true; }
-  bool onTouchEnd(coord_t x, coord_t y) override
-  {
-    deleteLater();
-    return true;
-  }
-  bool onTouchSlide(coord_t x, coord_t y, coord_t startX, coord_t startY,
-                    coord_t slideX, coord_t slideY) override
-  {
-    Window::onTouchSlide(x, y, startX, startY, slideX, slideY);
-    return true;
-  }
-#endif
-
  protected:
   rect_t carouselRect;
 };