From 9f3a447a43e35146f67dc27d176e5b00ceb0665e Mon Sep 17 00:00:00 2001
From: Vlad Zagorodniy <vladzzag@gmail.com>
Date: Fri, 19 Oct 2018 12:38:27 +0300
Subject: [PATCH] [effects/diminactive] Dim Wayland clients

Summary:
There is no concept of managed windows in Wayland, so every time we call
managed() on a Wayland client, it will return false.

This change addresses that problem by invoking managed() only for X11 clients.

CCBUG: 398100

Reviewers: #kwin, davidedmundson

Reviewed By: #kwin, davidedmundson

Subscribers: kwin

Tags: #kwin

Differential Revision: https://phabricator.kde.org/D16312
---
 effects/diminactive/diminactive.cpp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/effects/diminactive/diminactive.cpp b/effects/diminactive/diminactive.cpp
index 862e1617b5..11619b12cd 100644
--- a/effects/diminactive/diminactive.cpp
+++ b/effects/diminactive/diminactive.cpp
@@ -199,7 +199,11 @@ bool DimInactiveEffect::canDimWindow(const EffectWindow *w) const
         return false;
     }
 
-    if (!w->isManaged()) {
+    if (w->isPopupWindow()) {
+        return false;
+    }
+
+    if (w->isX11Client() && !w->isManaged()) {
         return false;
     }