Bug 459325

Summary: Keyboard shortcut change should be effective only after clicking the 'Apply' button
Product: [Applications] systemsettings Reporter: Patrick Silva <bugseforuns>
Component: kcm_kwintabboxAssignee: KWin default assignee <kwin-bugs-null>
Status: RESOLVED FIXED    
Severity: normal CC: nate, plasma-bugs, sbancuz
Priority: NOR    
Version: master   
Target Milestone: ---   
Platform: Neon   
OS: Linux   
Latest Commit: https://invent.kde.org/plasma/kwin/commit/76ac4be31632b436ea78611f07490c352a88b192 Version Fixed In: 5.27
Sentry Crash Report:

Description Patrick Silva 2022-09-18 12:42:19 UTC 
SUMMARY
Keyboard shortcut change should be effective only after clicking the 'Apply' button for consistency reasons,
because this is the behavior in Screen Locking and Notifications KCMs.

STEPS TO REPRODUCE
1. open Task Switcher KCM
2. click on any button used to set/change a keyboard shortcut
3. press the desired shortcut

OBSERVED RESULT
pressed shortcut is effective without click on 'Apply' button

EXPECTED RESULT
'Apply' button is activated after pressing the shortcut and the change is effective only after clicking on it

SOFTWARE/OS VERSIONS
Operating System: KDE neon Unstable Edition
KDE Plasma Version: 5.26.80
KDE Frameworks Version: 5.98.0
Qt Version: 5.15.6
Graphics Platform: Wayland
Comment 1 Nate Graham 2023-01-09 20:28:40 UTC 
*** Bug 463846 has been marked as a duplicate of this bug. ***
Comment 2 Bug Janitor Service 2023-01-11 00:18:57 UTC 
A possibly relevant merge request was started @ https://invent.kde.org/plasma/kwin/-/merge_requests/3417
Comment 3 Vlad Zahorodnii 2023-01-12 13:28:30 UTC 
Git commit 76ac4be31632b436ea78611f07490c352a88b192 by Vlad Zahorodnii, on behalf of Ismael Asensio.
Committed on 12/01/2023 at 12:15.
Pushed by vladz into branch 'master'.

kcms/tabbox: Make shortcuts config save on apply

Currently, tabbox shortcuts are directly saved to KGlobalAccel
when they change, which is inconsistent with the rest of the
KCM and leads to confusion regarding Defaults and isSaveNeeded
status.

Let's instead manage the changes internally and just save them
to KGlobalAccel on Apply/Save.

m_actionsCollection is only used as a helpful way to store this
internal configuration, and doesn't really needs saving to
or restoring from config.
FIXED-IN: 5.27

M  +62   -42   src/kcms/tabbox/kwintabboxconfigform.cpp
M  +5    -2    src/kcms/tabbox/kwintabboxconfigform.h
M  +6    -0    src/kcms/tabbox/main.cpp

https://invent.kde.org/plasma/kwin/commit/76ac4be31632b436ea78611f07490c352a88b192