From 1ac267fa99747efc35e0648b9141427309b5cc30 Mon Sep 17 00:00:00 2001 From: jeffvli Date: Fri, 12 Dec 2025 21:25:46 -0800 Subject: [PATCH] fix context menu not triggering on sidebar playlist --- .../sidebar/components/sidebar-playlist-list.tsx | 13 +++++++++---- 1 file changed, 9 insertions(+), 4 deletions(-) diff --git a/src/renderer/features/sidebar/components/sidebar-playlist-list.tsx b/src/renderer/features/sidebar/components/sidebar-playlist-list.tsx index f98b6203a..07274a8f6 100644 --- a/src/renderer/features/sidebar/components/sidebar-playlist-list.tsx +++ b/src/renderer/features/sidebar/components/sidebar-playlist-list.tsx @@ -41,7 +41,7 @@ import { Play } from '/@/shared/types/types'; interface PlaylistRowButtonProps extends Omit { item: Playlist; name: string; - onContextMenu: (e: MouseEvent, item: Playlist) => void; + onContextMenu: (e: MouseEvent, item: Playlist) => void; to: string; } @@ -164,7 +164,10 @@ const PlaylistRowButton = memo(({ item, name, onContextMenu, to }: PlaylistRowBu [styles.rowDraggedOver]: isDraggedOver, [styles.rowHover]: isHovered, })} - onContextMenu={(e: unknown) => onContextMenu(e as MouseEvent, item)} + onContextMenu={(e: MouseEvent) => { + e.preventDefault(); + onContextMenu(e, item); + }} onMouseEnter={() => setIsHovered(true)} onMouseLeave={() => setIsHovered(false)} ref={ref} @@ -301,7 +304,8 @@ export const SidebarPlaylistList = () => { ); const handleContextMenu = useCallback( - (e: MouseEvent, playlist: Playlist) => { + (e: MouseEvent, playlist: Playlist) => { + e.preventDefault(); e.stopPropagation(); ContextMenuController.call({ cmd: { items: [playlist], type: LibraryItem.PLAYLIST }, @@ -416,7 +420,8 @@ export const SidebarSharedPlaylistList = () => { ); const handleContextMenu = useCallback( - (e: MouseEvent, playlist: Playlist) => { + (e: MouseEvent, playlist: Playlist) => { + e.preventDefault(); e.stopPropagation(); ContextMenuController.call({ cmd: {