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

--- Comment #1 from Oded Arbel <o...@geek.co.il> ---
I think the problem is in kwin's `ModifierOnlyShortcuts` class. It uses
`KeyEvent::modifiersRelevantForGlobalShortcuts()` - which to the best of my
understanding supports detecting multiple modifiers, but it then casts it to
`Qt::KeyboardModifier` which can only represent one modifier -
https://invent.kde.org/plasma/kwin/-/blob/7fffe993/src/modifier_only_shortcuts.cpp#L52

Though I'm not sure that this is what the shortcuts KCM does (see
https://bugs.kde.org/show_bug.cgi?id=464805#c5), it is probably still a good
idea to fix this, and will likely also require changes to the config file
parser at
https://invent.kde.org/plasma/kwin/-/blob/7fffe993/src/options.cpp#L749 to be
able to read syntax such as "ShiftControl=...".

Also, I think it would be a really good idea if this mechanism would allow to
distinguish between physical keys - such as using "LeftShift" instead of any
"Shift". This will make a solution to bug #470257 much more useful for people
with more exotic group switching shortcuts, which xkbcommon supports - but this
can't be implemented while relying on `Qt::KeyboardModifier` as the underlying
data atom, as it doesn't make that distinction.

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

Reply via email to