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.

Sign up for GitHub

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

Jump to bottom

[Bug Report] Lyrics desync #24

Open

CompeyDev opened this issue Mar 9, 2024 · 4 comments

Labels

bug

CompeyDev commented Mar 9, 2024 •

edited

Loading

Describe the bug

In certain cases, the lyrics view desyncs from the actual media content.

To Reproduce

Play a song.
Either let it run in background for a while before checking the lyrics, or jump to a lyric and the lyrics view will not update to the media.

Error reports & Screenshots

N.A.

Device info

Jetispot Version: 0.1.7
Android Version: 14
ROM: OxygenOS (OnePlus)

Additional context

N.A.

CompeyDev added the bug label

Author

CompeyDev commented Mar 9, 2024

Lmk if there are any logs or screen recordings I may need to provide.

Owner

BobbyESP commented Mar 9, 2024

Jetispot codebase is really messy and really hard to maintain. It really has a lot work to put on and I don't have the enough time that I would like to fix it, so I don't know if I will be fixing this.

The media player implementation is bad, but understandable due to how hard is to reverse engineer Spotify 😆, and the lyrics sync works on top of this.

So sorry.

Author

CompeyDev commented Mar 10, 2024

In that case, I think it would be wise to put a maintainence notice, looking for maintainers who are willing to undertake large refactors.

Outlet7493 commented Mar 14, 2024

Agreed. However a rewrite might be easier than large refactors... 😦

Happy to participate, but time is a constraint.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment