diff options
| author | N-R-K <79544946+N-R-K@users.noreply.github.com> | 2021-12-01 00:41:22 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-12-01 00:41:22 +0100 |
| commit | 3bc7082f4e08e71fb40944bb085fafbb65f6c8cf (patch) | |
| tree | a47b2421ab25c4a28257252f6bb9fecfc3797adc /main.c | |
| parent | 68ddbe28d87eed9efe5158cdaa5613e0ca66e3c5 (diff) | |
| download | nsxiv-3bc7082f4e08e71fb40944bb085fafbb65f6c8cf.tar.zst | |
fix: send implicit_mod to process_bindings (#176)
* fix: send implicit_mod to process_bindings
this solves the edge case where someone might have `ShiftMask + A` in
their keybindings compared to a plain `A`.
Closes: https://github.com/nsxiv/nsxiv/pull/166#issuecomment-978853136
* code-style: smuggle small style fix in
win_draw_bar now mimics autoreload_nop.c functions
Diffstat (limited to 'main.c')
| -rw-r--r-- | main.c | 10 |
1 files changed, 5 insertions, 5 deletions
@@ -605,15 +605,15 @@ end: redraw(); } -static bool process_bindings(const keymap_t *keys, unsigned int len, - KeySym ksym_or_button, unsigned int state) +static bool process_bindings(const keymap_t *keys, unsigned int len, KeySym ksym_or_button, + unsigned int state, unsigned int implicit_mod) { unsigned int i; bool dirty = false; for (i = 0; i < len; i++) { if (keys[i].ksym_or_button == ksym_or_button && - MODMASK(keys[i].mask) == MODMASK(state) && + MODMASK(keys[i].mask | implicit_mod) == MODMASK(state) && keys[i].cmd.func && (keys[i].cmd.mode == MODE_ALL || keys[i].cmd.mode == mode)) { @@ -652,7 +652,7 @@ static void on_keypress(XKeyEvent *kev) prefix = prefix * 10 + (int) (key - '0'); return; } else { - dirty = process_bindings(keys, ARRLEN(keys), ksym, kev->state & ~sh); + dirty = process_bindings(keys, ARRLEN(keys), ksym, kev->state, sh); } if (dirty) redraw(); @@ -668,7 +668,7 @@ static void on_buttonpress(XButtonEvent *bev) if (mode == MODE_IMAGE) { set_timeout(reset_cursor, TO_CURSOR_HIDE, true); reset_cursor(); - dirty = process_bindings(buttons, ARRLEN(buttons), bev->button, bev->state); + dirty = process_bindings(buttons, ARRLEN(buttons), bev->button, bev->state, 0); if (dirty) redraw(); } else { |