From 4a6879e6ee737d85022c8f351e0496918786fefc Mon Sep 17 00:00:00 2001
From: ivan tkachenko <me@ratijas.tk>
Date: Tue, 13 Sep 2022 12:19:02 +0300
Subject: [PATCH] effects/Overview: Move delegate handlers to the bottom close
 to each other

---
 src/effects/overview/qml/ScreenView.qml | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/src/effects/overview/qml/ScreenView.qml b/src/effects/overview/qml/ScreenView.qml
index 8533ee9bf5..48304c63e0 100644
--- a/src/effects/overview/qml/ScreenView.qml
+++ b/src/effects/overview/qml/ScreenView.qml
@@ -243,12 +243,6 @@ FocusScope {
                 animationDuration: effect.animationDuration
                 animationEnabled: container.animationEnabled
                 organized: container.organized
-                onWindowClicked: {
-                    if (eventPoint.event.button !== Qt.MiddleButton) {
-                        return;
-                    }
-                    window.closeWindow();
-                }
                 Keys.priority: Keys.AfterItem
                 Keys.forwardTo: searchResults
                 model: KWinComponents.ClientFilterModel {
@@ -262,7 +256,6 @@ FocusScope {
                                 ~KWinComponents.ClientFilterModel.Desktop &
                                 ~KWinComponents.ClientFilterModel.Notification
                 }
-                onActivated: effect.deactivate();
                 delegate: WindowHeapDelegate {
                     windowHeap: heap
 
@@ -282,6 +275,12 @@ FocusScope {
                     opacity: 1 - downGestureProgress
                     onDownGestureTriggered: client.closeWindow()
                 }
+                onActivated: effect.deactivate();
+                onWindowClicked: {
+                    if (eventPoint.event.button === Qt.MiddleButton) {
+                        window.closeWindow();
+                    }
+                }
             }
 
             Milou.ResultsView {