-
Notifications
You must be signed in to change notification settings - Fork 881
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
[Feature Request]: Synced Lyrics: Manual lyrics refresh/fetch #3161
Comments
Does it take more than 2-3 seconds? |
It definitely takes longer than 2-3 seconds. I haven't kept track, but it feels like it can take up to a minute or so. I'm pretty sure at least one time even going back/forward songs in the queue didn't refresh it either. |
I suspect the UI doesn't update whilst it is searching for the lyrics?
|
I haven't been able to intentionally reproduce it, but it does happen relatively often, seemingly by random. Whenever it does happen, and it finally refreshes the lyrics, it swaps to the new ones on the fly. I doubt it would take that long to search for the lyrics, when usually it only takes a second, but I also don't know about the inner workings if there's an error or timeout with the request, or if there are any rate limits etc. |
Well I'll be damned, I'll have to resort to speculative fixes then 💀 |
ig this can wait for after #3162 ? |
Yeah, it's not a high priority feature/fix, just mildly annoying whenever it does happen. |
Preflight Checklist
Problem Description
Sometimes when playing or skipping a song (can't remember if it occurs otherwise) the synced lyrics will be that of the previous song, without refreshing to the new ones. The lyrics update after some time, but it can take a while.
Proposed Solution
Would it be possible to get at least an optional manual refresh button for when this occurs.
Alternatives Considered
Investigating and fixing the underlying issue would of course be ideal, but a simple manual refresh button would suffice for now.
Additional Information
This has been an issue ever since multiple lyrics providers became a feature, and is not new.
The text was updated successfully, but these errors were encountered: