Browse Source

fix(sticky keys): fix tapping-sticky-keys bug.

Sticky keys should be cleared when they were tapped in rapid succession before the timer runs out.
xmkb
Okke Formsma 4 years ago committed by Pete Johanson
parent
commit
e00eceb508
  1. 10
      app/src/behaviors/behavior_sticky_key.c
  2. 1
      app/tests/sticky-keys/9-sk-dn-up-dn-up/events.patterns
  3. 4
      app/tests/sticky-keys/9-sk-dn-up-dn-up/keycode_events.snapshot
  4. 27
      app/tests/sticky-keys/9-sk-dn-up-dn-up/native_posix.keymap

10
app/src/behaviors/behavior_sticky_key.c

@ -127,9 +127,13 @@ static int on_sticky_key_binding_pressed(struct zmk_behavior_binding *binding, @@ -127,9 +127,13 @@ static int on_sticky_key_binding_pressed(struct zmk_behavior_binding *binding,
struct zmk_behavior_binding_event event) {
struct device *dev = device_get_binding(binding->behavior_dev);
const struct behavior_sticky_key_config *cfg = dev->config_info;
struct active_sticky_key *sticky_key =
store_sticky_key(event.position, binding->param1, binding->param2, cfg);
struct active_sticky_key *sticky_key;
sticky_key = find_sticky_key(event.position);
if (sticky_key != NULL) {
stop_timer(sticky_key);
release_sticky_key_behavior(sticky_key, event.timestamp);
}
sticky_key = store_sticky_key(event.position, binding->param1, binding->param2, cfg);
if (sticky_key == NULL) {
LOG_ERR("unable to store sticky key, did you press more than %d sticky_key?",
ZMK_BHV_STICKY_KEY_MAX_HELD);

1
app/tests/sticky-keys/9-sk-dn-up-dn-up/events.patterns

@ -0,0 +1 @@ @@ -0,0 +1 @@
s/.*hid_listener_keycode_//p

4
app/tests/sticky-keys/9-sk-dn-up-dn-up/keycode_events.snapshot

@ -0,0 +1,4 @@ @@ -0,0 +1,4 @@
pressed: usage_page 0x07 keycode 0xe1 mods 0x00
released: usage_page 0x07 keycode 0xe1 mods 0x00
pressed: usage_page 0x07 keycode 0xe1 mods 0x00
released: usage_page 0x07 keycode 0xe1 mods 0x00

27
app/tests/sticky-keys/9-sk-dn-up-dn-up/native_posix.keymap

@ -0,0 +1,27 @@ @@ -0,0 +1,27 @@
#include <dt-bindings/zmk/keys.h>
#include <behaviors.dtsi>
#include <dt-bindings/zmk/kscan-mock.h>
/ {
keymap {
compatible = "zmk,keymap";
label ="Default keymap";
default_layer {
bindings = <
&sk LEFT_SHIFT &none
&none &none
>;
};
};
};
&kscan {
events = <
ZMK_MOCK_PRESS(0,0,10)
ZMK_MOCK_RELEASE(0,0,10)
/* the sticky key is pressed again, so the previous one must be cancelled */
ZMK_MOCK_PRESS(0,0,10)
ZMK_MOCK_RELEASE(0,0,1200)
>;
};
Loading…
Cancel
Save