deadbeef icon indicating copy to clipboard operation
deadbeef copied to clipboard

"Open file" replace entire playlist without warning

Open kiwec opened this issue 11 months ago • 4 comments

Using "Open file" with the intention to play a file (or add it to current playlist) nukes the current playlist without warning, even if it is unsaved.

There is no undo functionality so no way to recover the playlist.

Possible fixes:

  • Add a warning msgbox if the playlist is unsaved
  • Open the file in a new (temp?) playlist
  • Add undo functionality to playlist-altering actions

kiwec avatar Jul 31 '23 16:07 kiwec

I understand your frustration, but adding more warning dialogs in every menu action will only cause more frustration. Just remember to use Add files next time, or drag-drop into the playlist from a file manager, this would cover your needs.

Oleksiy-Yakovenko avatar Jul 31 '23 17:07 Oleksiy-Yakovenko

Maybe renaming the entry would be better. But the best thing would be, if it would just open in a new playlist.

Nama avatar Oct 12 '23 13:10 Nama

adding Undo functionality could be a lot of work, probably not the best solution.

Similar thing happened to me today with "load playlist" and I think the least damageable solution would be to change behaviour to create a new playlist ? That way if the intended action was to overwrite playlist, it's a simple operation to delete the old one. Current behaviour is destructive, has no warnings, no undo, and it's not intuitive that "Load" means "Load and overwrite". I am sure I will forget someday and do this again...

fenugrec avatar Feb 04 '24 15:02 fenugrec

adding Undo functionality could be a lot of work, probably not the best solution.

It was a lot of work indeed, but I just finished making something that works -- so now deadbeef has undo system. It's not in any stable release though, but it works in nightly builds.

Oleksiy-Yakovenko avatar Feb 04 '24 15:02 Oleksiy-Yakovenko