Skip to content

inlineChat: remove gutter affordance experiment#308512

Merged
jrieken merged 1 commit intomainfrom
joh/remove-gutter-affordance
Apr 8, 2026
Merged

inlineChat: remove gutter affordance experiment#308512
jrieken merged 1 commit intomainfrom
joh/remove-gutter-affordance

Conversation

@jrieken
Copy link
Copy Markdown
Member

@jrieken jrieken commented Apr 8, 2026

No description provided.

Copilot AI review requested due to automatic review settings April 8, 2026 14:03
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the “gutter” inline chat affordance experiment, leaving only the editor-position affordance option in the inline chat configuration and implementation.

Changes:

  • Removes 'gutter' from the inlineChat.affordance setting enum/options.
  • Deletes the gutter affordance implementation and wires InlineChatAffordance to only use the editor affordance.
  • Updates telemetry mode description to no longer reference gutter.
Show a summary per file
File Description
src/vs/workbench/contrib/inlineChat/common/inlineChat.ts Removes 'gutter' from the affordance setting enum/description.
src/vs/workbench/contrib/inlineChat/browser/inlineChatGutterAffordance.ts Removes the gutter affordance implementation entirely.
src/vs/workbench/contrib/inlineChat/browser/inlineChatAffordance.ts Drops gutter-mode handling and only instantiates/observes the editor affordance.

Copilot's findings

  • Files reviewed: 3/3 changed files
  • Comments generated: 2

Comment thread src/vs/workbench/contrib/inlineChat/browser/inlineChatAffordance.ts
Comment thread src/vs/workbench/contrib/inlineChat/browser/inlineChatAffordance.ts
@jrieken jrieken merged commit 82030bd into main Apr 8, 2026
27 checks passed
@jrieken jrieken deleted the joh/remove-gutter-affordance branch April 8, 2026 14:56
@vs-code-engineering vs-code-engineering Bot added this to the 1.116.0 milestone Apr 8, 2026
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.

3 participants