Bug 400278

Summary:	Widget keyboard shortcut works only for its activation - not working for deactivation
Product:	[Plasma] plasmashell	Reporter:	Alexander Mentyu <notuxius>
Component:	general	Assignee:	David Edmundson <kde>
Status:	RESOLVED FIXED
Severity:	normal	CC:	adam.m.fontenot+kde, bugseforuns, nate, plasma-bugs
Priority:	NOR	Keywords:	usability
Version:	master
Target Milestone:	1.0
Platform:	Other
OS:	Linux
Latest Commit:	https://invent.kde.org/frameworks/plasma-framework/commit/d6a5b10b3b9a4984209f6509a3ed9196b0c1d5d6	Version Fixed In:	5.76
Sentry Crash Report:

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