Review Request 115859: Force-allow background contrast while sliding desktops

Sebastian Kügler sebas at kde.org
Tue Feb 18 13:58:56 UTC 2014


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

(Updated Feb. 18, 2014, 1:58 p.m.)


Review request for kwin and Plasma.


Changes
-------

Next try ...

- only touch slide effect
- only force flag on relevant windows
- remember which windows we've force the flag on
- reset the flag for the windows we've set it on


Repository: kde-workspace


Description
-------

Force-allow background contrast while sliding

When the sliding effect is active, the background blur of the panel
would be disabled unless forced with WindowForceBackgroundContrastRole,
which is actually what we want: during sliding, the backgroundcontrast
effect would otherwise be disabled, leading to the panel and popups
flickering between contrast enabled and disabled.

With this patch, the panel keeps its coloring during desktop changes.


Diffs (updated)
-----

  kwin/effects/slide/slide.h c8e0a84 
  kwin/effects/slide/slide.cpp 8ecb2a6 

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


Testing
-------

Switched desktops, panel keeps color, fullscreen video player doesn't seem to be affected negatively, everything still works as expected, except that the flickering in Plasma Dialogs is gone.


Thanks,

Sebastian Kügler

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


More information about the Plasma-devel mailing list