Skip to content

Commit

Permalink
detail: better fix no playlist nav
Browse files Browse the repository at this point in the history
Actually a problem with the dialog. Very annoying execution flow bug.
  • Loading branch information
OxygenCobalt committed Feb 21, 2025
1 parent 6589cd4 commit 9ae3587
Showing 1 changed file with 3 additions and 0 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,9 @@ class DeletePlaylistDialog : ViewBindingMaterialDialogFragment<DialogDeletePlayl
builder
.setTitle(R.string.lbl_confirm_delete_playlist)
.setPositiveButton(R.string.lbl_delete) { _, _ ->
// Normally the navigateUp will occur after this, which then collides with the
// playlist view's navigation. Forcefully navigate up to stop this.
findNavController().navigateUp()
// Now we can delete the playlist for-real this time.
musicModel.deletePlaylist(
unlikelyToBeNull(pickerModel.currentPlaylistToDelete.value), rude = true)
Expand Down

0 comments on commit 9ae3587

Please sign in to comment.