Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Icons in problems view show black when not focused #174487

Merged
merged 3 commits into from Feb 15, 2023

Conversation

aeschli
Copy link
Contributor

@aeschli aeschli commented Feb 15, 2023

Wraps the severity icon in markers view in a container node that gets the severity icon color as foreground color.
That way, if the icon color of the list view entry is not set, the color of container is used.
That also works also with colors set as css variable. If a color is not set, the variable in invalid and inherit is used as color.
inherit means using the container color.

FYI @joaomoreno

Fixes #173148

@aeschli aeschli self-assigned this Feb 15, 2023
@aeschli aeschli enabled auto-merge (squash) February 15, 2023 18:25
@VSCodeTriageBot VSCodeTriageBot added this to the February 2023 milestone Feb 15, 2023
bpasero
bpasero previously approved these changes Feb 15, 2023
@aeschli aeschli merged commit dc619de into main Feb 15, 2023
1 check passed
@aeschli aeschli deleted the aeschli/dreadful-orca-779 branch February 15, 2023 21:34
c-claeys pushed a commit to c-claeys/vscode that referenced this pull request Feb 16, 2023
* Icons in problems view show black when not focused

* update smoke test
@quyenvsp
Copy link

@aeschli

This merge make icon in Problems as Table View always black?

I think it should be .markers-panel .marker-icon.codicon-error,,... not .markers-panel .marker-icon.error,,...

2023-03-24_163003

@github-actions github-actions bot locked and limited conversation to collaborators Apr 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Icons in problems view show black when not focused
5 participants