Skip to content

Comments

chat - hide outline on chat response except for HC themes#297075

Merged
bpasero merged 3 commits intomainfrom
ben/whole-goldfish
Feb 23, 2026
Merged

chat - hide outline on chat response except for HC themes#297075
bpasero merged 3 commits intomainfrom
ben/whole-goldfish

Conversation

@bpasero
Copy link
Member

@bpasero bpasero commented Feb 23, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 23, 2026 19:18
@bpasero bpasero enabled auto-merge (squash) February 23, 2026 19:18
@bpasero bpasero self-assigned this Feb 23, 2026
daviddossett
daviddossett previously approved these changes Feb 23, 2026
@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 23, 2026
Copy link
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

Adjusts chat widget styling so focused chat response rows don’t show the default list focus outline in non–high-contrast themes, aligning the chat UI’s focus visuals with the intended design while preserving HC visibility.

Changes:

  • Add a CSS rule to suppress the focused outline on response rows except in HC themes.
  • Tag list rows as response in the chat list renderer so styling can target response items specifically.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
src/vs/workbench/contrib/chat/browser/widget/media/chat.css Adds a non-HC rule to remove the focused outline on response list rows.
src/vs/workbench/contrib/chat/browser/widget/chatListRenderer.ts Adds a response class toggle on the list row container to enable response-specific styling.

@bpasero bpasero merged commit 0552a36 into main Feb 23, 2026
20 checks passed
@bpasero bpasero deleted the ben/whole-goldfish branch February 23, 2026 20:33
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