Reinstate colors of symbol codicons when used in custom tree views#320391
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates tree view icon rendering so theme icons without an explicit color get a fallback CSS class applied.
Changes:
- Add
codicon-coloredclass whennode.themeIconhas nocolorconfigured.
|
Validated using the extension-samples branch documented at #299479 (comment)
|
|
@gjsjohnmurray can #311339 be affected by this? (SVG |
|
@gjsjohnmurray did some testing of this PR, maybe I'm doing something wrong, but can't confirm colors fallback to defaults:
Icons assigned as Can be related to the issue - even some |
|
@n-gist is the source of your test extension available somewhere? I'm puzzled that symbol-* codicons aren't colored in the quickpick. |
|
@gjsjohnmurray it is not released yet, but I can share current version. Do you need source code or vsix would be enough? Send it somewhere? |
|
@gjsjohnmurray sent an invite to https://github.com/n-gist/temp.vscode.focus To test icons - open a project and drop some files/folders to the extension view, and their icons will be customizeable.
Open issue in repo for any needs, I would be happy to assist with testing/fixing icons issues. |
|
@n-gist thanks, invitation accepted but I probably won't get time to investigate much before the end of June. |
|
@gjsjohnmurray I ran another test for this PR using https://github.com/alythobani/vscode-extension-samples/tree/show-299479 and can confirm that the color fallback is working. The source of confusion in the previous test was that the color is not visible until the item loses focus. This screencapture confirms that colors work both in the referenced vscode-extension-samples and in my extension. Lol, sorry. |
I have created #321773 for this. |
📬 CODENOTIFYThe following users are being notified based on files changed in this PR: @benibenjMatched files:
|
alexr00
left a comment
There was a problem hiding this comment.
(sorry for the approval and commit noise, needed to test something)



This PR fixes #299479
Issue was caused by #267787