Zander671/river
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Mapping: Do not translate keys with xkb

Browse Source 
When checking keys for matching mappings, previously we did two checks:

1. Keysyms translated by xkb.
2. Raw keysyms

This commit removes the first check, so only the second is checked.

We're doing this because of strange behavior that xkb shows for some
layouts and keys. When pressing `Shift Space` on some layouts (Swedish
among others), xkb reports `Shift` as consumed. This leads to the case
that we cannot distinguish between `Space` and `Shift Space` presses
when doing a correct translation with xkb.
This commit is contained in:
Peter Kaplan 2022-05-15 17:45:59 +02:00 committed by Isaac Freund
parent d47be3b592
commit 47c02ebcbc
1 changed files with 11 additions and 26 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

37
river/Mapping.zig
View File

@ -69,7 +69,7 @@ pub fn deinit(self: Self) void {
pub fn match(
self: Self,
keycode: xkb.Keycode,
modifiers_raw: wlr.Keyboard.ModifierMask,
modifiers: wlr.Keyboard.ModifierMask,
released: bool,
xkb_state: *xkb.State,
) bool {
@ -82,43 +82,28 @@ pub fn match(
// will fall back to the active layout if so.
const layout_index = self.layout_index orelse xkb_state.keyGetLayout(keycode);
// Raw keysyms and modifiers as if modifiers didn't change keysyms.
// Get keysyms from the base layer, as if modifiers didn't change keysyms.
// E.g. pressing `Super+Shift 1` does not translate to `Super Exclam`.
const keysyms_raw = keymap.keyGetSymsByLevel(
const keysyms = keymap.keyGetSymsByLevel(
keycode,
layout_index,
0,
);
if (std.meta.eql(modifiers_raw, self.modifiers)) {
for (keysyms_raw) |sym| {
if (std.meta.eql(modifiers, self.modifiers)) {
for (keysyms) |sym| {
if (sym == self.keysym) {
return true;
}
}
}
// Keysyms and modifiers as translated by xkb.
// Modifiers used to translate the key are consumed.
// E.g. pressing `Super+Shift 1` translates to `Super Exclam`.
const keysyms_translated = keymap.keyGetSymsByLevel(
keycode,
layout_index,
xkb_state.keyGetLevel(keycode, layout_index),
);
// We deliberately choose not to translate keysyms and modifiers with xkb,
// because of strange behavior that xkb shows for some layouts and keys.
// When pressing `Shift Space` on some layouts (Swedish among others),
// xkb reports `Shift` as consumed. This leads to the case that we cannot
// distinguish between `Space` and `Shift Space` presses when doing a
// correct translation with xkb.
const consumed = xkb_state.keyGetConsumedMods2(keycode, xkb.ConsumedMode.xkb);
const modifiers_translated = @bitCast(
wlr.Keyboard.ModifierMask,
@bitCast(u32, modifiers_raw) & ~consumed,
);
if (std.meta.eql(modifiers_translated, self.modifiers)) {
for (keysyms_translated) |sym| {
if (sym == self.keysym) {
return true;
}
}
}
return false;
}