Bug 400278 - Widget keyboard shortcut works only for its activation - not working for deactivation
Summary: Widget keyboard shortcut works only for its activation - not working for deac...
Status: RESOLVED FIXED
Alias: None
Product: plasmashell
Classification: Plasma
Component: general (show other bugs)
Version: master
Platform: Other Linux
: NOR normal
Target Milestone: 1.0
Assignee: David Edmundson
URL:
Keywords: usability
: 426660 (view as bug list)
Depends on:
Blocks:
 
Reported: 2018-10-25 06:55 UTC by Alexander Mentyu
Modified: 2020-11-07 12:44 UTC (History)
4 users (show)

See Also:
Latest Commit:
Version Fixed In: 5.76


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexander Mentyu 2018-10-25 06:55:25 UTC
STEPS TO REPRODUCE
1. Start Wayland.
2. Set shortcut for any panel or desktop widget - for example right click on digital clock panel widget -> Keyboard shortcuts -> set shortcut to Ctrl+Alt+Y -> apply.
3. Invoke digital clock widget by newly set shortcut.
4. Press the shortcut one more time.

OBSERVED RESULT
The widget not deactivates

EXPECTED RESULT
Widget deactivates - X behavior

SOFTWARE VERSIONS
Operating System: KDE neon Developer Edition
KDE Plasma Version: 5.14.80
Qt Version: 5.11.2
KDE Frameworks Version: 5.52.0
Comment 1 Patrick Silva 2020-06-17 15:30:24 UTC
Still an issue.

Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.19.80
KDE Frameworks Version: 5.72.0
Qt Version: 5.14.2
Comment 2 Nate Graham 2020-06-24 03:54:34 UTC
This is happening on X11 too.
Comment 3 Nate Graham 2020-10-04 14:58:22 UTC
*** Bug 426660 has been marked as a duplicate of this bug. ***
Comment 4 Alexander Lohnau 2020-10-07 16:18:58 UTC
Git commit d6a5b10b3b9a4984209f6509a3ed9196b0c1d5d6 by Alexander Lohnau.
Committed on 07/10/2020 at 16:18.
Pushed by alex into branch 'master'.

Toggle plasmoids when shortcut is activated
FIXED-IN: 5.76

Now the plasmoids are toggled by default. The system tray is a
special case, which needs to be in plasma-workspace adjusted.

M  +1    -1    src/plasmaquick/appletquickitem.cpp

https://invent.kde.org/frameworks/plasma-framework/commit/d6a5b10b3b9a4984209f6509a3ed9196b0c1d5d6
Comment 5 Konrad Materka 2020-11-07 12:44:14 UTC
Git commit 04112536133184eeb53af6016e9aeddf9db546d4 by Konrad Materka.
Committed on 07/11/2020 at 12:03.
Pushed by kmaterka into branch 'master'.

[applets/systemtray] Rework of "expanded" state handling

Refactor and simply logic of "expanded" state when dialog with active applet or list of hidden applets is shown.

Fixes a problem with keyboard shortcut activation.
It also fixes a bug when dialog closes on "go-back" button click.
Related: bug 427685
FIXED-IN: 5.21

M  +2    -3    applets/systemtray/container/package/contents/ui/main.qml
M  +20   -2    applets/systemtray/package/contents/ui/CurrentItemHighLight.qml
M  +8    -18   applets/systemtray/package/contents/ui/ExpandedRepresentation.qml
M  +6    -7    applets/systemtray/package/contents/ui/ExpanderArrow.qml
M  +1    -1    applets/systemtray/package/contents/ui/HiddenItemsView.qml
M  +3    -3    applets/systemtray/package/contents/ui/PlasmoidPopupsContainer.qml
A  +93   -0    applets/systemtray/package/contents/ui/SystemTrayState.qml     [License: LGPL (v2+)]
M  +1    -1    applets/systemtray/package/contents/ui/items/AbstractItem.qml
M  +4    -12   applets/systemtray/package/contents/ui/items/PlasmoidItem.qml
M  +11   -52   applets/systemtray/package/contents/ui/main.qml
M  +12   -0    applets/systemtray/systemtray.cpp

https://invent.kde.org/plasma/plasma-workspace/commit/04112536133184eeb53af6016e9aeddf9db546d4