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

White text on grey background when using light theme #179

Open
noughtmare opened this issue Jan 8, 2024 · 2 comments
Open

White text on grey background when using light theme #179

noughtmare opened this issue Jan 8, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@noughtmare
Copy link

noughtmare commented Jan 8, 2024

I'm one of those weird people using a light theme and with this extension the hyperlinks in goals show up as white text on a light grey background:

image

image

I've tried enabling and disabling the option to use highlighting, but that doesn't seem to change anything.

I observe a similar behavior with any of the standard light themes. The normal text changes between black and white, but the links always stay pure white.

I am using vscodium with this full version info:

Version: 1.85.1
Release: 23348
Commit: 08e6c15293922dd53a864bb041be381322fee401
Date: 2023-12-14T21:19:52.407Z (3 wks ago)
Electron: 25.9.7
ElectronBuildId: undefined
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: 11.4.183.29-electron.0
OS: Darwin x64 23.1.0

And version 0.4.7 of the agda-mode plugin.

@noughtmare
Copy link
Author

noughtmare commented Jan 11, 2024

The issue seems to be that this extension is using the --vscode-button-foreground property which stays completely white even in most light themes, because it expects to be inside a button with a dark background.

The code that needs to be changed is here:

@link : var(--vscode-button-foreground);
@link-hover: var(--vscode-button-foreground);

An easy way to fix it would be to treat holes and links the same and give them both the --vscode-textLink-foreground color, but I don't know if that is acceptable.

In the meantime I can work around this issue by changing the button colors in my settings:

"workbench.colorCustomizations": {
    "button.background": "#ffffff",
    "button.foreground": "#005fb8",
    "button.hoverBackground": "#cccccc"
}

@banacorn banacorn added the bug Something isn't working label Mar 20, 2024
@banacorn
Copy link
Owner

Yes, we should give them both --vscode-textLink-foreground, thank you!

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