https://bugs.kde.org/show_bug.cgi?id=375355
Martin Gräßlin <mgraess...@kde.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest Commit| |https://commits.kde.org/kwi
| |n/5a87fa3f92cfc29944b8dc66c
| |2320e3062b532ee
Resolution|--- |FIXED
Status|CONFIRMED |RESOLVED
Version Fixed In| |5.8.6
--- Comment #7 from Martin Gräßlin <mgraess...@kde.org> ---
Git commit 5a87fa3f92cfc29944b8dc66c2320e3062b532ee by Martin Gräßlin.
Committed on 25/01/2017 at 12:39.
Pushed by graesslin into branch 'Plasma/5.8'.
Support modifier-only-shortcuts when capslock is on
Summary:
So far we didn't trigger modifier-only-shortcuts when capslock was
enabled. In fact we even ensured that the shortcuts did not trigger.
This seems not to be what our users expect. Meta should still trigger
if capslock is on.
This change modifies the logic to determine which modifier is currently
pressed by using the modifiersRelevantForGlobalShortcuts. The difference
to the "normal" modifiers is that this excludes capslock from modifiers
and excludes consumed modifiers. The latter is not really relevant as
modifier-only-shortcuts do not trigger if multiple keys are pressed,
which is required to have a modifier consumed.
FIXED-IN: 5.8.6
Test Plan: Only with adjusted autotest
Reviewers: #kwin, #plasma
Subscribers: plasma-devel, kwin
Tags: #kwin
Differential Revision: https://phabricator.kde.org/D4241
M +14 -8 autotests/integration/modifier_only_shortcut_test.cpp
M +3 -3 keyboard_input.cpp
https://commits.kde.org/kwin/5a87fa3f92cfc29944b8dc66c2320e3062b532ee
--
You are receiving this mail because:
You are watching all bug changes.
