SUMMARY If Overview effect is set to open and close with meta key, opening it works fine, but when trying to close it with meta key, it closes and immediately reopens. STEPS TO REPRODUCE 1. Set overview effect to open with Meta key only 2. Open overview with meta key 3. Close overview with meta key OBSERVED RESULT Overview immediately reopens EXPECTED RESULT Overview should just close SOFTWARE/OS VERSIONS Operating System: Fedora Linux 40 KDE Plasma Version: 6.1.90 KDE Frameworks Version: 6.7.0 Qt Version: 6.7.2 Kernel Version: 6.10.10-200.fc40.x86_64 (64-bit) Graphics Platform: Wayland Processors: 12 × AMD Ryzen 5 3600 6-Core Processor Memory: 15,5 GiB of RAM Graphics Processor: AMD Radeon RX 6600 ADDITIONAL INFORMATION In overvieweffect.cpp:355 changing the QEvent::KeyPress to QEvent::KeyRelease seems to fix the issue, since KeyPress for some reason sends event when the key is raised too.
Hmm, cannot reproduce with today's git master.
can confirm, and I also see why it doesn't happen for Nate. The overview effect gets the keyboard shortcuts on startup, so you need to have the shortcut configured when the effect gets loaded.
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/6487
Git commit ff2566562d413dc268d87b3c72fd56a6061a5d40 by Xaver Hugl. Committed on 25/09/2024 at 11:57. Pushed by zamundaaa into branch 'master'. plugins/overview, windowview: don't handle keyboard shortcuts on Wayland The only reason for handling them is that the effect grabs the keyboard, which disables global shortcuts on Xorg. In the Wayland session though, global shortcuts still work while the effects are active, and usually that works fine, as the global shortcuts filter filters out the key press events before they reach the effect. Modifier-only shortcuts though trigger on release, so they don't get filtered out and the shortcut effectively gets triggered twice, once by the effect on press, and once by kglobalacceld on release. M +21 -19 src/plugins/overview/overvieweffect.cpp M +1 -1 src/plugins/windowview/windowvieweffect.cpp https://invent.kde.org/plasma/kwin/-/commit/ff2566562d413dc268d87b3c72fd56a6061a5d40
Git commit 1654ddd270849e7c0b093324f5b094cb12d13c88 by Xaver Hugl. Committed on 25/09/2024 at 19:05. Pushed by zamundaaa into branch 'Plasma/6.2'. plugins/overview, windowview: don't handle keyboard shortcuts on Wayland The only reason for handling them is that the effect grabs the keyboard, which disables global shortcuts on Xorg. In the Wayland session though, global shortcuts still work while the effects are active, and usually that works fine, as the global shortcuts filter filters out the key press events before they reach the effect. Modifier-only shortcuts though trigger on release, so they don't get filtered out and the shortcut effectively gets triggered twice, once by the effect on press, and once by kglobalacceld on release. (cherry picked from commit ff2566562d413dc268d87b3c72fd56a6061a5d40) Co-authored-by: Xaver Hugl <xaver.hugl@gmail.com> M +21 -19 src/plugins/overview/overvieweffect.cpp M +1 -1 src/plugins/windowview/windowvieweffect.cpp https://invent.kde.org/plasma/kwin/-/commit/1654ddd270849e7c0b093324f5b094cb12d13c88
*** Bug 493650 has been marked as a duplicate of this bug. ***