config: don't allow Shift in mouse bindings

Shift is a special modifier that is used to enable selection when the
client application is grabbing the mouse.
This commit is contained in:
Daniel Eklöf 2020-08-11 10:44:27 +02:00
parent 28410f1b99
commit 517d13fbce
No known key found for this signature in database
GPG key ID: 5BBD4992C116573F
2 changed files with 9 additions and 0 deletions

View file

@ -1026,6 +1026,11 @@ parse_mouse_combos(struct config *conf, const char *combos, key_combo_list_t *ke
*key = '\0';
if (!parse_modifiers(conf, combo, key - combo, &modifiers, path, lineno))
goto err;
if (modifiers.shift) {
LOG_AND_NOTIFY_ERR("%s:%d: Shift cannot be used in mosue bindings",
path, lineno);
goto err;
}
key++; /* Skip past the '+' */
}

View file

@ -369,6 +369,10 @@ of the modifiers *must* be valid XKB key names, and the button name
*must* be a valid libinput name. You can find the button names using
*libinput debug-events*.
Note that *Shift* cannot be used as a modifier in mouse bindings since
it is used to enable selection when the client application is grabbing
the mouse.
The trailing *COUNT* is optional and specifies the click count
required to trigger the binding. The default if *COUNT* is omitted is
_1_.