Skip to content

fix: add keybinding expression for hiding inline suggestions#308187

Merged
jrieken merged 2 commits intomainfrom
joh/shocked-bee
Apr 8, 2026
Merged

fix: add keybinding expression for hiding inline suggestions#308187
jrieken merged 2 commits intomainfrom
joh/shocked-bee

Conversation

@jrieken
Copy link
Copy Markdown
Member

@jrieken jrieken commented Apr 7, 2026

fixes #306706

@jrieken jrieken requested review from Copilot and hediet April 7, 2026 10:49
@jrieken jrieken enabled auto-merge (squash) April 7, 2026 10:49
@jrieken jrieken self-assigned this Apr 7, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 7, 2026

Screenshot Changes

Base: 9f457435 Current: 8b714501

Changed (2)

editor/suggestWidget/MethodCompletions/Dark
Before After
before after
editor/suggestWidget/MethodCompletions/Light
Before After
before after

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 fixes an Escape keybinding conflict where editor.action.inlineSuggest.hide was being invoked while inline chat was open, preventing Escape from closing inline chat when ghost text is visible.

Changes:

  • Restricts the default Escape keybinding for “Hide Inline Suggestion” to only apply when the editor text has focus (EditorContextKeys.editorTextFocus).
Show a summary per file
File Description
src/vs/editor/contrib/inlineCompletions/browser/controller/commands.ts Narrows the Escape keybinding context for hiding inline suggestions so it doesn’t steal Escape outside the editor text focus (e.g. inline chat UI).

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@jrieken jrieken removed the request for review from hediet April 8, 2026 12:57
@jrieken jrieken merged commit 71404a1 into main Apr 8, 2026
24 checks passed
@jrieken jrieken deleted the joh/shocked-bee branch April 8, 2026 13:08
@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.

Escape doesn't hide inline chat v3 if ghost text is visible

3 participants