]> git.mdlowis.com Git - proto/labwc.git/commit
src/keyboard.c: do not end window-cycling on modifier release only
authorJohan Malm <jgm323@gmail.com>
Tue, 30 Aug 2022 14:47:00 +0000 (15:47 +0100)
committerJohan Malm <jgm323@gmail.com>
Wed, 21 Sep 2022 20:25:13 +0000 (21:25 +0100)
commit20c4ffa539d3065bf96db30089aa3b0e9955753a
tree55a6eac2e97d5a6bd442684c00dc8e2825cb751f
parentffb2efe7333a7aa4100cf247ce6e153886de2892
src/keyboard.c: do not end window-cycling on modifier release only

If a user lets go of the modifier (e.g. alt) before the 'normal' key (e.g.
tab) when window-cycling, we do not end the cycling until both keys have
been released.  If we end the window-cycling on release of the modifier
only, some XWayland clients such as hexchat realise that tab is pressed
(even though we did not forward the event) and because we absorb the
equivalent release event it gets stuck on repeat.

Just to clarify the position here: Issue #176 describes a behaviour
whereby dmenu gets stuck on repeat after being launched with a keybind.
This patch does not resolve that issue but reflects that in Wayland, the
client is responsible for implementing "key repeat".

Changing the key repeat rate/delay in (labwc/labwc@e62bb51) was dirty fix
that need should never have been made.
include/key-state.h
src/key-state.c
src/keyboard.c