Skip to content

add better padding for thinking + persistant progress#311603

Merged
justschen merged 2 commits intomainfrom
justin/skrelp
Apr 21, 2026
Merged

add better padding for thinking + persistant progress#311603
justschen merged 2 commits intomainfrom
justin/skrelp

Conversation

@justschen
Copy link
Copy Markdown
Collaborator

Copilot AI review requested due to automatic review settings April 21, 2026 05:13
@justschen justschen enabled auto-merge (squash) April 21, 2026 05:14
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

Adjusts the layout spacing of the “thinking” UI while persistent streaming is active, to avoid the content feeling visually cramped at the bottom during streaming updates.

Changes:

  • Tweaks the bottom spacing for .chat-thinking-persistent-streaming by changing margin-bottom from 0 to 3px.
Show a summary per file
File Description
src/vs/workbench/contrib/chat/browser/widget/chatContentParts/media/chatThinkingContent.css Adds a small bottom margin for persistent streaming state to improve perceived padding/spacing.

Copilot's findings

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

@github-actions
Copy link
Copy Markdown
Contributor

Screenshot Changes

Base: 2f926976 Current: 2c7c5af4

Changed (4)

chat/aiCustomizations/aiCustomizationManagementEditor/McpBrowseMode/Dark
Before After
before after
editor/inlineCompletions/other/JumpToHint/Dark
Before After
before after
agentSessionsViewer/CompletedUnread/Dark
Before After
before after
agentSessionsViewer/CompletedUnread/Light
Before After
before after

@justschen justschen merged commit 938f78a into main Apr 21, 2026
26 checks passed
@justschen justschen deleted the justin/skrelp branch April 21, 2026 05:52
@vs-code-engineering vs-code-engineering Bot added this to the 1.118.0 milestone Apr 21, 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