From 2fa6c9de94501054e1cf797f491b394055ad83e3 Mon Sep 17 00:00:00 2001
From: jeffvli <jeffvictorli@gmail.com>
Date: Sat, 29 Nov 2025 13:24:58 -0800
Subject: [PATCH] support playlist sidebar drag to queue

---
 .../sidebar/components/sidebar-item.tsx       |  7 +++
 .../components/sidebar-playlist-list.tsx      | 57 +++++++++++++++++--
 2 files changed, 60 insertions(+), 4 deletions(-)

diff --git a/src/renderer/features/sidebar/components/sidebar-item.tsx b/src/renderer/features/sidebar/components/sidebar-item.tsx
index 36fb09003..fb5c68d70 100644
--- a/src/renderer/features/sidebar/components/sidebar-item.tsx
+++ b/src/renderer/features/sidebar/components/sidebar-item.tsx
@@ -15,6 +15,11 @@ export const SidebarItem = ({ children, className, to, ...props }: SidebarItemPr
     const toPath = typeof to === 'string' ? to : to.pathname || '';
     const isActive = location.pathname === toPath;
 
+    const handleLinkDragStart = (e: React.DragEvent<HTMLButtonElement>) => {
+        e.preventDefault();
+        e.stopPropagation();
+    };
+
     return (
         <Button
             className={clsx(
@@ -31,6 +36,8 @@ export const SidebarItem = ({ children, className, to, ...props }: SidebarItemPr
                 label: styles.label,
             }}
             component={Link}
+            draggable={false}
+            onDragStart={handleLinkDragStart}
             to={to}
             variant="subtle"
             {...props}
diff --git a/src/renderer/features/sidebar/components/sidebar-playlist-list.tsx b/src/renderer/features/sidebar/components/sidebar-playlist-list.tsx
index a02c2c713..8defb85d7 100644
--- a/src/renderer/features/sidebar/components/sidebar-playlist-list.tsx
+++ b/src/renderer/features/sidebar/components/sidebar-playlist-list.tsx
@@ -7,6 +7,8 @@ import { generatePath, Link } from 'react-router';
 
 import styles from './sidebar-playlist-list.module.css';
 
+import { getDraggedItems } from '/@/renderer/components/item-list/helpers/get-dragged-items';
+import { ContextMenuController } from '/@/renderer/features/context-menu/context-menu-controller';
 import { usePlayer } from '/@/renderer/features/player/context/player-context';
 import { playlistsQueries } from '/@/renderer/features/playlists/api/playlists-api';
 import { CreatePlaylistForm } from '/@/renderer/features/playlists/components/create-playlist-form';
@@ -30,19 +32,34 @@ import {
 import { DragOperation, DragTarget } from '/@/shared/types/drag-and-drop';
 import { Play } from '/@/shared/types/types';
 
-interface PlaylistRowButtonProps extends Omit<ButtonProps, 'onPlay'> {
+interface PlaylistRowButtonProps extends Omit<ButtonProps, 'onContextMenu' | 'onPlay'> {
+    item: Playlist;
     name: string;
+    onContextMenu: (e: MouseEvent<HTMLButtonElement>, item: Playlist) => void;
     onPlay: (id: string, playType: Play) => void;
     to: string;
 }
 
-const PlaylistRowButton = ({ name, onPlay, to, ...props }: PlaylistRowButtonProps) => {
+const PlaylistRowButton = ({ item, name, onContextMenu, onPlay, to }: PlaylistRowButtonProps) => {
     const url = generatePath(AppRoute.PLAYLISTS_DETAIL_SONGS, { playlistId: to });
     const { t } = useTranslation();
 
     const [isHovered, setIsHovered] = useState(false);
 
-    const { isDraggedOver, ref } = useDragDrop<HTMLDivElement>({
+    const { isDraggedOver, isDragging, ref } = useDragDrop<HTMLDivElement>({
+        drag: {
+            getId: () => {
+                const draggedItems = getDraggedItems(item, undefined);
+                return draggedItems.map((draggedItem) => draggedItem.id);
+            },
+            getItem: () => {
+                const draggedItems = getDraggedItems(item, undefined);
+                return draggedItems;
+            },
+            itemType: LibraryItem.PLAYLIST,
+            operation: [DragOperation.ADD],
+            target: DragTarget.PLAYLIST,
+        },
         drop: {
             canDrop: (args) => {
                 return (
@@ -127,14 +144,17 @@ const PlaylistRowButton = ({ name, onPlay, to, ...props }: PlaylistRowButtonProp
             onMouseEnter={() => setIsHovered(true)}
             onMouseLeave={() => setIsHovered(false)}
             ref={ref}
+            style={{
+                opacity: isDragging ? 0.5 : 1,
+            }}
         >
             <SidebarItem
                 className={clsx({
                     [styles.rowHover]: isHovered,
                 })}
+                onContextMenu={(e) => onContextMenu(e, item)}
                 to={url}
                 variant="subtle"
-                {...props}
             >
                 {name}
             </SidebarItem>
@@ -241,6 +261,17 @@ export const SidebarPlaylistList = () => {
         [player, server.id],
     );
 
+    const handleContextMenu = useCallback(
+        (e: MouseEvent<HTMLButtonElement>, playlist: Playlist) => {
+            e.stopPropagation();
+            ContextMenuController.call({
+                cmd: { items: [playlist], type: LibraryItem.PLAYLIST },
+                event: e,
+            });
+        },
+        [],
+    );
+
     const memoizedItemData = useMemo(() => {
         const base = { handlePlay: handlePlayPlaylist };
 
@@ -317,8 +348,10 @@ export const SidebarPlaylistList = () => {
             <Accordion.Panel>
                 {memoizedItemData?.items?.map((item, index) => (
                     <PlaylistRowButton
+                        item={item}
                         key={index}
                         name={item.name}
+                        onContextMenu={handleContextMenu}
                         onPlay={handlePlayPlaylist}
                         to={item.id}
                     />
@@ -352,6 +385,20 @@ export const SidebarSharedPlaylistList = () => {
         [player, server.id],
     );
 
+    const handleContextMenu = useCallback(
+        (e: MouseEvent<HTMLButtonElement>, playlist: Playlist) => {
+            e.stopPropagation();
+            ContextMenuController.call({
+                cmd: {
+                    items: [playlist],
+                    type: LibraryItem.PLAYLIST,
+                },
+                event: e,
+            });
+        },
+        [],
+    );
+
     const memoizedItemData = useMemo(() => {
         const base = { handlePlay: handlePlayPlaylist };
 
@@ -386,8 +433,10 @@ export const SidebarSharedPlaylistList = () => {
             <Accordion.Panel>
                 {memoizedItemData?.items?.map((item, index) => (
                     <PlaylistRowButton
+                        item={item}
                         key={index}
                         name={item.name}
+                        onContextMenu={handleContextMenu}
                         onPlay={handlePlayPlaylist}
                         to={item.id}
                     />