Skip to content

Support color override for sessions#304479

Merged
benibenj merged 2 commits intomainfrom
benibenj/amateur-marlin
Mar 24, 2026
Merged

Support color override for sessions#304479
benibenj merged 2 commits intomainfrom
benibenj/amateur-marlin

Conversation

@benibenj
Copy link
Contributor

Copilot Generated Description: Introduce a method to update the default color of a color identifier in the color registry. Override the panel background color to use the editor background in the sessions window.

Copilot AI review requested due to automatic review settings March 24, 2026 14:55
@benibenj benibenj enabled auto-merge March 24, 2026 14:55
@benibenj benibenj self-assigned this Mar 24, 2026
@vs-code-engineering vs-code-engineering bot added this to the 1.114.0 milestone Mar 24, 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

Adds a mechanism to override the default value of an existing theme color contribution, and uses it in the Agent Sessions window to make the panel background follow the editor background.

Changes:

  • Extend the color registry API with updateDefaultColor to mutate defaults of an existing color contribution.
  • In sessions theme initialization, override PANEL_BACKGROUND to default to editorBackground.

Reviewed changes

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

File Description
src/vs/sessions/common/theme.ts Applies a sessions-specific default override so panel background tracks the editor background.
src/vs/platform/theme/common/colorUtils.ts Adds IColorRegistry.updateDefaultColor and implements it in ColorRegistry.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@benibenj benibenj merged commit 3582df3 into main Mar 24, 2026
20 checks passed
@benibenj benibenj deleted the benibenj/amateur-marlin branch March 24, 2026 15:18
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.

4 participants