Feature Request: Thumbs Being Visible

[livejamie]

I love everything about GPMDP but one of the larger annoyances is a lack of thumbs up/thumbs down in the miniplayer.

Google moves the shuffle and repeat icons outside of the bar in their version.

For reference

GPMDP:

Google's Official Chrome App:

GPMDP does show thumbs when listening to the radio.

We discussed some options here to toggle this view by default potentially.

[bpclarke]

I've just discovered GPMDP and love the idea of having a separate program for GPM and the design but use the thumbs up/ thumbs down feature in the mini player way too much to use the program (have a lot of music I'm gradually sifting through). Would love it if this feature was implemented!

In my opinion the the best way of doing this would be to replace repeat / shuffle buttons with thumbs down / thumbs up buttons in the mini player. I very rarely use them and if I do it's once before I start playing music. It appears that this has already been done for when a radio station is being played although I don't have a subscription so can't check.

Obviously not everyone wants this feature so I guess the best way to implement this would be an option to choose shuffle / repeat or thumbs up / thumbs down in this menu:

I have very little programming experience (have only used MATLAB on my engineering university course) but will look into it over the summer if nothing's been done by then! I've been meaning to teach myself to code anyway :) Might need a reminder though, feel free to send me one in June.

[varzaman]

So I started working on this, I should have something soon.
Having the thumbs show up was non trivial, just some css changes.
I'm working on making showing thumbs a toggle in the settings menu.

[bpclarke]

Awesome, made any progress since your post?

[varzaman]

Sorry, got busy and forgot about this.

Before I forgot though, I did somewhat implement it. The issue I have now is it doesn't dynamically change. The application needs to be restarted to have the thumbs and shuffle/repeat buttons switched.

Once I figure that out I'll request a pull request.

[bpclarke]

That's great! Don't you have to restart the application to change some other settings anyway? I'd be pretty happy with that implementation. Just put "(requires application restart)" after the checkbox and description?

[varzaman]

Yea for some things it does require restart, I guess I could do that indeed.

Let me just clean up the code, and merge in the latest changes.

[bpclarke]

Nice one!

[MarshallOfSound]

@bpclarke @Lettuceman44 Requiring a user restart is only on ONE setting and that is a core OS level API limitation. A user should not have to restart the player to shows some thumbs 😢

I've left a comment explaining how to do this without restarting

#8 (comment)

[varzaman]

Understood, will get that changed.

[varzaman]

Just as a quick update as I've been busy, I have it so it switches from the shuffle/repeat to thumbs dynamically without restarting, that wasn't hard at all, just copy paste :P. It doesn't work vice versa though, and I haven't figured it out yet.

Unfortunately for me, I haven't had any internet this week, waiting for that to get fixed as well.