From 61ec341c27a89c94cea11f08e77f9ec8508a49a6 Mon Sep 17 00:00:00 2001
From: Elian Doran <contact@eliandoran.me>
Date: Sun, 7 Sep 2025 22:57:27 +0300
Subject: [PATCH] Revert "fix: close context menu when clicking items with
 submenus"

This reverts commit 2f93af4d6fdb89ba0773385c646db817ded2bf29.
---
 apps/client/src/menus/context_menu.ts | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/apps/client/src/menus/context_menu.ts b/apps/client/src/menus/context_menu.ts
index ef477c576..4411db9dc 100644
--- a/apps/client/src/menus/context_menu.ts
+++ b/apps/client/src/menus/context_menu.ts
@@ -62,7 +62,7 @@ class ContextMenu {
         if (this.isMobile) {
             this.$cover.on("click", () => this.hide());
         } else {
-            $(document).on("mouseup", (e) => this.hide());
+            $(document).on("click", (e) => this.hide());
         }
     }
 
@@ -225,10 +225,9 @@ class ContextMenu {
                 $item.on("mouseup", (e) => {
                     // Prevent submenu from failing to expand on mobile
                     if (!this.isMobile || !("items" in item && item.items)) {
+                        e.stopPropagation();
                         // Hide the content menu on mouse up to prevent the mouse event from propagating to the elements below.
-                        if (("command" in item) || ("handler" in item)) {
-                            this.hide();
-                        }
+                        this.hide();
                         return false;
                     }
                 });