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

Spytify does not detect title changes when spotify is hidden #398

Closed
supsm opened this issue Oct 27, 2021 · 1 comment
Closed

Spytify does not detect title changes when spotify is hidden #398

supsm opened this issue Oct 27, 2021 · 1 comment

Comments

@supsm
Copy link

supsm commented Oct 27, 2021

What is the bug ?

A clear and concise description of what the bug is.

When spotify is minimized to tray, spytify does not detect that the track has changed or an advertisement has started/ended.

Steps to reproduce the behavior or describe the expected behavior:

Start spying
Play something
Minimize spotify to tray
When the song is finished playing, spytify will not recognize that and will continue recording

I know this definitely isn't a windows API issue, as hidden windows can still change their title text and other applications can detect it. Currently I'm too lazy to dig into source code, so I will take a bit of a guess. Maybe the HWND can be cached at the start of spying, then it will be used to get the title text. Spotify should not create any new windows, so the HWND will be valid until spotify is closed. I'm think what's currently happening is that all open windows are being checked for the spotify window every time spytify wants to update. If this is the case, caching the HWND will also improve performance and potentially allow spytify to update more frequently.

@supsm supsm added the bug 🐞 label Oct 27, 2021
@jwallet
Copy link
Owner

jwallet commented Apr 4, 2022

that's a known limitation, Spotify does not provide changes when minimized. I already tried to get the current track info from SpotifyAPI but users told me that the app was requesting too many times (60k requests a day) so I removed it.

@jwallet jwallet closed this as completed Apr 4, 2022
@jwallet jwallet added invalid 🕳 Issue out of scope won't do 🗑 and removed invalid 🕳 Issue out of scope labels Apr 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants