diff options
author | Christian Soffke <christian.soffke@gmail.com> | 2021-02-15 13:55:30 +0100 |
---|---|---|
committer | Solomon Peachy <pizza@shaftnet.org> | 2021-02-27 14:38:29 +0000 |
commit | 317908e8c31ad07d1cfeaf0e97d9e95bc9fcc99d (patch) | |
tree | 56fb38fe645614b979083dd229245040d42a1686 /apps | |
parent | 05f10062461adb658a885c1b034e5c4519c19bc4 (diff) | |
download | rockbox-317908e8c31ad07d1cfeaf0e97d9e95bc9fcc99d.tar.gz rockbox-317908e8c31ad07d1cfeaf0e97d9e95bc9fcc99d.zip |
Warn before permanently reshuffling the current playlist
Reshuffling a playlist using the WPS context menu for the current playlist is a destructive operation since the previous order of items can't be restored. A warning should be displayed, so that the user can still cancel the operation in case they intended to select another menu item (e.g. one of the harmless "Playing Time" or "Save Current Playlist" items right next to it).
Change-Id: Ib9fc379439d155be907872d77cf54d5035bd08a0
Diffstat (limited to 'apps')
-rw-r--r-- | apps/onplay.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/apps/onplay.c b/apps/onplay.c index 03470a279f..e7f7e589db 100644 --- a/apps/onplay.c +++ b/apps/onplay.c | |||
@@ -446,6 +446,8 @@ static bool playing_time(void) | |||
446 | /* CONTEXT_WPS playlist options */ | 446 | /* CONTEXT_WPS playlist options */ |
447 | static bool shuffle_playlist(void) | 447 | static bool shuffle_playlist(void) |
448 | { | 448 | { |
449 | if (!warn_on_pl_erase()) | ||
450 | return false; | ||
449 | playlist_sort(NULL, true); | 451 | playlist_sort(NULL, true); |
450 | playlist_randomise(NULL, current_tick, true); | 452 | playlist_randomise(NULL, current_tick, true); |
451 | 453 | ||