Review Request 116614: Reduce glitches when moving panels around

Commit Hook null at kde.org
Wed Mar 5 15:09:27 UTC 2014


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


This review has been submitted with commit 78c6e452dfc2bbbdec430cde84ee09bc1f4122ce by Aleix Pol to branch master.

- Commit Hook


On March 5, 2014, 3:09 p.m., Aleix Pol Gonzalez wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/116614/
> -----------------------------------------------------------
> 
> (Updated March 5, 2014, 3:09 p.m.)
> 
> 
> 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/610ac366/attachment.html>


More information about the Plasma-devel mailing list