Review Request 116614: Reduce glitches when moving panels around

Aleix Pol Gonzalez aleixpol at kde.org
Wed Mar 5 13:36:24 UTC 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/116614/
-----------------------------------------------------------

Review request for Plasma.


Repository: plasma-framework


Description
-------

There are 2 (quite separate) changes:
* Never use the window size to figure out the thickness: This is bad because sometimes this will get called before the move hasn't been effective, which results in full-screen struts. Use the thickness method from KConfig instead.
* Don't call setPosition and resize, but do it all at once in a setGeometry call. It simplifies the life of the QPA and WM, also it's quite faster as we reduce the calls to the WM by half (both setPosition and resize are syntax-sugar for setGeometry).


Diffs
-----

  src/shell/panelview.cpp a09a61a 

Diff: https://git.reviewboard.kde.org/r/116614/diff/


Testing
-------

Played with it, KWin didn't decide to throw my windows to the other screen.


Thanks,

Aleix Pol Gonzalez

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140305/890ae80f/attachment.html>


More information about the Plasma-devel mailing list