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

1.37.1 integrated terminal keyboard input repeat delay has increased from 1.36.1 #80228

Closed
henrik-jensen opened this issue Sep 2, 2019 · 9 comments
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s)

Comments

@henrik-jensen
Copy link

  • VSCode Version: 1.37.1
  • OS Version: Microsoft Windows [Version 10.0.18362.295]

Steps to Reproduce:

  1. Install fresh portable zip versions of 1.36.1 and 1.37.1 with data dir in install root
  2. Set Repeat delay/rate to Short/Fast as in illustration:
    image
  3. Start New terminal in both versions.
  4. Press and hold a key, say 'a', and notice the delay of the repeated 'a's in 1.37.1 vs. 1.36.1.

Does this issue occur when all extensions are disabled?: Yes

@Tyriar
Copy link
Member

Tyriar commented Sep 6, 2019

/duplicate xtermjs/xterm.js#469 (comment)

@Tyriar Tyriar closed this as completed Sep 6, 2019
@vscodebot vscodebot bot added the *duplicate Issue identified as a duplicate of another issue(s) label Sep 6, 2019
@henrik-jensen
Copy link
Author

henrik-jensen commented Sep 7, 2019

/duplicate xtermjs/xterm.js#469 (comment)

This is a duplicate of adding emoji to the terminal, seriously?!!!
Anyway are emojis really more important than a responsive input?

@Tyriar
Copy link
Member

Tyriar commented Sep 7, 2019

The root cause is that the terminal listens to keydown, not input, the same reason emoji IMEs don't work.

@henrik-jensen
Copy link
Author

The root cause is that the terminal listens to keydown, not input, the same reason emoji IMEs don't work.

Fair enough. But why the "regression" in delay from 1.36.1, or put in another way,- can't we wait to incorporate those newer versions of xterm.js until these problems are fixed? ( as the delay problem was not present in 1.36.1)?

@Tyriar
Copy link
Member

Tyriar commented Sep 9, 2019

Both 1.38 and 1.36 have the same delay for me when I tweak that setting on Windows so I'm not sure why it's differing for you. It might be that the terminal is laggier because of an Electron update or something, does it still happen if you change terminal.integrated.rendererType to "dom"?

@henrik-jensen
Copy link
Author

Yes,- both dom, canvas and auto gives the delay. It's like the keys are sticky but then suddenly catches up and fires all the keys pressed. (This is 1.37.1 vs 1.36.1. I see there's an 1.38 update. I'll try that and report back)

@henrik-jensen
Copy link
Author

At first I thought 1.38 had fixed the problem, but then I started a cmd shell from the ps prompt and the stickiness, returned, also when exiting the cmd prompt to return to the PS prompt. This behavior is also present in 1.37.1 but not in 1.36.1.

@henrik-jensen
Copy link
Author

It seems especially noticeable when a subshell (cmd and wsl) has been started and stays when finished.

@Tyriar
Copy link
Member

Tyriar commented Sep 12, 2019

This might be one of the performance problems we've seen in Windows, you're probably seeing this one #74620

@vscodebot vscodebot bot locked and limited conversation to collaborators Oct 21, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
*duplicate Issue identified as a duplicate of another issue(s)
Projects
None yet
Development

No branches or pull requests

2 participants