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

suggestion commit character evaluated before suggestions updated #68273

Closed
mgabeler-lee-6rs opened this issue Feb 8, 2019 · 3 comments
Closed
Assignees
Labels
*duplicate Issue identified as a duplicate of another issue(s)

Comments

@mgabeler-lee-6rs
Copy link

Issue Type: Bug

I've noticed since the 1.31.0 update some interaction with one of my extensions has made the suggestions popup slow to update (if I disable all extensions, it is not slow, but I have not figured out which extension or combination is causing the slowness).

However, with the slowness to update, it has exposed an underlying issue:

When I press a commit character, it commits whatever suggestion is currently at the top. It does not wait for the suggestions to update with the text I've typed, so it can cause lots of good text to be discarded in favor of an obsolete suggestion.

For example, typing up a bunch of mocha tests using chancejs, I'm seeing:

  • context('console('
  • chance.chai.
  • chance.date(chance.d10(

And so on. Others are even worse, but these are simple and easy to make sense of without tons of context from my code.

VS Code version: Code 1.31.0 (7c66f58, 2019-02-06T08:51:24.856Z)
OS version: Linux x64 4.19.0-1-amd64

System Info
Item Value
CPUs Intel(R) Core(TM) i7-4770HQ CPU @ 2.20GHz (8 x 2519)
GPU Status 2d_canvas: enabled
checker_imaging: disabled_off
flash_3d: enabled
flash_stage3d: enabled
flash_stage3d_baseline: enabled
gpu_compositing: enabled
multiple_raster_threads: enabled_on
native_gpu_memory_buffers: disabled_software
rasterization: disabled_software
surface_synchronization: enabled_on
video_decode: unavailable_off
webgl: enabled
webgl2: enabled
Load (avg) 4, 4, 4
Memory (System) 15.55GB (2.14GB free)
Process Argv --unity-launch
Screen Reader no
VM 0%
Extensions (17)
Extension Author (truncated) Version
ng-template Ang 0.1.11
markdown-preview-github-styles bie 0.1.4
bracket-pair-colorizer-2 Coe 0.0.26
vscode-markdownlint Dav 0.23.0
vscode-eslint dba 1.8.0
EditorConfig Edi 0.12.8
vscode-npm-script eg2 0.3.5
beautify Hoo 1.4.8
bash-ide-vscode mad 1.3.3
terraform mau 1.3.7
Go ms- 0.9.1
debugger-for-chrome msj 4.11.1
vscode-docker Pet 0.5.2
vscode-nginx sha 0.6.0
html-preview-vscode tht 0.2.5
reflow-paragraph Tro 1.3.0
vscode-proto3 zxh 0.2.2

(1 theme extensions excluded)

@vscodebot vscodebot bot added the new release label Feb 8, 2019
@mgabeler-lee-6rs
Copy link
Author

mgabeler-lee-6rs commented Feb 8, 2019

Loosely related to: #67698

Disabling the editor.acceptSuggestionOnCommitCharacter partially works around this. I say partial because:

  • It removes a useful feature instead of fixing it
  • The <enter> key seems to not be affected by that setting, so the bug still sometimes impacts me even with that setting disabled, as I can sometimes type half a dozen or more characters, before the suggestions update (the suggestion lag is bad and I can type pretty fast).

@jrieken
Copy link
Member

jrieken commented Feb 11, 2019

/duplicate of #66868

@vscodebot vscodebot bot added the *duplicate Issue identified as a duplicate of another issue(s) label Feb 11, 2019
@vscodebot
Copy link

vscodebot bot commented Feb 11, 2019

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for existing issues here. See also our issue reporting guidelines.

Happy Coding!

@vscodebot vscodebot bot closed this as completed Feb 11, 2019
@vscodebot vscodebot bot locked and limited conversation to collaborators Mar 28, 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