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

"selectQuickSuggestions: off" behavior change request #169830

Closed
gil-koifman opened this issue Dec 22, 2022 · 5 comments
Closed

"selectQuickSuggestions: off" behavior change request #169830

gil-koifman opened this issue Dec 22, 2022 · 5 comments
Assignees
Labels
feature-request Request for new features or functionality on-testplan suggest IntelliSense, Auto Complete
Milestone

Comments

@gil-koifman
Copy link

Hi,
This request is a follow up for the issue:
#151336

We would like to have selectQuickSuggestions turned off, but with a small behavior change.
If the user pressed the "triggerCharacters", in our case it's "space", then when the suggestion list shows - nothing will be selected. This is implemented today with turning off selectQuickSuggestions.
But if the user started typing something, or started deleting characters from an existing word, then we do want the first option to be selected. In the current behavior, nothing will be selected.

Thanks!

@jrieken jrieken added this to the January 2023 milestone Dec 22, 2022
@jrieken jrieken added the suggest IntelliSense, Auto Complete label Dec 22, 2022
@jrieken
Copy link
Member

jrieken commented Dec 22, 2022

To clarify: like we differentiate between trigger editor.suggestOnTriggerCharacters and editor.quickSuggestions you like to be able to configure select mode for each of these auto-modes independently, right?

So, don't have editor.suggest.selectQuickSuggestions as true|false but as onTriggerCharacter | onQuickSuggest | always | never

@jrieken jrieken added the feature-request Request for new features or functionality label Dec 23, 2022
jrieken added a commit that referenced this issue Dec 23, 2022
…low to differentiate between auto suggestions trigger by typing (aka quick suggest) and by trigger characters

re #139825 and #169830
@jrieken
Copy link
Member

jrieken commented Dec 23, 2022

As discussed offline, this is now implemented with a revised setting: editor.suggest.selectionMode. demo: #139825 (comment)

jrieken added a commit that referenced this issue Dec 23, 2022
…low to differentiate between auto suggestions trigger by typing (aka quick suggest) and by trigger characters (#169933)

re #139825 and #169830
@gil-koifman
Copy link
Author

Looks great! Thank you.

@jrieken jrieken closed this as completed Jan 3, 2023
@jrieken
Copy link
Member

jrieken commented Jan 3, 2023

Thanks for catching up.

@jrieken
Copy link
Member

jrieken commented Jan 23, 2023

test plan covered by #171988

@github-actions github-actions bot locked and limited conversation to collaborators Feb 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality on-testplan suggest IntelliSense, Auto Complete
Projects
None yet
Development

No branches or pull requests

2 participants