-
Notifications
You must be signed in to change notification settings - Fork 647
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
pause after last track has finished #81
Comments
Comment by markubiak +1 would really like to see this. @herrernst any way you could include this PR in your raspberry pi builds? |
Comment by awiouy What about adding this as a runtime option? |
Comment by markubiak I don't think it should be included as a runtime option. This is a good stopgap measure until someone gets the repeat button working. |
Comment by markubiak I would agree that that is the best possible option. Unfortunately, I do not have any experience with rust and it doesn't seem to share a lot of similarities with languages I am familiar with. If you have the time and energy to implement that, it would be fantastic. It does seem that the repeat status is received in protocol/src/spirc.rs, and the original file (src/spirc.rs) could simply have the logic self.load_track(!was_last_track || is_repeating); |
Comment by kingosticks So I had a go at integrating this with the connect side of things, my first stab at rust: plietar/librespot@master...kingosticks:master However, it's not quite there yet as it's missing the ability to repeat just one song which is available on the native client (you toggle repeat, repeat one, and off through the same control on the Android app). The repeat state is just a boolean so maybe there's a different bit of state to control repeat one (single?). Any ideas welcome. I also wanted to do shuffle but doing the actual track shuffle is a bit more work. |
I think this was fixed due to #2. I've just tested it and it stopped at the end of a playlist. Could someone else confirm the same? |
This was fixed, however just looking over that PR, we need to open a new issue for Shuffle and Repeat functionality, which isn't fully implemented. |
@sashahilton00 I think #19 covers this. |
@ComlOnline agreed. |
Issue by herrernst
Saturday Aug 26, 2017 at 17:20 GMT
Originally opened as plietar/librespot#239
Currently, librespot plays endlessly: After the last track in a playlist/album it continues playing the first one.
This changes the behaviour to match the official Spotify client: The first track is loaded again, but the player is paused.
See #237
herrernst included the following code: https://github.com/plietar/librespot/pull/239/commits
The text was updated successfully, but these errors were encountered: