qmk / qmk_firmware

Open-source keyboard firmware for Atmel AVR and Arm USB families
https://qmk.fm
GNU General Public License v2.0
18.28k stars 39.39k forks source link

Space Cadet shift starts with extra Shift up and down, and frequently double-taps #5780

Closed petergaultney closed 5 years ago

petergaultney commented 5 years ago

Describe the Bug

At my last compilation in January, Space Cadet Shift worked perfectly. The event output from my keyboard for a single Space Cadet shift tap looked like this:

Screenshot 2019-05-04 09 49 06

Now event output for a single keystroke (my tapping term is still 200) looks like this:

Screenshot 2019-05-04 09 54 01 short

Note the extra Shift down/up before even starting the combination, and note that it then repeats itself twice despite there being only a single tap.

System Information

XScorpion2 commented 5 years ago

Hmm, ya, I can add a check if the tap & hold mods are the same to skip the extra up and down in this case. The double tap might be due to debounce, perf has been increased quite a bit and @drashna was saying he needed to increase debounce time because of this.

XScorpion2 commented 5 years ago

Here ya go: https://github.com/qmk/qmk_firmware/pull/5781

skorokithakis commented 4 years ago

@petergaultney Which application did you use to generate those keystroke logs?

petergaultney commented 4 years ago

Karabiner Elements on Mac. https://pqrs.org/osx/karabiner/

skorokithakis commented 4 years ago

Thank you.