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.12.1] editor.acceptSuggestionOnCommitCharacter not being respected #26569

Closed
scriptorum opened this issue May 12, 2017 · 6 comments
Closed
Assignees
Labels
info-needed Issue requires more information from poster suggest IntelliSense, Auto Complete

Comments

@scriptorum
Copy link

scriptorum commented May 12, 2017

I'm working in C#. I like suggestions, but I never want Code to accept them without my explicit say so, such as the tab key. So I turned off acceptSuggestionOnCommitCharacter in my settings.json file:

image

But it still accepts the suggestion when I press . or ).

vscode-issue

Am I misusing this setting?

  • VSCode Version: Code 1.12.1 (f6868fc, 2017-05-04T21:18:32.269Z)
  • OS Version: Darwin x64 16.5.0
  • Extensions:
Extension Author Version
theme-glacier Kal 0.0.1
uncrustify LaurentTreguier 1.5.1
magoon Northerntwig 1.1.0
theme-darktooth Poorchop 0.0.2
unity-debug Unity 1.1.0
nord-visual-studio-code arcticicestudio 0.1.1
Theme-Monokai-Easylight davidbabel 1.0.0
vs-code-xml-format fabianlauer 0.1.5
Theme-Amy gerane 0.0.2
Theme-Bliss gerane 0.0.2
Theme-Brunette gerane 0.0.4
Theme-Colonoscopy gerane 0.0.2
Theme-Earthsong gerane 0.0.2
Theme-FlatlandDark gerane 0.0.2
Theme-Hail gerane 0.0.3
Theme-monokai-inverse gerane 0.0.2
theme-dusk gllwy 0.0.7
theme-dark-starry-night jerry 1.0.2
theme-onedark-sublime joshpeng 0.3.6
FlatUI lkytal 1.2.1
theme-chester marcuskaz 0.1.1
cpptools ms-vscode 0.10.5
csharp ms-vscode 1.8.1
theme-whitehat nisimiko 1.22.6
mac-classic-theme nkjoep 0.0.1
theme-oceandeep pierrenel 0.0.1
ClassyNaming reflectiondm 1.0.0
vscode-format ryannaddy 0.0.6
sblight sbovyrin 2.1.1
dark-plus-material vangware 1.0.0
gitflow vector-of-bool 1.1.0
agila-theme whtsky 2.0.1

Steps to Reproduce:

  1. Turn off acceptSuggestionOnCommitCharacter
  2. Type an identifier
  3. Use . or ) to IGNORE the suggestion
  4. Witness the suggestion being all like I'll do what I want you're not the boss of me
@jrieken jrieken added info-needed Issue requires more information from poster suggest IntelliSense, Auto Complete labels May 15, 2017
@jrieken
Copy link
Member

jrieken commented May 15, 2017

@scriptorum Can you check if you have a keybinding for . or ) that is specific to C# and accepting a completion? Accept on commit used to be "implemented" with a keybinding

@scriptorum
Copy link
Author

scriptorum commented May 15, 2017

I see a binding to . under Keyboard Shortcuts:
image
If I remove it then . no longer accepts suggestions. One down.

As for the closing paren, I don't see ) in Keyboard Shortcuts. Where would I find language-specific bindings?

@jrieken
Copy link
Member

jrieken commented May 16, 2017

Where would I find language-specific bindings?

To right of this there should be a column when listing more information about this. It also contains the language for which it should be active

@scriptorum
Copy link
Author

There it is. Filed under ⇧0 / ⇧9 instead of ./).

image

Removing these shortcuts restored the desired behavior. Thanks @jrieken!

@jrieken
Copy link
Member

jrieken commented May 17, 2017

Thanks. I will follow up with the csharp-folks to use our improved story

@jrieken
Copy link
Member

jrieken commented May 17, 2017

the issue on the omnisharp side: dotnet/vscode-csharp#1487. closing this as a dupe of that one now

@jrieken jrieken closed this as completed May 17, 2017
@vscodebot vscodebot bot locked and limited conversation to collaborators Nov 18, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
info-needed Issue requires more information from poster suggest IntelliSense, Auto Complete
Projects
None yet
Development

No branches or pull requests

2 participants