Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Repetition when Pressing Function Keys #60

Closed
MarioRicalde opened this issue Sep 18, 2013 · 12 comments
Closed

Repetition when Pressing Function Keys #60

MarioRicalde opened this issue Sep 18, 2013 · 12 comments
Labels

Comments

@MarioRicalde
Copy link

If you use Fast "Key Repeat" and Short "Delay Until Repeat" like in this image:

You will randomly get repetitions for letters and backspaces, to reproduce try the following;

  1. Press any letter OR backspace and immediately press FN.
  2. See it repeat several times.

Example of what you should expect to see:

aaaaa
ssss
eeee
rrrrr
tttt
yyyyyy
uuuuuu
pppppp

@MarioRicalde
Copy link
Author

This definitely has to do with TAPPING_TERM, lowered it and the problem went away. It's definitely an issue, though.

@tmk
Copy link
Owner

tmk commented Sep 18, 2013

Ubuntu has keyboard setting dialog like this.
image

WIth this crazy fastest repeat and shortest delay I can't type at all with both tmk controller and stock controller(Topre).
In my case FN is not related to this. It doesn't seem your issue comes from tmk controller.

I'm not sure exactly what your problem is. Your problem may come from your keymap. Post you keymap.c.

Are you sure that you can type collectly with this extream setting with your other keyboard? Which keyboard do you use?

BTW, do you really need this extreme repeating setting? :)

This is what I got with the setting.
aaaaaaaaaaaaaaaaaa
sssssssssssss
dddddddddddddddddddd
fffffffffffffff
gggggggggggg
hhhhhhhhhh
iiiiiiiiiiiiii ccccccccccccccccccccccaaaaaaaaaaaaaaaaannnnnnnnnnnnnnnnn''''''''''''''''''''''''' tttttttttt tttttttttttttttyyyyyyyyyyyyyyyyppppppppppppppppppppppeeeeeeeeeeeeeeeeee!!!!!!!!!!!!!!!!!!!!!!!!

@MarioRicalde
Copy link
Author

In the case of Mac OSX I don't think that it's that crazy fast.

I'm using a HHKB using the default keymap.c that in this repo. The problem happens only when you type any letter and right after that a FN key. It also happens with backspace, which makes it somewhat annoying when you're using shift as a function key.

The way I noticed this issue was by quickly typing "hello!", where the exclamation mark turns the text output into: "helloooooo!".

@MarioRicalde
Copy link
Author

Steps to reproduce:

Type "hello" immediately followed by "Shift + 1 ( ! )".
Expected result: hello!
Actual result: helloooo!

Type "a" immediately followed by pressing and holding FN button on the HHKB.
Expected result: a
Actual result: aaaa

Hit BACKSPACE followed by the HHKB FN key in the following string: "HelloWorld"
Expected result: "HelloWorl"
Actual result: "He"

@tmk
Copy link
Owner

tmk commented Sep 18, 2013

Ah, I finally got this problem. Thanks for your patience and explanation.
It is exactly a problem on tapping code. I'll look into this problem later.

I can't believe I didn't catch this anonying bug for long time. Maybe because I usually use long delay and slow repeating rate.

Plz use very long delay(around 500ms?) until this is fixed. Long delay will mitigate this problem, I think.

@MarioRicalde
Copy link
Author

I virtualised Ubuntu and got it to fail similarly to my Mac OSX System:

@MarioRicalde
Copy link
Author

@tmk thank you! expect me to be helping more with the documentation, I plan to use this every day !

Kudos!

@tmk
Copy link
Owner

tmk commented Sep 18, 2013

I pushed fix to new branch 'tapping_fix60'. Can you try this code?
https://github.com/tmk/tmk_keyboard/tree/tapping_fix60
This is the fix commit.
4d0b3aa

I'll merge this into master branch after some test run.

Bug description:

Tap key prevetns processing key release event till settlement of tapping(TAPPING_TERM(200ms) at worst). As the result of this unexpected repeating occurs even if the key is alreadly released.

key sequence:

key ~~~~_____~~~~~~~~~~                   
tap ~~~~~~~_______~~~~~                   
          |--------|  TAPPING_TERM

This bug apears when you press Tap key druing pressing a normal key, especially using OS keyboard setting with relatively short 'Delay'(< 200ms) and fast 'Rate/Speed'( > 30strokes/min).

Workaround

Longer Repeat Dealy than TAPPING_TERM(200ms) should help.

@MarioRicalde
Copy link
Author

It works!

I just tested everything and it's working perfectly now.

どうも 有難う 御座います.

@MarioRicalde
Copy link
Author

Hello! I've been using this for a whole day of work ( typed a lot ! ) and I got no errors at all!

Kudos!

@MarioRicalde
Copy link
Author

I think it's ready for merge into master?

@tmk
Copy link
Owner

tmk commented Sep 19, 2013

I also didn't got problem so far. I merged into master branch now. 58ca7b9

Thanks.

@tmk tmk closed this as completed Sep 19, 2013
tufty pushed a commit to tufty/tmk_keyboard that referenced this issue Nov 4, 2013
- See tmk#60
- **Except for modifiers** a key pressed before the tapping starts should be released immediately
- 'Mod-Tap key'(like shift-;) didn't work from this fix: 4d0b3aa Fix Tapping: release of a key pressed before tap
This key sequence should register ':', not ';'. With the fix Shift is
released before settlement of tapping, this registers ';'.
    Shift   ~~___~~~~~~~
    ;(Tap)  ~~~____~~~~
kairyu pushed a commit to kairyu/tmk_keyboard_custom that referenced this issue May 21, 2015
- See tmk/tmk_keyboard#60
- **Except for modifiers** a key pressed before the tapping starts should be released immediately
- 'Mod-Tap key'(like shift-;) didn't work from this fix: 4d0b3aa Fix Tapping: release of a key pressed before tap
This key sequence should register ':', not ';'. With the fix Shift is
released before settlement of tapping, this registers ';'.
    Shift   ~~___~~~~~~~
    ;(Tap)  ~~~____~~~~
kairyu pushed a commit to kairyu/tmk_keyboard_custom that referenced this issue May 22, 2015
- See tmk/tmk_keyboard#60
- **Except for modifiers** a key pressed before the tapping starts should be released immediately
- 'Mod-Tap key'(like shift-;) didn't work from this fix: 4d0b3aa Fix Tapping: release of a key pressed before tap
This key sequence should register ':', not ';'. With the fix Shift is
released before settlement of tapping, this registers ';'.
    Shift   ~~___~~~~~~~
    ;(Tap)  ~~~____~~~~
ddacunha pushed a commit to ddacunha/tmk_keyboard that referenced this issue Jan 7, 2016
added dvorak keymap and layout picture
alex-ong added a commit to alex-ong/tmk_keyboard that referenced this issue Jan 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants