MinhThienDX / win10widgets

A fork from https://github.com/tjmarkham/win10widgets

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

"Error: Spotify Plugin not working" instead of artist/song title

Shushuda opened this issue · comments

Hi, I'm having a problem with setting up the Spotify widget and cannot seem to fix it. Perhaps I am doing something wrong - I'd rather ask here in case it's a real bug and not my fault.

I've installed Spicetify (to get the .dll) and added manually WebNowPlaying .dll file to the plugin dir. I've loaded the Spotify widget (smallest version) and it's correctly controlling the play/pause, next and previous buttons (I can use those fine to control Spotify), but their images are not changing (not sure if the play should change to a pause image when the song is playing) and the artist and song title is nowhere to be seen - instead, I get "Error: Spotify Plugin not working" in their place. Timestamps and covers are also not displayed at all.

sadasdasda

This means that the skin has no connection to Spotify via WebNowPlaying. Have you successfully installed spicetify using spicetify-cli?
Also, why didn't you install WebNowPlaying using their .rmskin?

@InterFelix I see your PR at tjmarkham/win10widgets, but that repo is abandoned.
If you make a PR here, I will merge it.

Will do once I am back at my desk. Probably in an hour or so. I'm very interested in bringing this project forward, but I'm new to rainmeter.

@InterFelix No rush, this is an open source project, you can participate at your own pace whenever you want.
Don't worry about being new, there is always a first time for everything 😉

Okay. I'm trying to integrate my changes into the existing, tweaked spotify widget, but I now have a similar problem to the one @Shushuda has. No cover displayed, no Track/Artist-names, controls work, but play/pause changes and also the progress bar works. However, also no Duration and progress in minutes.
So I undid all my changes (fetch origin, so all files are fresh from the repo) and the problem persisted.
I think I'm going to replace all your measures with mine, so it can work more reliably. I basically adapted them from another Skin calles SpotifyControl by fuckyeahlucas, which works all the time (I have it open for comparison).

My PR #29 should fix this issue. We should add a release for this, so @Shushuda can easily download the new version.

@InterFelix Done.
@Shushuda You can check the plugin and then close this issue.

Thanks a lot. However, I think we need a new release, since the last one is from 2016. @MinhThienDX , can you create a new one? I have already built an .rmskin file, but you can do so yourself if you prefer.
This would make it much easier for users to use our updated skins.

Actually I am against the release of rmskin file.

For a few reason below:

  1. It take time and effort for insignificant gain since rmskin file is just a modified zip file
  2. To update a skin, git pull is much faster and easier than using rmskin file (even download as zip from GitHub is better as the author don't have to create zip file)
  3. Rainmeter users probably are tech-savvy enough to get by without rmskin

All that time and effort can be used to actually improve the skin instead 😉
And don't worry, we don't need a new release because no one have never asked about rmskin file.
You are the first to mention it 😂
Think of this repo is the latest release without builtin auto update but the users can easily "add" auto update "feature" with git 😉

Okay, fair enough. But, you know, it's very easy to do using the built-in feature in Rainmeter.
However, your choice, so do what you think is right. I of course use git for this, but the majority of skins is released in .rmskin form, because everyone can install an .rmskin file, while maybe not everyone can git pull. But whatever, fair point. Most people that care enought about their desktop to use Rainmeter can wrap their head around basic git 😉

I deleted the v1.0.0 tag, it is not even a rmskin file 😂

Yes, that's right. Maybe best to do so

However, I don't really know how I should efficiently use git for updating. My skins folder has more than just win10widgets in it, of course. And if I clone into a subdirectory, the widgets break for whatever reason...

Untitled
You can make it like so

  1. Clone the repo some where
  2. Move .git folder (hidden) and everything else to Documents/Rainmeter/Skins

That's it.

Yeah, okay. Since I can't directly commit I don't need .gitignore anyways

Just use your fork and you can direct commit or do whatever your heart desire 😉
You can put secret/personal stuff in another branch and public stuff in master branch.
That way you can easily create PR.

Yeah, of course. I'll just look around and fix whatever I come across (if I manage to do so)

Thank you in advance.
However, I don't ask/expect you to fix all the issue.
Just fix what affect you and give back to the community if you want.
Give and take, don't use too much of your personal time to give stuff for others to use.
Take care of yourself first.

Hi, sorry for the long absence. I've downloaded the newer version after the fix got merged and the Spotify widget is working perfectly fine now. Thank you for your help! I will close the issue now.

Do you see covers? After the latest Spotify update there are no covers display anymore (for me at least)