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

Vlad Zahorodnii <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Latest Commit|                            |https://invent.kde.org/plas
                   |                            |ma/kglobalacceld/-/commit/9
                   |                            |74081aa833e4e1d05d0bdc90292
                   |                            |9966f16f9ce0
             Status|ASSIGNED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #2 from Vlad Zahorodnii <[email protected]> ---
Git commit 974081aa833e4e1d05d0bdc902929966f16f9ce0 by Vlad Zahorodnii.
Committed on 14/05/2026 at 12:20.
Pushed by vladz into branch 'master'.

Workaround kglobalacceld not grabbing some modifier-only combinations

53adba9315cc1a3cff961b7682336ee4d479dba4 made KGlobalAccelD lookup
global shortcuts in _active_keys, which was a fairly reasonable idea.
_active_keys should contain all the monitored keys, so we can use that
instead of going through components.

Unfortunately, things are different in practice. It looks like
kglobalacceld fails to grab key combinations such as Meta+Control, they
still work though, not sure why. This is a pre-existing issue. Things
used to work because _active_keys was not used to lookup global
shortcuts until recently.

In order to "fix" some modifier-only global shortcuts in the Xorg
session, this change makes KGlobalAccelD go through all available global
shortcuts to find a match, i.e. it restores the previous behavior for
the X11 session.

On Wayland, _active_keys is populated as expected, so that code can be
kept as is.

M  +1    -4    src/component.cpp
M  +20   -7    src/globalshortcutcontext.cpp
M  +1    -0    src/globalshortcutcontext.h
M  +20   -6    src/globalshortcutsregistry.cpp

https://invent.kde.org/plasma/kglobalacceld/-/commit/974081aa833e4e1d05d0bdc902929966f16f9ce0

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

Reply via email to