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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/kwin/-/commit/e398289287
                   |                            |d3e25ec48111c4f2a2b9d2a1195
                   |                            |c00
             Status|ASSIGNED                    |RESOLVED

--- Comment #2 from Vlad Zahorodnii <vlad.zahorod...@kde.org> ---
Git commit e398289287d3e25ec48111c4f2a2b9d2a1195c00 by Vlad Zahorodnii, on
behalf of Joshua Goins.
Committed on 19/10/2023 at 08:45.
Pushed by vladz into branch 'master'.

Add QKeySequence to VirtualDesktopManager::addAction overload

There are four usages of this overload, two of them are for the
intentionally disabled "Switch to Next/Previous Desktop" actions and the
rest are for "Switch One Desktop to X" actions. Due to the order they
were added, an empty keybind was set as the default and the actual
keybind is never enabled.

Now there's a QKeySequence argument to this overload, so an unexpected
empty keybind is never added. The two usages of addAction that depend on
this empty keybind behavior now pass in an empty QKeySequence.

M  +8    -12   src/virtualdesktops.cpp
M  +3    -3    src/virtualdesktops.h

https://invent.kde.org/plasma/kwin/-/commit/e398289287d3e25ec48111c4f2a2b9d2a1195c00

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

Reply via email to