Allow wrapping to last track with Previous button. Closes: #1726#1794
Conversation
…media-player#1726 When at the first track and "Previous" is pressed, jump to the last track in the playlist if "Repeat Playlist" is active. This makes the behavior more intuitive and consistent with the "Next" button.
radioactiveman
force-pushed
the
inverse-playlist-repeat
branch
from
f8c1e84 to
bcf29d2
Compare
|
Testing with "Repeat" and "Shuffle" enabled together I noticed that the behavior of "Previous" and "Next" is still different. |
|
LGTM.
Without shuffle enabled, I think it's reasonable for "repeat backward" to wrap around to the end of the playlist, as you have here. I don't think it's clear what "repeat backward" would do (i.e. where it would wrap to) with shuffle enabled. I think the most conservative choice is for it to do nothing; i.e. not wrap at all. |
2 participants
When at the first track and "Previous" is pressed, jump to the last track in the playlist if "Repeat Playlist" is active.
This makes the behavior more intuitive and consistent with the "Next" button.