Bug 404856

Summary: [Wayland] Right click on desktop and panel stop working after exiting desktop grid effect
Product: [Plasma] plasmashell Reporter: Patrick Silva <bugseforuns>
Component: generalAssignee: David Edmundson <kde>
Status: RESOLVED FIXED    
Severity: normal CC: aleixpol, grasm, me, nate, plasma-bugs, postix, qydwhotmail
Priority: NOR Keywords: wayland
Version: 5.26.0   
Target Milestone: 1.0   
Platform: Arch Linux   
OS: Linux   
Latest Commit: Version Fixed In: 6.0
Sentry Crash Report:

Description Patrick Silva 2019-02-26 19:53:01 UTC
SUMMARY
Only reproducible on Wayland.
Both Arch Linux and neon dev unstable are affected.

STEPS TO REPRODUCE
1. minimize all windows
2. activate desktop grid effect (default shortcut is ctrl+f8)
3. leave desktop grid effect by clicking some desktop or by pressing the same shortcut
4. right click on desktop

OBSERVED RESULT
nothing happens. Right click works again after you click on toolbox (icon in the upper right corner).

EXPECTED RESULT
right click on desktop always works.


SOFTWARE/OS VERSIONS
Operating System: Arch Linux 
KDE Plasma Version: 5.15.2
KDE Frameworks Version: 5.55.0
Qt Version: 5.12.1

Operating System: KDE neon Developer Edition
KDE Plasma Version: 5.15.80
KDE Frameworks Version: 5.56.0
Qt Version: 5.12.0
Comment 1 Vlad Zahorodnii 2019-03-14 08:35:38 UTC
Hmm, so the problem seems to be that the desktop grid effect eats ctrl release event. So, after you leave desktop grid, plasmashell thinks that Ctrl is still pressed.
Comment 2 Vlad Zahorodnii 2019-03-14 09:15:40 UTC
s/pressed/depressed/
Comment 3 Vlad Zahorodnii 2019-03-14 11:58:55 UTC
Hmm, it seems like this is a client issue indeed. When the desktop grid effect is activated, kwin will send wl_keyboard.leave. so, once plasma receives that event, it probably has to reset its view of keyboard state.
Comment 4 Nate Graham 2021-03-04 19:58:29 UTC
Can reproduce. Seems to affect the panel too.
Comment 5 Aleix Pol 2022-01-12 16:21:01 UTC
It seems to be working now, several parts have changed in this area. Please reopen if it's still a problem for you.
Comment 6 Patrick Silva 2022-01-12 16:32:56 UTC
it's still reproducible on neon unstable.
Comment 7 Nate Graham 2022-01-12 17:17:49 UTC
Huh, it's working for me too with current git master.

I wonder if this got fixed by some Qt backport that Neon doesn't have or something.
Comment 8 Fushan Wen 2022-09-17 17:12:24 UTC
Can reproduce on master
Comment 9 postix 2022-10-21 12:13:11 UTC
It's not only reproducible with the grid effect but with _all_ window-heap based effects: grid, overview and present windows.

Also for the latter two I can 100% reproduce it by opening the effect, pressing CTRL+Meta+Left/Right arrow-key to switch the virtual desktop and then exit the effect by the corresponding hotkey.
Comment 10 Nate Graham 2022-10-21 15:47:01 UTC
I can't reproduce that.
Comment 11 postix 2022-10-21 15:55:39 UTC
(In reply to Nate Graham from comment #10)
> I can't reproduce that.

I recall: I cannot 100% reproduce it, must have been some luck that it happened many times in a row. 

Anyway, I can reproduce the issue with all windows heap based effects: Open a effect, move the cursor over the windows or switch the VDs by some hotkey and exit the effect. In some cases the panel's context menu stops responding until an applet is opened by a left click.
Comment 12 Nate Graham 2022-10-21 16:09:01 UTC
That sounds very much like Bug 445213, which is caused by an upstream issue that's not fixed yet.
Comment 13 Nate Graham 2023-04-27 21:05:18 UTC
This was fixed a while ago.
Comment 14 Patrick Silva 2023-04-27 21:39:54 UTC
Fixed on Plasma 6? It's still reproducible on Plasma 5.27.4 with the following steps:

1. activate Desktop Grid by pressing ctrl+f8
2. click on an virtual desktop without any running app
3. right-click on desktop or task manager

Result: the context menu does not open

Operating System: Arch Linux 
KDE Plasma Version: 5.27.4
KDE Frameworks Version: 5.105.0
Qt Version: 5.15.9
Graphics Platform: Wayland
Comment 15 Nate Graham 2023-05-14 17:41:36 UTC
In 6, yeah.