From 669d269fd9c98f537859f971c2614bc97aff2c35 Mon Sep 17 00:00:00 2001
From: binwiederhier <pheckel@datto.com>
Date: Sat, 11 Feb 2023 10:52:19 -0500
Subject: [PATCH] Popup click should not open page

---
 server/server.go                 | 1 -
 web/src/components/Navigation.js | 9 ++++++++-
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/server/server.go b/server/server.go
index f232e0f5..36e76410 100644
--- a/server/server.go
+++ b/server/server.go
@@ -44,7 +44,6 @@ import (
 - MEDIUM: Test new token endpoints & never-expiring token
 - MEDIUM: Tests for manager.go
 - LOW: UI: Flickering upgrade banner when logging in
-- LOW: Menu item -> popup click should not open page
 - LOW: get rid of reservation id, replace with DELETE X-Topic: ...
 
 */
diff --git a/web/src/components/Navigation.js b/web/src/components/Navigation.js
index f4a20215..a7d0da0e 100644
--- a/web/src/components/Navigation.js
+++ b/web/src/components/Navigation.js
@@ -291,7 +291,14 @@ const SubscriptionItem = (props) => {
                     </ListItemIcon>
                 }
                 <ListItemIcon edge="end" sx={{minWidth: "26px"}}>
-                    <IconButton size="small" onMouseDown={(e) => e.stopPropagation()} onClick={(e) => setMenuAnchorEl(e.currentTarget)}>
+                    <IconButton
+                        size="small"
+                        onMouseDown={(e) => e.stopPropagation()}
+                        onClick={(e) => {
+                            e.stopPropagation();
+                            setMenuAnchorEl(e.currentTarget);
+                        }}
+                    >
                         <MoreVert fontSize="small"/>
                     </IconButton>
                 </ListItemIcon>