Skip to content

Commit dd67ab7

Browse files
authored
show confirmation when deleting a playlist (#308)
1 parent c029198 commit dd67ab7

File tree

1 file changed

+18
-2
lines changed

1 file changed

+18
-2
lines changed

BardMusicPlayer/UI_Classic/Classic_Playlist.cs

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -208,7 +208,7 @@ private void Playlist_Delete_Button_Click(object sender, RoutedEventArgs e)
208208
if (pls == null)
209209
return;
210210

211-
BmpCoffer.Instance.DeletePlaylist(pls);
211+
DeleteWithConfirmation(pls);
212212
PlaylistContainer.ItemsSource = BmpCoffer.Instance.GetPlaylistNames();
213213
return;
214214
}
@@ -217,11 +217,27 @@ private void Playlist_Delete_Button_Click(object sender, RoutedEventArgs e)
217217
return;
218218

219219
_showingPlaylists = true;
220-
BmpCoffer.Instance.DeletePlaylist(_currentPlaylist);
220+
DeleteWithConfirmation(_currentPlaylist);
221221
PlaylistContainer.ItemsSource = BmpCoffer.Instance.GetPlaylistNames();
222222
PlaylistHeader.Header = "Playlists";
223223
_currentPlaylist = null;
224224
}
225+
/// <summary>
226+
/// Delete playlist with showing the confirmation window
227+
/// </summary>
228+
/// <param name="playlist"></param>
229+
private void DeleteWithConfirmation(IPlaylist playlist)
230+
{
231+
MessageBoxResult confirmDelete = MessageBox.Show(
232+
$"Are you sure you want to delete this playlist?\n\nPlaylist name : {playlist.GetName()}",
233+
"Confirmation",
234+
MessageBoxButton.YesNo,
235+
MessageBoxImage.Question);
236+
if (confirmDelete == MessageBoxResult.Yes)
237+
{
238+
BmpCoffer.Instance.DeletePlaylist(playlist);
239+
}
240+
}
225241
#endregion
226242

227243
#region PlaylistContainer actions

0 commit comments

Comments
 (0)