mirrors/wlroots
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/wlroots/wlroots.git synced 2025-10-29 05:40:12 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

Revert "wlr_keyboard: don't emit key event for duplicated keycodes"

Browse source 
This reverts commit 86eaa44a3a.

That commit caused a regression for IME users in many compositors:
when a input_method is activated while a key is pressed, and a virtual
keyboard is created by IME, the following key-release event via the
virtual keyboard is missed since the key in the virtual keyboard haven't
been pressed. For example, pressing and releasing Ctrl+F in Firefox with
fcitx5 running triggered repeated keys (ffffff...) in the opened input
box.
This commit is contained in:
tokyo4j 2025-02-15 01:13:15 +09:00
parent e9450a9947
commit bd8454d3bc
2 changed files with 4 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

2
include/types/wlr_keyboard.h
View file

@ -1,6 +1,6 @@
#include <wlr/types/wlr_keyboard.h>
bool keyboard_key_update(struct wlr_keyboard *keyboard,
void keyboard_key_update(struct wlr_keyboard *keyboard,
struct wlr_keyboard_key_event *event);
bool keyboard_modifier_update(struct wlr_keyboard *keyboard);

8
types/wlr_keyboard.c
View file

@ -66,9 +66,8 @@ bool keyboard_modifier_update(struct wlr_keyboard *keyboard) {
return true;
}
bool keyboard_key_update(struct wlr_keyboard *keyboard,
void keyboard_key_update(struct wlr_keyboard *keyboard,
struct wlr_keyboard_key_event *event) {
size_t old_num_keycodes = keyboard->num_keycodes;
if (event->state == WL_KEYBOARD_KEY_STATE_PRESSED) {
set_add(keyboard->keycodes, &keyboard->num_keycodes,
WLR_KEYBOARD_KEYS_CAP, event->keycode);
@ -79,8 +78,6 @@ bool keyboard_key_update(struct wlr_keyboard *keyboard,
}
assert(keyboard->num_keycodes <= WLR_KEYBOARD_KEYS_CAP);
return old_num_keycodes != keyboard->num_keycodes;
}
void wlr_keyboard_notify_modifiers(struct wlr_keyboard *keyboard,
@ -102,9 +99,8 @@ void wlr_keyboard_notify_modifiers(struct wlr_keyboard *keyboard,
void wlr_keyboard_notify_key(struct wlr_keyboard *keyboard,
struct wlr_keyboard_key_event *event) {
if (keyboard_key_update(keyboard, event)) {
keyboard_key_update(keyboard, event);
wl_signal_emit_mutable(&keyboard->events.key, event);
}
if (keyboard->xkb_state == NULL) {
return;