Bug 433841

Summary: Chord keyboard shortcuts are confusing
Product: [Plasma] plasmashell Reporter: Dan Dascalescu <ddascalescu+kde>
Component: generalAssignee: David Edmundson <kde>
Status: RESOLVED FIXED    
Severity: normal CC: anditosan1000, bugseforuns, kdedev, makosol, nate, plasma-bugs-null
Priority: NOR Keywords: usability
Version First Reported In: 5.21.1   
Target Milestone: 1.0   
Platform: Other   
OS: Linux   
See Also: https://bugs.kde.org/show_bug.cgi?id=433292
Latest Commit: Version Fixed In: 6.4.0
Sentry Crash Report:
Attachments: Confusing shortcuts when editing the panel

Description Dan Dascalescu 2021-03-02 07:58:37 UTC
Created attachment 136313 [details]
Confusing shortcuts when editing the panel

I'm unclear about the shortcuts displayed when right clicking the panel:

* Configure Task Manager / Alt+D, S - brings up Alternative Widgets instead
* Add Widgets / Alt+D, A - works
* Edit Panel / Alt+D, Alt+S - brings up Alternative Widgets instead

Why the need for Alt+D anyway? Pressing just `S` or `A` does bring up the Alternative Widgets and Add Widgets pages.

Another note: the accelerators aren't underlined until I press Alt. But they *are* triggered before pressing Alt. For example, right clicking on the panel and pressing 'C' does bring up the Task Manager Settings dialog.
Comment 1 Nate Graham 2021-03-02 18:36:03 UTC
Indeed, chord keyboard shortcuts are confusing. Because nobody can figure out how to use them, they break a lot without people noticing. See Bug 433292.
Comment 2 Nate Graham 2021-08-30 16:30:27 UTC
*** Bug 441733 has been marked as a duplicate of this bug. ***
Comment 3 Nate Graham 2022-01-24 22:26:42 UTC
*** Bug 449060 has been marked as a duplicate of this bug. ***
Comment 4 Bug Janitor Service 2025-01-28 19:02:05 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/print-manager/-/merge_requests/223
Comment 5 Bug Janitor Service 2025-01-28 19:02:27 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-pa/-/merge_requests/316
Comment 6 Bug Janitor Service 2025-01-28 19:02:53 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/powerdevil/-/merge_requests/499
Comment 7 Bug Janitor Service 2025-01-28 19:03:17 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-nm/-/merge_requests/411
Comment 8 Bug Janitor Service 2025-01-28 19:04:45 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/5129
Comment 9 Bug Janitor Service 2025-01-28 19:10:36 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/libplasma/-/merge_requests/1253
Comment 10 Bug Janitor Service 2025-01-28 19:11:23 UTC
A possibly relevant merge request was started @ https://invent.kde.org/plasma/plasma-desktop/-/merge_requests/2760
Comment 11 Nate Graham 2025-01-28 21:04:42 UTC
Git commit bab5b41905d3c5a833f67e223791103890850658 by Nate Graham.
Committed on 28/01/2025 at 20:59.
Pushed by ngraham into branch 'master'.

applets/batterymonitor: remove broken chord keyboard shortcut

This fundamentally doesn't work since multiple widgets set the same
shortcut, and all can be in the same focus group.

Part of https://invent.kde.org/plasma/plasma-workspace/-/issues/148.
Related: bug 433292

M  +0    -1    applets/batterymonitor/package/contents/ui/main.qml

https://invent.kde.org/plasma/powerdevil/-/commit/bab5b41905d3c5a833f67e223791103890850658
Comment 12 Nate Graham 2025-01-28 22:32:36 UTC
Git commit 6a9022fae183360c3c1e6e2b8027ce52720d403c by Nate Graham.
Committed on 28/01/2025 at 19:04.
Pushed by ngraham into branch 'master'.

Remove broken and unneeded chord keyboard shortcuts

This fundamentally doesn't work since multiple widgets set the same
shortcut, and all can be in the same focus group. And the Activity
Switcher already has a better global-scope Meta+Q shortcut anyway.

Part of https://invent.kde.org/plasma/plasma-workspace/-/issues/148.
Related: bug 433292

M  +0    -1    applets/clipboard/contents/ui/main.qml
M  +0    -1    applets/devicenotifier/package/contents/ui/main.qml
M  +0    -1    shell/shellcorona.cpp

https://invent.kde.org/plasma/plasma-workspace/-/commit/6a9022fae183360c3c1e6e2b8027ce52720d403c
Comment 13 Nate Graham 2025-01-28 22:32:54 UTC
Git commit 2b9b93a3fbf9c46c947654d3a26da2f28874dc89 by Nate Graham.
Committed on 28/01/2025 at 18:57.
Pushed by ngraham into branch 'master'.

applet: remove broken chord keyboard shortcut

This fundamentally doesn't work since multiple widgets set the same
shortcut, and all can be in the same focus group.

Part of https://invent.kde.org/plasma/plasma-workspace/-/issues/148.
Related: bug 433292

M  +0    -1    applet/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-nm/-/commit/2b9b93a3fbf9c46c947654d3a26da2f28874dc89
Comment 14 Nate Graham 2025-01-28 22:33:03 UTC
Git commit 73d17bd7bfa080f236d158f60923346afb24f8d9 by Nate Graham.
Committed on 28/01/2025 at 18:57.
Pushed by ngraham into branch 'master'.

applet: remove broken chord keyboard shortcut

This fundamentally doesn't work since multiple widgets set the same
shortcut, and all can be in the same focus group.

Part of https://invent.kde.org/plasma/plasma-workspace/-/issues/148.
Related: bug 433292

M  +0    -1    applet/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-pa/-/commit/73d17bd7bfa080f236d158f60923346afb24f8d9
Comment 15 Nate Graham 2025-01-28 22:33:34 UTC
Git commit 1d08d1592a530c364d113c5cb06b9c8cb3be3d53 by Nate Graham.
Committed on 28/01/2025 at 18:59.
Pushed by ngraham into branch 'master'.

Remove chord keyboard shortcuts

Don't set chord keyboard shortcuts on widgets and containments. For
widgets, it won't work anyway since the shortcuts will conflict when
widgets inhabit the same parent container (which is almost always). For
containments, users can't figure out what these shortcuts are and how
they work, so they're largely useless visual noise anyway.

Part of https://invent.kde.org/plasma/plasma-workspace/-/issues/148.
Related: bug 433292

M  +0    -1    src/plasma/corona.cpp
M  +0    -2    src/plasma/private/applet_p.cpp
M  +0    -3    src/plasma/private/containment_p.cpp

https://invent.kde.org/plasma/libplasma/-/commit/1d08d1592a530c364d113c5cb06b9c8cb3be3d53
Comment 16 Nate Graham 2025-01-28 22:42:27 UTC
Git commit aa53fc92b798dd0f1689a95daf3ab0a4678fdf0d by Nate Graham.
Committed on 28/01/2025 at 18:58.
Pushed by ngraham into branch 'master'.

plasmoid: remove broken chord keyboard shortcut

This fundamentally doesn't work since multiple widgets set the same
shortcut, and all can be in the same focus group.

Part of https://invent.kde.org/plasma/plasma-workspace/-/issues/148.
Related: bug 433292

M  +0    -1    src/plasmoid/package/contents/ui/main.qml

https://invent.kde.org/plasma/print-manager/-/commit/aa53fc92b798dd0f1689a95daf3ab0a4678fdf0d
Comment 17 Nate Graham 2025-01-30 16:16:51 UTC
Git commit 7e7523ad6e8333a58b14d94d81d00fe09ec55c14 by Nate Graham.
Committed on 30/01/2025 at 16:16.
Pushed by ngraham into branch 'master'.

desktop: Replace chord keyboard shortcut with something comprehensible

The desktop currently has a local-scoped shortcut to open the config
window. This is odd, but because the desktop isn't global (there can be
multiple desktops), in principle is can make sense for keyboard
accessibility.

Unfortunately it uses a chord shortcut, which users don't understand.
Replace it with a normal-style shortcut.

Part of https://invent.kde.org/plasma/plasma-workspace/-/issues/148.
Related: bug 433292

M  +2    -2    appiumtests/desktop/desktoptest.py
M  +1    -1    containments/desktop/package/contents/ui/main.qml

https://invent.kde.org/plasma/plasma-desktop/-/commit/7e7523ad6e8333a58b14d94d81d00fe09ec55c14
Comment 18 Nate Graham 2025-01-30 16:17:52 UTC
All have been removed or replaced with comprehensible shortcuts for Plasma 6.4.