https://bugs.kde.org/show_bug.cgi?id=459325

Vlad Zahorodnii <vlad.zahorod...@kde.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/kwin/commit/76ac4be31632
                   |                            |b436ea78611f07490c352a88b19
                   |                            |2
   Version Fixed In|                            |5.27
             Status|ASSIGNED                    |RESOLVED

--- Comment #3 from Vlad Zahorodnii <vlad.zahorod...@kde.org> ---
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

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to