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

acceptSuggestionOnCommitCharacter with ghost text #134040

Open
TylerLeonhardt opened this issue Sep 28, 2021 · 2 comments
Open

acceptSuggestionOnCommitCharacter with ghost text #134040

TylerLeonhardt opened this issue Sep 28, 2021 · 2 comments
Assignees
Labels
feature-request Request for new features or functionality inline-completions suggest IntelliSense, Auto Complete
Milestone

Comments

@TylerLeonhardt
Copy link
Member

Testing #133907

So I just realized that acceptSuggestionOnCommitCharacter is on by default, this means that when you hit ; at this point:
image
it will accept the completion (not the inline completion):
image

I wanted to call this out because I think it's worth discussing... do you think that ; would accept only the completion or the whole suggestion?

In my eyes, a ; is saying "I wanna commit to this whole statement" and so I might expect that the inline completion would also be accepted. This might be a nice way for a user to accept the completion and the inline completion in one keystroke (if such a thing doesn't exist already).

What do you think @hediet?

@hediet
Copy link
Member

hediet commented Sep 29, 2021

Would the semicolon be added twice then?

@hediet hediet added feature-request Request for new features or functionality inline-completions labels Sep 29, 2021
@hediet hediet added this to the Backlog milestone Sep 29, 2021
@TylerLeonhardt
Copy link
Member Author

I would say no. If there's a ; at the end of an online completion then don't add a 2nd?

@hediet hediet added the suggest IntelliSense, Auto Complete label Apr 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request Request for new features or functionality inline-completions suggest IntelliSense, Auto Complete
Projects
None yet
Development

No branches or pull requests

3 participants