drag and drop of the tracks beneath a custom playlist

Bug #249286 reported by AnthonyWaters
2
Affects Status Importance Assigned to Milestone
Exaile
Won't Fix
Low
AnthonyWaters
Nominated for 0.3.x by AnthonyWaters

Bug Description

With the tracks showing beneath the custom playlist now it opens up some new doors for adding new drag/drop functionality (and also breaking some of the old stuff like dragging a track to the playlist tab throws an exception because it expects a playlist to be dragged not a track)

What I was thinking of adding for functionality was as follows

1. Drag the tracks to the playlist tab
2. Drag the tracks within the current playlist (for ordering them)
3. Drag tracks into other playlist (not sure if this one will be done, maybe hard to use for the user)

AnthonyWaters (awaters1)
Changed in exaile:
assignee: nobody → awaters1
Revision history for this message
reacocard (reacocard) wrote :

this is actually not as hard as you'd think. all you have to do is make sure all track dragging uses the same method (currently sends a list of locs), and then just put a tiny bit of logic in each potential destination to handle the insertion/deletion. it would be nice if we could get copy/paste also working via a similar manner.

a few additional notes on behavior:

if tracks are dragged onto an existing playlist, insert them at that point (if on the title append them)

if tracks are dragged into empty space, make a new playlist for them and ask the user for the name to give the playlist (hitting cancel in the naming dialog will cause the playlist to not be made)

if one playlist is dragged onto another, offer to merge them. insertion location works just as if all the contained tracks had been dragged instead.

AnthonyWaters (awaters1)
Changed in exaile:
status: New → In Progress
Revision history for this message
AnthonyWaters (awaters1) wrote :

The only tricky part is setting up the drag and drop handlers. For example dragging a track over tracks that are within the same playlist would be a move operation but dragging that same track over another playlist would be a copy operation, lastly tragging that track onto a Smart Playlist would be an invalid operation.

Revision history for this message
AnthonyWaters (awaters1) wrote :

this patch adds in the ability to drag/drop the tracks beneath the playlist

Revision history for this message
AnthonyWaters (awaters1) wrote :

updated patch to latest bzr and applied fixes

Revision history for this message
AnthonyWaters (awaters1) wrote :

there are some bugs when dropping the playlist themselves onto the panel from which they were dragged from. But the way I think I am going to do it is if the user is dragging a playlist within the playlist panel the following actions could occur

-dragging onto another playlist (does ordering of the playlist, which means the playlist manager needs to support the ordering of playlist)
-dragging onto tracks of another playlist copies the tracks of the original playlist

Revision history for this message
AnthonyWaters (awaters1) wrote :

latest patch includes the ability to reorder playlists (and saves their order for future loads) it also includes the contents of the previous patches

Revision history for this message
reacocard (reacocard) wrote :

Patch doesn't apply cleanly to latest bzr, please update it.

Revision history for this message
Steve Dodier-Lazaro (sidi) wrote :

Hi, whats the status on this bug ? Anthony, do you have the time to update your patch to Exaile 0.3.x ?

Changed in exaile:
importance: Undecided → Low
milestone: none → 0.3.1
Revision history for this message
reacocard (reacocard) wrote :

closing bug due to lack of activity on this feature.

Changed in exaile:
status: In Progress → Won't Fix
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.