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