mirrors/foot
1
0
Fork 0
mirror of https://codeberg.org/dnkl/foot.git synced 2026-04-05 07:15:30 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

input: match unshifted key-bindings before shifted

Browse source 
That is, try to match e.g. Control+shift+a, before trying to match
Control+A.

In most cases, order doesn't matter. There are however a couple of
symbols where the layout consumes the shift-modifier, and the
generated symbol is the same in both the shifted and unshifted
form. One such example is backspace.

Before this patch, key-bindings with shift-backspace would be ignored,
if there were another key-binding with backspace.

So, for example, if we had one key-binding with Control+Backspace, and
another with Control+Shift+Backspace, the latter would never trigger,
as we would always match the first one.

By checking for unshifted matches first, we ensure
Control+Shift+Backspace does match.
This commit is contained in:
Daniel Eklöf 2025-01-31 09:07:42 +01:00
parent bee17a95b8
commit 51128a3484
No known key found for this signature in database
GPG key ID: 5BBD4992C116573F
3 changed files with 42 additions and 42 deletions
Download patch file Download diff file Expand all files Collapse all files

26
url-mode.c
View file

@ -182,19 +182,6 @@ urls_input(struct seat *seat, struct terminal *term,
* Key bindings
*/
/* Match translated symbol */
tll_foreach(bindings->url, it) {
const struct key_binding *bind = &it->item;
if (bind->k.sym == sym &&
bind->mods == (mods & ~consumed))
{
execute_binding(seat, term, bind, serial);
return;
}
}
/* Match untranslated symbols */
tll_foreach(bindings->url, it) {
const struct key_binding *bind = &it->item;
@ -209,6 +196,19 @@ urls_input(struct seat *seat, struct terminal *term,
}
}
/* Match translated symbol */
tll_foreach(bindings->url, it) {
const struct key_binding *bind = &it->item;
if (bind->k.sym == sym &&
bind->mods == (mods & ~consumed))
{
execute_binding(seat, term, bind, serial);
return;
}
}
/* Match raw key code */
tll_foreach(bindings->url, it) {
const struct key_binding *bind = &it->item;