shaunsingh / solarized.nvim

Port of the Solarized colorscheme for vim, written in lua, with treesitter support.
GNU General Public License v2.0
181 stars 36 forks source link

Unrecognizable colors in simple terminal #14

Open fjbaldon opened 1 year ago

fjbaldon commented 1 year ago

https://imgur.com/a/UoDv8Aq

Attached is a screenshot where #include isn't being displayed correctly. I've patched st with st-no_bold_colors-0.8.1.diff and st-solarized-both-20220617-baa9357.diff .

shaunsingh commented 1 year ago

Does st support termguicolors?

fjbaldon commented 1 year ago

I'm not on solarized rn but yes. Running :termguicolors changes some colors in st

towry commented 1 year ago

截屏2022-12-14 17 13 42

stefumies commented 1 year ago

Same here on Mac

rzeigler commented 1 year ago

To chime in, this seems to show good colors for me with background=light.

cultab commented 1 year ago

I can replicate this in windows terminal, image and wezterm image

It seems to not be st specific and affects all filetypes (in lua filetypes variables are affected instead of #defines).

peterlvilim commented 1 year ago

I'm also hitting this. Light seems to work fine but having trouble with dark. On mac and tried kitty and alacritty. Have tried messing with various different options but haven't found anything yet.

Screenshot 2023-03-10 at 13 17 49
peterlvilim commented 1 year ago

I think I maybe found the problem. I opened a pull request #18 with some example fixes but looking for feedback

Sleepful commented 1 year ago

+1, I can confirm that @peterlvilim's is much more readable.

Notice the grey letters, those were impossible to read:

Screen Shot

Using Kitty terminal on OSX, having the same issues everyone described.

Sleepful commented 1 year ago

for comparison:

Screen Shot
peterlvilim commented 1 year ago

I have a bunch of commits on my local machine cleaning up various things with dark mode beyond the example diff. It doesn’t fix everything quite yet but been fixing as I notice. Will try to open a pull request with it in next few days.

lytex commented 1 year ago

Commit 34c2245a6ddfd85766f6127768f6b04b0ae2f84a which is the one I had pined before, doesn't have this issue