solarized.nvim icon indicating copy to clipboard operation
solarized.nvim copied to clipboard

Unrecognizable colors in simple terminal

Open fjbaldon opened this issue 2 years ago • 12 comments

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 .

fjbaldon avatar Dec 11 '22 00:12 fjbaldon

Does st support termguicolors?

shaunsingh avatar Dec 11 '22 05:12 shaunsingh

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

fjbaldon avatar Dec 12 '22 22:12 fjbaldon

截屏2022-12-14 17 13 42

towry avatar Dec 14 '22 09:12 towry

Same here on Mac

stefumies avatar Jan 27 '23 11:01 stefumies

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

rzeigler avatar Jan 30 '23 18:01 rzeigler

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).

cultab avatar Mar 10 '23 01:03 cultab

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 avatar Mar 10 '23 21:03 peterlvilim

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

peterlvilim avatar Mar 11 '23 11:03 peterlvilim

+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 avatar Apr 05 '23 01:04 Sleepful

for comparison:

Screen Shot

Sleepful avatar Apr 05 '23 01:04 Sleepful

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.

peterlvilim avatar Apr 05 '23 01:04 peterlvilim

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

lytex avatar Apr 08 '23 17:04 lytex