Rockbox

  • Status Closed
  • Percent Complete
    100%
  • Task Type Patches
  • Category Playlists
  • Assigned To No-one
  • Operating System All players
  • Severity Low
  • Priority Very Low
  • Reported Version
  • Due in Version Undecided
  • Due Date Undecided
  • Votes
  • Private
Attached to Project: Rockbox
Opened by David Rothenberger - 2006-07-18
Last edited by Dominik Riebeling - 2011-06-04

FS#5677 - Remove current track from current playlist.

The attached patch adds a “Remove from current” option to the playlist menu. This option
- Removes the current track from the playlist.
- Saves the playlist unless the playlist is dynamic.
- Skips to the next track. (*)

* If the track is the last track in the playlist, Repeat is not set to All,

and the playlist was not saved because it is dynamic or an error occurred, 
the track is not skipped so it's easier to save the playlist with the change.

I’m not sure whether saving the playlist automatically is a good idea or not.
I like it because I often invoke this option from the remote while driving and
it’s difficult to save the playlist the normal way under those circumstances.
Also, I rarely remove tracks from the playlist. However, if you were planning to
remove many tracks, saving after each removal is a big waste of battery. Comments
on this aspect (or any aspect) of the patch would be appreciated.

Closed by  Dominik Riebeling
2011-06-04 21:29
Reason for closing:  Out of Date
Additional comments about closing:  

No activity since years. Plus, you can remove tracks from the playlists context menu these days.

David Rothenberger commented on 2006-07-18 14:33

Updated patch that applies to CVS HEAD after the playlist catalog was committed.

David Rothenberger commented on 2006-07-20 23:20

Updated for CVS HEAD. Also, verified it compiles for all targets and sim targets.

David Rothenberger commented on 2006-07-25 20:27

Updated for CVS HEAD. Verified it compiles for all targets and sim targets.

David Rothenberger commented on 2006-08-14 22:35

Updated for CVS HEAD. Verified it compiles for all targets and sim targets.

Kris Weissmann commented on 2007-03-13 17:55

Why has this patch never made it into builds / releases?
I miss this feature so much.
[How can the patch be implemented?]

David Rothenberger commented on 2007-05-19 17:35

Synced with Rockbox SVN r13402

   remove-from-playlist.patch (2.5 KiB)
Admin
Björn Stenberg commented on 2007-05-22 12:43

David, please upload your latest patch again. I accidentally removed the file in the flyspray transition. :-(

David Rothenberger commented on 2007-05-22 21:46

Here’s the latest version again. This was synced with r13402. I verified it compiles for all targets.

Loading...

Available keyboard shortcuts

Tasklist

Task Details

Task Editing