Skip to content

Fix visual editor response to quickSuggestion setting for new default - #1216

Open
vezwork wants to merge 1 commit into
mainfrom
bugfix/quickSuggestionsSetting
Open

vezwork wants to merge 1 commit into
mainfrom
bugfix/quickSuggestionsSetting

Conversation

@vezwork

@vezwork vezwork commented Nov 3, 2026 •

Copy link
Copy Markdown
Member

VSCode changed its default setting for editor.quickSuggestions to offWhenInlineCompletions and the Visual Editor was not checking the value correctly, causing snippets to not show up in the visual editor when they should.

This PR makes the VisualEditor count more values of editor.quickSuggestions to count as enabled, including editor.quickSuggestions.

@vezwork
vezwork requested a review from juliasilge November 3, 2026 18:44
@vezwork

vezwork commented Nov 3, 2026

Copy link
Copy Markdown
Member Author

@juliasilge please take a very brief glance at this. I would almost just merge it but am curious if any thoughts occur to you about settings stuff like this.

@juliasilge juliasilge left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this looks good! Can we make the changelog entry more clear, though?

Comment thread apps/vscode/CHANGELOG.md

## 1.140.0 (Unreleased)

- In the visual editor, fix completions to count as enabled when the `editor.quickSuggestions` is `offWhenInlineCompletions`, which is the new default as of VS Code 1.121.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we make this more clear?

Suggested change
- In the visual editor, fix completions to count as enabled when the `editor.quickSuggestions` is `offWhenInlineCompletions`, which is the new default as of VS Code 1.121.
- In the visual editor, code cells show completions and snippets automatically again while you type. Previously, they did not show when `editor.quickSuggestions` used its new default value, `offWhenInlineCompletions` (<https://github.com/quarto-dev/quarto/pull/1216>).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants