search

jsalatas / plasma-pstate

Intel P-state and CPUFreq Manager Widget
GNU General Public License v2.0
274 stars 49 forks source link

Possible to add to System Tray? #7

Open trgeiger opened 5 years ago

trgeiger commented 5 years ago

Is there any possibility that this could live inside the KDE System Tray, much like how the Redshift widget can be enabled there?

jsalatas commented 5 years ago

This was my original intention. The "problem" with the tray is that the widget cannot expand to its own size, but the tray imposes some kind of fixed size for it: Try for example clicking on several plasma widgets on your system tray and you will see that all of these have the same fixed size.

So I guess I should either introduce a scrollview, which looks really ugly or somehow redesign the whole user interface to fit in that size :\

trgeiger commented 5 years ago

Ah yeah I totally forgot about that limitation. I mean, it would be a great option eventually, but I know redesigning the interface for a small space like that would be a lot of work.

A few of the standard parts of the System Tray have scrollable parts, so maybe if some of the interface was smaller it wouldn't be so bad to scroll? Not sure.

Also, separately, I just thought it might be nice if the icon used for the widget could be changed manually by the user?

jsalatas commented 5 years ago

A few of the standard parts of the System Tray have scrollable parts, so maybe if some of the interface was smaller it wouldn't be so bad to scroll? Not sure.

Yeah! Maybe a less fancy interface, eg dropdown boxes instead of big fancy buttons with icons. I'll give it a try... :)

I just thought it might be nice if the icon used for the widget could be changed manually by the user?

Sure!

jsalatas commented 5 years ago

Could you please test the latest code? You should be able to change the icon.