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

Contrast issues #3

Closed
phil294 opened this issue Jan 1, 2022 · 4 comments
Closed

Contrast issues #3

phil294 opened this issue Jan 1, 2022 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@phil294
Copy link
Contributor

phil294 commented Jan 1, 2022

Hi! Thanks for this extension. I noticed that in the search bar, when you select text, it is not readable anymore:

image

Text color should probably be white. I quickly looked into this and it seems that there is no such setting as selection.foreground?? So is there even any way to solve this? If not, I'd suggest setting selection.background to a light grey color instead the dark blue :-/

Another issue: The action buttons on the top right are in the wrong color:

image

There are actually multiple icons in the top row, but you can barely see them. (GitLens icons, mostly). I also looked into this: Currently, dev tools say

.hc-black .monaco-action-bar .action-item.menu-entry .action-label, .vs-dark .monaco-action-bar .action-item.menu-entry .action-label {
    background-image: var(--menu-entry-icon-dark);
}

If you change this to --meny-entry-icon-light, then it gets readable:
image

VSCode seems to apply these flags based on the fact that win-nt-theme is marked as a dark theme. Can this be changed to be a light theme? Would also make more sense imo.

@manekinekko manekinekko self-assigned this Jan 3, 2022
@manekinekko manekinekko added the bug Something isn't working label Jan 3, 2022
@phil294
Copy link
Contributor Author

phil294 commented Jan 3, 2022

The issue for point 2 seems to be in package.json contributes.themes.uiTheme which needs to be "vs" instead of "vs-dark". microsoft/vscode#140006, slack-theme/visual-studio-code#9 (comment), https://code.visualstudio.com/api/references/contribution-points#theme-example.
Cannot find any documentation about this though...

@manekinekko
Copy link
Owner

Hey, thank you for looking at this. Do you want to send a PR to fix this issue?

@phil294
Copy link
Contributor Author

phil294 commented Jan 4, 2022

Hey, thank you for looking at this. Do you want to send a PR to fix this issue?

sure! #4

don't know what to do about the selection color issue though

@manekinekko
Copy link
Owner

manekinekko commented May 7, 2022

Closed by #4 and 6539d31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants