From 751ad55d02dccffbb2ba1b65e0ca883b25156d05 Mon Sep 17 00:00:00 2001 From: Kendall Garner <17521368+kgarner7@users.noreply.github.com> Date: Wed, 10 Sep 2025 17:49:13 -0700 Subject: [PATCH] remove all node selectors in useClickOutside --- .../features/context-menu/context-menu-provider.tsx | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/src/renderer/features/context-menu/context-menu-provider.tsx b/src/renderer/features/context-menu/context-menu-provider.tsx index 39732b44..6a2184d3 100644 --- a/src/renderer/features/context-menu/context-menu-provider.tsx +++ b/src/renderer/features/context-menu/context-menu-provider.tsx @@ -103,19 +103,13 @@ export const ContextMenuProvider = ({ children }: ContextMenuProviderProps) => { const { t } = useTranslation(); const [opened, setOpened] = useState(false); - const [contextMenuRef, setContextMenuRef] = useState(null); - const [ratingsRef, setRatingsRef] = useState(null); const [rating, setRating] = useState(0); useEffect(() => { setRating(0); }, [opened]); - const clickOutsideRef = useClickOutside( - () => setOpened(false), - ['mousedown', 'touchstart'], - [contextMenuRef, ratingsRef], - ); + const clickOutsideRef = useClickOutside(() => setOpened(false), ['mousedown', 'touchstart']); const viewport = useViewportSize(); const server = useCurrentServer(); @@ -882,7 +876,7 @@ export const ContextMenuProvider = ({ children }: ContextMenuProviderProps) => { leftIcon: , onClick: () => {}, rightIcon: ( - + { handleUpdateRating(e); @@ -950,7 +944,7 @@ export const ContextMenuProvider = ({ children }: ContextMenuProviderProps) => { {opened && ( - + {ctx.menuItems?.map((item) => { return (