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

(Updated Feb. 18, 2014, 12:54 p.m.)


Review request for kwin and Plasma.


Changes
-------

Improved version:

- This also works for the first animation (the previous patch only set the 
property after the first time the animation runs)
- Also enables the background effect for the slidingpopups, leading to a less 
flickery appearance there
- Slower default animation time when disappearing -- in line with appearing 
default time


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.cpp 8ecb2a6 
  kwin/effects/slidingpopups/slidingpopups.cpp fd697f0 

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

_______________________________________________
Plasma-devel mailing list
Plasma-devel@kde.org
https://mail.kde.org/mailman/listinfo/plasma-devel

Reply via email to