From 9a31b0d7270f590625fe1d37ad03b740d59234f4 Mon Sep 17 00:00:00 2001
From: Aleix Pol <aleixpol@kde.org>
Date: Tue, 12 Apr 2022 02:31:16 +0200
Subject: [PATCH] x11: trigger shortcuts when releasing rather than pressing

This way we can issue a keyboard grab without problems
---
 src/backends/x11/standalone/x11_platform.cpp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/backends/x11/standalone/x11_platform.cpp b/src/backends/x11/standalone/x11_platform.cpp
index 2a86824edb..5e2a9c6d1c 100644
--- a/src/backends/x11/standalone/x11_platform.cpp
+++ b/src/backends/x11/standalone/x11_platform.cpp
@@ -40,6 +40,7 @@
 
 #include <KConfigGroup>
 #include <KCrash>
+#include <KGlobalAccel>
 #include <KLocalizedString>
 
 #include <QOpenGLContext>
@@ -368,7 +369,10 @@ void X11StandalonePlatform::startInteractivePositionSelection(std::function<void
 
 void X11StandalonePlatform::setupActionForGlobalAccel(QAction *action)
 {
-    connect(action, &QAction::triggered, kwinApp(), [action] {
+    connect(KGlobalAccel::self(), &KGlobalAccel::globalShortcutActiveChanged, kwinApp(), [action](QAction *triggeredAction, bool active) {
+        if (triggeredAction != action)
+            return;
+
         QVariant timestamp = action->property("org.kde.kglobalaccel.activationTimestamp");
         bool ok = false;
         const quint32 t = timestamp.toULongLong(&ok);