Review Request 115857: Fix flickering of sliding popups while disappearing

Martin Gräßlin mgraesslin at kde.org
Tue Feb 18 06:37:05 UTC 2014


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


I'm sorry, but you are fixing symptoms and not the problem. Calling the repaint is correct, otherwise we might end with artefacts. The problem must be that another effect is still holding a reference to the window and thus it's being shown again. Try disabling the fade effect or any other closing animation effect you might be using.

- Martin Gräßlin


On Feb. 18, 2014, 1:52 a.m., Sebastian Kügler wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/115857/
> -----------------------------------------------------------
> 
> (Updated Feb. 18, 2014, 1:52 a.m.)
> 
> 
> Review request for kwin and Plasma.
> 
> 
> Repository: kde-workspace
> 
> 
> Description
> -------
> 
> Fix flickering of sliding popups while disappearing
> 
> The last frame was fully painted for Plasma 2 panel popups, as it's
> repainted in full on the last frame (without opacity and position
> transformations applied).
> 
> Removing the repaint on this last frame makes the popups disappear
> smoothly for me, without other side-effects such as artifacts on the
> screen or other repaint problems.
> 
> 
> Diffs
> -----
> 
>   kwin/effects/slidingpopups/slidingpopups.cpp fd697f0c62e9099d4be71e1f9d99e64ecb5ff663 
> 
> Diff: https://git.reviewboard.kde.org/r/115857/diff/
> 
> 
> Testing
> -------
> 
> Ran kwin with this on Intel driver, flickering is gone in Plasma 2 and Yakuake 4.x.
> 
> 
> Thanks,
> 
> Sebastian Kügler
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20140218/8d83b451/attachment.html>


More information about the Plasma-devel mailing list