Skip to content

Comments

Align expanded chat entry border-radius with cornerRadius-medium#297058

Merged
daviddossett merged 1 commit intomainfrom
daviddossett/align-chat-border-radius
Feb 23, 2026
Merged

Align expanded chat entry border-radius with cornerRadius-medium#297058
daviddossett merged 1 commit intomainfrom
daviddossett/align-chat-border-radius

Conversation

@daviddossett
Copy link
Collaborator

Replace hardcoded 4px/6px border-radius values in expanded chat entries with var(--vscode-cornerRadius-medium) to align with codeblocks, markdown tables, and chat-confirmation-widget2.

Changes

Component File Before After
Thinking collapsible list chatThinkingContent.css 4px var(--vscode-cornerRadius-medium)
Terminal content title chatTerminalToolProgressPart.css 4px var(--vscode-cornerRadius-medium)
Terminal output container chatTerminalToolProgressPart.css 4px var(--vscode-cornerRadius-medium)
Confirmation widget chatConfirmationWidget.css 4px var(--vscode-cornerRadius-medium)
Confirmation message container chatConfirmationWidget.css 4px var(--vscode-cornerRadius-medium)
Confirmation message (tool invocation) chatConfirmationWidget.css 4px var(--vscode-cornerRadius-medium)
Tool output part chat.css 6px var(--vscode-cornerRadius-medium)

Replace hardcoded 4px/6px border-radius values in chat expanded entries
(thinking collapsible, terminal content, confirmation widgets, tool output)
with var(--vscode-cornerRadius-medium) to match codeblocks, markdown tables,
and confirmation widget2.
Copilot AI review requested due to automatic review settings February 23, 2026 18:39
@daviddossett daviddossett marked this pull request as ready for review February 23, 2026 18:42
@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

This PR standardizes border-radius values across chat entry components by replacing hardcoded pixel values (4px and 6px) with the CSS variable var(--vscode-cornerRadius-medium) which is defined as 6px. This aligns the visual appearance of thinking lists, terminal content, confirmation widgets, and tool output with existing chat UI elements like codeblocks, markdown tables, and the chat-confirmation-widget2.

Changes:

  • Replaced all hardcoded 4px and 6px border-radius values with var(--vscode-cornerRadius-medium) in chat entry components
  • Ensures visual consistency across all expanded chat entry elements (thinking lists, terminal content, tool output, confirmation widgets)
  • Components previously using 4px will now render with 6px border-radius for better alignment with codeblocks and tables

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
chat.css Updated tool-output-part border-radius from 6px to use CSS variable
chatThinkingContent.css Updated thinking collapsible list border-radius from 4px to use CSS variable
chatTerminalToolProgressPart.css Updated terminal content title and output container border-radius from 4px to use CSS variable
chatConfirmationWidget.css Updated confirmation widget, message container, and tool invocation message border-radius from 4px to use CSS variable

@daviddossett daviddossett merged commit b204273 into main Feb 23, 2026
24 checks passed
@daviddossett daviddossett deleted the daviddossett/align-chat-border-radius branch February 23, 2026 19:16
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