Bug 459237 - It's possible to see, set, and modify shortcuts for disabled KWin effects/scripts, which can cause user confusion
Summary: It's possible to see, set, and modify shortcuts for disabled KWin effects/scr...
Status: CONFIRMED
Alias: None
Product: systemsettings
Classification: Applications
Component: kcm_keys (show other bugs)
Version: 6.1.3
Platform: Arch Linux Linux
: NOR normal
Target Milestone: ---
Assignee: Plasma Bugs List
URL:
Keywords: usability
Depends on:
Blocks:
 
Reported: 2022-09-16 20:53 UTC by iodreamify
Modified: 2024-10-11 19:10 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
short demo (336.26 KB, video/x-matroska)
2022-09-16 20:53 UTC, iodreamify
Details

Note You need to log in before you can comment on or make changes to this bug.
Description iodreamify 2022-09-16 20:53:38 UTC
Created attachment 152132 [details]
short demo

SUMMARY
Setting any shortcut for Minimize All Windows in System Settings doesn't seem to do anything. Executing the shortcut seems to behave as if the meta key isn't pressed, other key combinations issue similar results.
The only way to trigger Minimize all seems to be to place the Minimize All Windows widget on the panel and then assign a key combination to activate it, then it works.

STEPS TO REPRODUCE
1. Assign a keyboard shortcut to Minimize All function in KWin in system settings 
2. Try to use it

OBSERVED RESULT
Nothing happens, key combo ignored

EXPECTED RESULT
All windows are minimized

Operating System: Arch Linux
KDE Plasma Version: 5.25.5
KDE Frameworks Version: 5.98.0
Qt Version: 5.15.6
Kernel Version: 5.19.8-arch1-1 (64-bit)
Graphics Platform: X11
Comment 1 Nate Graham 2022-09-22 18:20:27 UTC
You said that the meta key doesn't work for other shortcuts too... can you try to narrow this down? Like, is the Meta key broken in all shortcuts, but shortcuts using only Ctrl or Alt as modifiers work?

What keyboard layout are you using?

Are you doing any key re-mapping?
Comment 2 iodreamify 2022-09-22 19:03:38 UTC
> You said that the meta key doesn't work for other shortcuts too... 
Sorry for the confusion, I meant to say no other key combination for this MinimizeAll function works.

I think I've found the culprit. It turns out my MinimizeAll KWin script was turned off altogether. As soon as I turned it on it started working.
I totally forgot KWin has two ways to hide all windows: via the ShowDesktop function(which is by default assigned to Meta+D and as far as i know can't be turned off) and MinimizeAll. And it turns out MinimizeAll is handled like an external script which can be turned off at will. 

However as soon as you turn off the script, it still lets you assign a shortcut for a given function(which was created when i first enabled the MinimizeAll script for the first time as far as i can tell) and doesn't inform the user that the script is actually disabled, so this may a UX issue. 

Sorry again for the confusion, i guess the bug report may be closed.
Comment 3 Nate Graham 2022-09-23 18:23:59 UTC
Yep, that makes sense.

Letting you set shortcuts for disabled scripts/effects is not ideal though, yeah. We should probably hide these shortcuts from the KCM while the scripts/effects they control are themselves disabled.
Comment 4 Arek Guzinski 2024-10-11 19:10:33 UTC
The UX issue is even worse than described, as Shortcuts from KWin scripts are visible in Systemsettings even after the script has been uninstalled.
As a workaround it is possible to delete their lines in ~/.config/kglobalshortcutsrc -- they're still visible until logout though.

(Tested with Plasma 6.2)