Skip to content
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

Starting a playlist from a specific song only adds the remainder of the playlist to the queue (Web UI) #703

Closed
danshilm opened this issue Dec 26, 2020 · 8 comments

Comments

@danshilm
Copy link
Contributor

The expected behaviour was that when I start playing a playlist from a specific song, the whole playlist is added to the queue and the specific song starts playing. That way, I can still go the previous song in the playlist since it is all in the queue.

However, in the Web UI, only the songs after the specific song are added to the queue.
This is not the case in Music Stash nor DSub.

@deluan
Copy link
Member

deluan commented Dec 30, 2020

Sorry for the delay. Yes this was necessary when I first implemented the play queue management, as the upstream react-player project didn't have better ways to manipulate the queue. It changed in recent versions, so I may take another look and see if this can be improved.

@danshilm
Copy link
Contributor Author

danshilm commented Jan 1, 2021

Sure, you want me to keep this open until you see if its possible now?

@deluan
Copy link
Member

deluan commented Jan 2, 2021

Yes, let's keep this open. I'll try to take a look at this this week

@deluan
Copy link
Member

deluan commented Jul 3, 2021

This should be fixed in ace5c90. Please test the latest build and let me know.

@danshilm
Copy link
Contributor Author

danshilm commented Jul 3, 2021

Works as desired in ace5c90. Thanks a lot 👍🏼 This is one of those things that I do a lot that works the same way in all the other players I used which breaks Jakob's Law like you mentioned.

@deluan
Copy link
Member

deluan commented Jul 3, 2021

Thanks @YashJipkate for your previous PR #976, you were right: The "Play Next" behaviour is already broken in master, it is a bug in the React Player. I'll open an issue with that project and will try to fix it this weekend.

@deluan deluan closed this as completed Jul 3, 2021
@YashJipkate
Copy link
Contributor

There is an issue already opened by me, you might want to take a look at lijinke666/react-music-player#349.

@github-actions
Copy link

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants