From b1b1360ec3fb0ddddcfed2d3db90985cf8495aa8 Mon Sep 17 00:00:00 2001 From: David Edmundson Date: Wed, 10 Oct 2018 11:45:13 +0100 Subject: [PATCH] [effects/fadedesktop] Set animation as a fullscreen effect Summary: This lets other effects know whether they should inhibit or not which reduces visual glitches of overlapping animations. BUG: 321201 FIXED-IN: 5.15.0 Test Plan: Switched desktops, they still faded Reviewers: #kwin, zzag Subscribers: zzag, kwin Tags: #kwin Differential Revision: https://phabricator.kde.org/D15474 --- .../fadedesktop/package/contents/code/main.js | 21 ++++++++++++++++--- 1 file changed, 18 insertions(+), 3 deletions(-) diff --git a/effects/fadedesktop/package/contents/code/main.js b/effects/fadedesktop/package/contents/code/main.js index 9f3b2e3eb6..064989dea8 100644 --- a/effects/fadedesktop/package/contents/code/main.js +++ b/effects/fadedesktop/package/contents/code/main.js @@ -27,8 +27,11 @@ effect.configChanged.connect(function() { loadConfig(); }); effects['desktopChanged(int,int)'].connect(function(oldDesktop, newDesktop) { + if (effects.hasActiveFullScreenEffect && !effect.isActiveFullScreenEffect) { + return; + } var stackingOrder = effects.stackingOrder; - for (var i=0; i