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

Not sure if colors are right on st; what can I do to help ? #3

Open
ghost opened this issue Jun 8, 2021 · 2 comments
Open

Not sure if colors are right on st; what can I do to help ? #3

ghost opened this issue Jun 8, 2021 · 2 comments

Comments

@ghost
Copy link

ghost commented Jun 8, 2021

Hello. I'm running suckless st, latest commit + st-solarized-light-20190306-ed68fe7, st-anysize-0.8.4 and st-bold-is-not-bright-20190127-3be4cf1 patches. I'm not sure if the colors are ok, especially this "cyan". I've been using solarized light for many years and I've never actually seen it.

https://share.dmca.gripe/2LyM0QL8JI3vzRFN.png

This is also present on a non-customized alacritty.

https://share.dmca.gripe/ewyZxWnaPrjvzGrr.png

@shaunsingh
Copy link
Owner

The cyan color is definitely an issue and not part of the theme. What's happening is that the built-in regex highlighting is either incorrectly putting the cyan color, or I forgot to define an attribute and so its using cyan (which iirc is the default fallback color)

Its a high priority issue I'm planning to take a look at soon. the fixes should be merged into main sometime this week. In case you want to take a look as well, the code that you would need to look at is here: https://github.com/shaunsingh/solarized.nvim/blob/master/lua/solarized/theme.lua

@EPadronU
Copy link

EPadronU commented Apr 1, 2022

How I've dealt with this

-- Fix https://github.com/shaunsingh/solarized.nvim/issues/3
vim.cmd("highlight Identifier ctermfg=13 guifg=" .. require("solarized.colors").purple)

image

shaunsingh pushed a commit that referenced this issue Feb 9, 2023
Change all theme references to incorrect pink color to gray, since pink was defined as a gray hex code.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants