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

Test decorations on gutter seem to have low contrast in Light High Contrast Theme #143835

Closed
luabud opened this issue Feb 24, 2022 · 6 comments
Closed
Assignees
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Milestone

Comments

@luabud
Copy link
Member

luabud commented Feb 24, 2022

Testing #143425

If you open the test explorer and run tests that fail, the error icons on the gutter seem to have low contrast:

image

Same for when the tests pass:

image

Here's a repo that has tests enabled and configured: https://github.com/luabud/sample_project

@daviddossett daviddossett added this to the March 2022 milestone Feb 24, 2022
@daviddossett daviddossett added accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug labels Feb 24, 2022
@daviddossett
Copy link
Contributor

Matched the colors used for success/error in other areas of the light HC theme.

For some reason it looks like these colors get somewhat dimmed in certain cases. Doesn't look related to this theme but I'll dig a little deeper and see if I can find out what's happening.

CleanShot 2022-03-07 at 10 17 31@2x

@daviddossett
Copy link
Contributor

daviddossett commented Mar 7, 2022

Subtle, but here's an example. It's ever-so-slightly faded before the tests have been run. @connor4312 does this sound familiar at all?

Before running tests

CleanShot 2022-03-07 at 10 19 15@2x

After running tests

CleanShot 2022-03-07 at 10 21 15@2x

@connor4312
Copy link
Member

Yes, tests run before the current window are 'outdated' and are faded slightly. There used to be way for extensions to mark tests as outdated, but this is no longer a thing, so we could get rid of the outdated notion.

@daviddossett
Copy link
Contributor

Yes, tests run before the current window are 'outdated' and are faded slightly. There used to be way for extensions to mark tests as outdated, but this is no longer a thing, so we could get rid of the outdated notion.

Ok great, thanks. Looks like removing this did the trick. I can push this if that's all to consider here:

.test-explorer .computed-state.retired,
.testing-run-glyph.retired {
opacity: 0.7 !important;
}

@connor4312
Copy link
Member

Please leave it for now, I'll clean it up as part of #144618

@daviddossett
Copy link
Contributor

Sounds good, thank you!

@aeschli aeschli added the verified Verification succeeded label Mar 24, 2022
@github-actions github-actions bot locked and limited conversation to collaborators Apr 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues bug Issue identified by VS Code Team member as probable bug insiders-released Patch has been released in VS Code Insiders verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

5 participants
@connor4312 @aeschli @daviddossett @luabud and others