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

Unable to find Spotify window #6

Open
GitHubbaBubby opened this issue Mar 20, 2018 · 10 comments
Open

Unable to find Spotify window #6

GitHubbaBubby opened this issue Mar 20, 2018 · 10 comments

Comments

@GitHubbaBubby
Copy link

I get the error message in the subject when I type /spotify in Hexchat.

@requiemninja
Copy link
Contributor

That's due to Spotify not using the same handle. @rahatarmanahmed or @TingPing need to change it to grab title from process name. Process name is still Spotify.exe. I'd work on fixing this but the hassle of setting up VS for hexchat stuff again is out of my scope due to other constraints.

@TingPing
Copy link
Contributor

I'd work on fixing this but the hassle of setting up VS for hexchat stuff again is out of my scope due to other constraints.

It is building a single plugin, HexChat isn't involved.

@requiemninja
Copy link
Contributor

I see. Back before You fixed up some of the code, it required a bit of work to setup. It builds no problem now. I just don't have the c++ experience necessary to change it from going by handle.

@rahatarmanahmed
Copy link
Owner

I unfortunately no longer have access to a Windows machine to make and test this change with. Pull Requests are definitely welcome! (I didn't know much C when I published this either, I just copied from some forum and learned enough to make it work 🤫)

@TingPing
Copy link
Contributor

TingPing commented Mar 23, 2018

@Samanthajoy All you need to do is find out the new Window id (there is probably some tool for that) and change this line: https://github.com/rahatarmanahmed/hcspotify/blob/master/src/spotify.c#L16

@GitHubbaBubby
Copy link
Author

Thank you all for your replies, I will try and see if I can change the script.

@requiemninja
Copy link
Contributor

@TingPing Spotify is now an electron app, so it reports the same handle as any other electron app.
Chrome_RenderWidgetHostHWND
It needs to be switched to process name.

@TingPing
Copy link
Contributor

TingPing commented Apr 8, 2018

Bummer.

@frandominguezl
Copy link

Any progress on this?

@Oldiesmann
Copy link

Would love to use this. Sad that nobody has fixed it in 2 years

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants