Skip to content

Prevent symbol-* codicons from displaying colored on toolbars (fix #267766)#267787

Merged
jrieken merged 18 commits intomicrosoft:mainfrom
gjsjohnmurray:fix-267766
Jan 30, 2026
Merged

Prevent symbol-* codicons from displaying colored on toolbars (fix #267766)#267787
jrieken merged 18 commits intomicrosoft:mainfrom
gjsjohnmurray:fix-267766

Conversation

@gjsjohnmurray
Copy link
Contributor

This PR fixes #267766

Copilot AI review requested due to automatic review settings September 22, 2025 14:49
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 fixes an issue where symbol icons (codicon-symbol-*) were displaying with colors in toolbars when they should remain uncolored. The fix prevents these symbol icons from applying their themed colors when they appear within toolbar contexts.

  • Adds :not(.monaco-toolbar .codicon.codicon-symbol-*) exclusions to all symbol icon color rules
  • Ensures symbol icons in toolbars maintain default appearance without themed coloring
  • Preserves existing colored symbol icon behavior in non-toolbar contexts

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

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

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

@gjsjohnmurray
Copy link
Contributor Author

To verify, use the repro steps at #267766 (comment) and confirm that the button is no longer colored.

@gjsjohnmurray
Copy link
Contributor Author

Please consider taking this simple bugfix PR in time for next week's endgame.

@gjsjohnmurray
Copy link
Contributor Author

Please approve and merge this.

Copy link
Member

@jrieken jrieken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we do it, we should have better CSS rules here, but lets see how the discussion about the "if" goes

Copy link
Member

@jrieken jrieken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(pressed Approve when I wanted changes)

@vs-code-engineering
Copy link

vs-code-engineering bot commented Dec 18, 2025

📬 CODENOTIFY

The following users are being notified based on files changed in this PR:

@bpasero

Matched files:

  • src/vs/workbench/browser/parts/views/treeView.ts

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

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

@jrieken jrieken added this to the January 2026 milestone Jan 6, 2026
@gjsjohnmurray gjsjohnmurray requested a review from jrieken January 13, 2026 18:25
@jrieken jrieken modified the milestones: January 2026, February 2026 Jan 30, 2026
Copy link
Member

@jrieken jrieken left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@jrieken jrieken enabled auto-merge January 30, 2026 12:00
@jrieken jrieken merged commit 0da4260 into microsoft:main Jan 30, 2026
17 checks passed
@gjsjohnmurray gjsjohnmurray deleted the fix-267766 branch January 30, 2026 20:34
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.

symbol-* icons contributed to toolbars by extensions unexpectedly display colored

5 participants