View Issue Details

IDProjectCategoryView StatusLast Update
0009692MMAPlaylistspublic2022-09-08 20:54
Reporterrusty Assigned To 
PriorityurgentSeverityfeatureReproducibilityalways
Status closedResolutionfixed 
Target Version1.1.0Fixed in Version1.1.0 
Summary0009692: Improved delete confirmation dialog that allows for deletion of tracks from Playlists
DescriptionRemoval of tracks from Now Playing / Playlists works, but there's no mechanism for deleting such tracks. A confirmation dialog should be included for both usecases.

Note: this would also have the benefit of simplifying the confirmation dialogs (currently there are different dialogs depending on which node tracks are being deleted from)

Confirm deletion
x tracks, y videos have been selected.
(o) Remove from now playing
( ) Remove from now playing and delete from the device
[OK] [Cancel]

Confirm deletion
x tracks, y videos have been selected.
(o) Remove from the playlist
( ) Remove from the playlist and delete from the device
[OK] [Cancel]
Additional InformationCurrent strings (as of build 142) are:
    <string name="playlist_item_will_be_deleted">Item will be removed. No tracks will be deleted.</string>
    <string name="playlist_items_will_be_deleted">Items will be removed. No tracks will be deleted.</string>
    <string name="playlist_will_be_deleted">Playlist will be removed. No tracks will be deleted.</string>
    <string name="playlists_will_be_deleted">Playlists will be removed. No tracks will be deleted.</string>
    
TagsNo tags attached.
Fixed in build338

Relationships

related to 0008630 closedjiri MMA Items should be removable 
related to 0011683 closedmartin MMA Now Playing Art view context menu is missing some commands but has no room 
related to 0015978 closedmartin MMA Playlist edits fail to sync in some cases 
related to 0019363 newmartin MMA 'Remove from Now Playing and selected playlists': dialog is confusing and text gets truncated 
parent of 0012426 closedmartin MMA Delete from device confirmation text needs to be tweaked 
parent of 0013034 closedrusty MMA Delete/Remove from playlists options have been removed (regression?) 
related to 0010989 closedpeke MMA Implement sync confirmations for bi-di sync 
related to 0010990 closedmartin MMA Optimize/Simplify delete confirmation dialog 
related to 0011047 closedmartin MMA Refreshing of MiniPlayer, NowPlaying view and NowPlaying list when current playing track was deleted, is not working well. 
related to 0012390 closedmartin MMA Remove from Now Playing confirmation dialog is unneeded 
related to 0012391 new MMA Make 'Now Playing' aware of source of its content 
related to 0012427 resolvedrusty MMA Delete from Playlists / Now Playing should allow independent management of playlists 
related to 0014373 newmartin MMA Strange menus/options for deleting hierarchical playlists 

Activities

rusty

2013-01-11 05:18

administrator   ~0034381

Set to 1.0.1 as 'high'. I find this to be a significant omission in daily use.

rusty

2014-02-05 15:20

administrator   ~0039518

Last edited: 2014-09-19 20:27

Note: user suggested at http://www.mediamonkey.com/forum/viewtopic.php?f=21&t=75303 that there should also be a mechanism for deleting a track in NP from the playlist(s) containing it. Similar requeset at http://www.mediamonkey.com/forum/viewtopic.php?f=21&t=75814 .

This might be a completely separate feature, implemented similarly to MMW, but I'm noting it here because it might make sense to integrate this functionality.

e.g.
Confirm deletion
x tracks, y videos have been selected.
(o) Remove from now playing
( ) Remove from now playing and selected playlists
 [ ] Playlist 1
 [ ] Playlist 2
 [ ] Playlist 3
( ) Remove from now playing and delete from the device
[OK] [Cancel]

Note: for performance reason, it may be preferable to list Playlists only after the user has chosen to 'Remove from now playing and selected playlists...'

martin

2014-09-22 13:24

developer   ~0040528

Fixed in build 1.1.0.333

rusty

2014-11-11 22:56

administrator   ~0040964

Last edited: 2014-11-11 22:57

The text for the various options is truncated (for both Playlists and Now Playing)! Probably > 1 line should be used if the text doesn't fit in the dialog. I've set this to 'immediate' because it should be trivial to fix.

martin

2014-11-12 11:58

developer   ~0040971

Fixed in build 338.

rusty

2014-11-12 20:51

administrator   ~0040992

Last edited: 2014-11-28 16:05

Tested 338. Remove from Now Playing/Playlist and Remove from Now Playing/Playlist and delete from the device are both working as expected.

There are 2 issues that I'll track in new bugs:
1) Some users find the extra step of a confirmation dialog annoying
2) The 'Remove from Now Playing and selected playlists' isn't a great design

rusty

2014-11-28 16:06

administrator   ~0041208

Verified 1.1.0.343