Widget keeps re-positioning itself in the top left corner of the desktop on every login.

jsalatas / plasma-pstate

Intel P-state and CPUFreq Manager Widget

GNU General Public License v2.0

274 stars 49 forks source link

Widget keeps re-positioning itself in the top left corner of the desktop on every login. #5

Closed arslancharyev31 closed 5 years ago

arslancharyev31 commented 5 years ago

I have installed the latest version of the widget according to the instructions and it works as expected. However every time I login and start my session, I find it positioned at the top left corner of my desktop. Any suggestions on how I can debug its behaviour in order to identify what causes such behaviour? Relevant system info:

Operating System: Manjaro Linux 
KDE Plasma Version: 5.14.4
Qt Version: 5.12.0
KDE Frameworks Version: 5.53.0
Kernel Version: 4.19.12-2-MANJARO

P.S. I have not added user to the sudoers group as has been stated in the instructions, since the widget works for me without doing so.

jsalatas commented 5 years ago

However every time I login and start my session, I find it positioned at the top left corner of my desktop.

I'm not sure what you mean here. Could you please provide a screenshot?

arslancharyev31 commented 5 years ago

Could you please provide a screenshot?

Sure:

Before restart:

![beforerestart](https://user-images.githubusercontent.com/20893266/50661439-d5a9a580-0fab-11e9-956d-9425586a5d06.jpeg)

After Restart:

![afterrestart](https://user-images.githubusercontent.com/20893266/50661455-e22dfe00-0fab-11e9-826b-08523a08d287.jpeg)

jsalatas commented 5 years ago

Could you please try the latest commit?

arslancharyev31 commented 5 years ago

I did try it and appears to solver the problem, thank you.

P.S. It would be nice if the installer would auto-remove old version, since one has to manually remove /usr/share/plasma/plasmoids/gr.ictpro.jsalatas.plasma.pstate folder before attempting to execute install.sh

P.S.S. I've noticed that widget's vertical dimension has increased a little bit, as compared to the previous version, but it's not that crucial.

New version

![screenshot_20190106_190726](https://user-images.githubusercontent.com/20893266/50739283-80a5a380-11e6-11e9-817c-70486522de20.jpeg)

jsalatas commented 5 years ago

It would be nice if the installer would auto-remove old version

My bad! Sorry! I assumed that somehow you should know that after installation you don't need to run the install script again, you just need to copy the gr.ictpro.jsalatas.plasma.pstate to /usr/share/plasma/plasmoids :)

I've noticed that widget's vertical dimension has increased a little bit, as compared to the previous version.

That was actually half of the solution: previously I tried to force it's size to fit the content, ignoring plasma's grid, but when you put it on the desktop and it doesn't follow the grid size, strange things can happen. So know I'm just hinting it for the minimum size and it apparently expands a little in order to snap to the grid.

Thanks for your time and help! I really appreciate it.