Popup overflows desktop boundries on KDE Neon

jsalatas / plasma-pstate

Intel P-state and CPUFreq Manager Widget

GNU General Public License v2.0

274 stars 49 forks source link

Popup overflows desktop boundries on KDE Neon #1

Closed ghost closed 5 years ago

ghost commented 5 years ago

Operating System: KDE neon 5.14 KDE Plasma Version: 5.14.4 Qt Version: 5.11.2 KDE Frameworks Version: 5.53.0 Kernel Version: 4.15.0-43-generic OS Type: 64-bit Processors: 8 × Intel® Core™ i7-6700K CPU @ 4.00GHz Memory: 15.6 GiB of RAM

EDIT: Desktop resolution is 2560x1440 if it makes a difference

jsalatas commented 5 years ago

Fixed https://github.com/jsalatas/plasma-pstate/commit/1a357ca1a47e55258593ad96e24b1537da0e0fff Thanks!

ghost commented 5 years ago

I saw that commit come in before your reply and tested it and it didn't seem to help. I just tested a 2nd time again and still no luck

I removed the widget from my panel, used uninstall.sh, re-cloned, and reinstalled again. Is there something else I need to do to get this fix to kick in?

Verified my changes applied locally in /usr/share/plasma/plasmoids/gr.ictpro.jsalatas.plasma.pstate/contents/ui/main.qml .. and the issue still exists

jsalatas commented 5 years ago

Hmmm.... It should :\

jsalatas commented 5 years ago

I take it that you reinstalled AND restarted plasma. Right? :\

ghost commented 5 years ago

I didn't restart plasma but I'll do that now :+1:

ghost commented 5 years ago

That did the trick, thanks! I guess something gets cached in plasma

jsalatas commented 5 years ago

Yeap. It can be very annoying while developing. :)

Happy that it worked!