New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix shuffle history #4255
Fix shuffle history #4255
Conversation
Side note: I guess we do need a strong "songlike type" than Any. |
(I think the Windows job is failing because of a different version of anyway it just wants an annotation on |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
ed7aa31
to
1dd6703
Compare
Instead of replacing the whole list reorder the existing one. This allows play order implementations to continue to use any saved TreeIters. Fixes quodlibet#4108
This reverts commit 68ba678.
1dd6703
to
8448d31
Compare
Thanks @slosd ! |
This reverts commit 500d0d1.
Revert "Fix shuffle history (#4255)"
Check-list
main
What this change is adding / fixing
Reset play order in queue for new set of songs
Used a specialized
QueueModel
because the testtest_random_queue_666
requires behavior that's slightly different fromPlaylistModel
.Preserve shuffle history when songlist is reordered
Reorder songlist in place
Instead of replacing the whole list reorder the existing one. This
allows play order implementations to continue to use any saved
TreeIters.
Fixes #4108
Revert "Ignore auto_sort when shuffle is enabled (#3581)"
This reverts commit 68ba678. This limitation shouldn't be necessary anymore.