Skip to content

Fix for extension button separator wrong color precedence#293120

Open
EmrecanKaracayir wants to merge 1 commit intomicrosoft:mainfrom
EmrecanKaracayir:emrecankaracayir/extension-button-separator
Open

Fix for extension button separator wrong color precedence#293120
EmrecanKaracayir wants to merge 1 commit intomicrosoft:mainfrom
EmrecanKaracayir:emrecankaracayir/extension-button-separator

Conversation

@EmrecanKaracayir
Copy link
Copy Markdown
Contributor

Fixes: #293118

Copilot AI review requested due to automatic review settings February 5, 2026 15:13
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 the CSS variable precedence for the extension button separator color. The issue was that when both extensionButton.border and extensionButton.separator theme tokens were customized, the separator was incorrectly using the border color instead of the separator color.

Changes:

  • Swapped the CSS variable fallback order in extensionActions.css to prioritize extensionButton.separator over extensionButton.border

@mrleemurray
Copy link
Copy Markdown
Contributor

Thanks @EmrecanKaracayir !

@vs-code-engineering vs-code-engineering bot added this to the February 2026 milestone Feb 6, 2026
@mjbvz
Copy link
Copy Markdown
Collaborator

mjbvz commented Mar 5, 2026

Moving this to 1.112 and disabling auto merge so that we can get the 1.111 milestone into good shape by the end of Thursday

Please take this in as a candidate if it must go into 1.111

@mrleemurray
Copy link
Copy Markdown
Contributor

Waiting till 1.112

@mjbvz mjbvz modified the milestones: 1.112.0, 1.113.0 Mar 17, 2026
@jrieken jrieken modified the milestones: 1.113.0, On Deck Mar 25, 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.

Extension button separator color wrong precedence

6 participants