deno-cliffy icon indicating copy to clipboard operation
deno-cliffy copied to clipboard

Improve colors for help text to support more terminal themes

Open c4spar opened this issue 3 years ago • 2 comments

As discussed in the discord channel https://discord.com/channels/748965956377772055/804514019301916693/853276012619890699 we should think about using different colors for the generated help text, because in some terminals like Powershell some parts of the help text are not or badly visible.

legacy powershell theme powershell theme

dragonstyle from the Discord channel suggested to use some rgb colors like 5DADE2 for blue and A569BD for purple, which are more visible than the other colors in the legacy powershell theme, and I like the colors too.

blue on blue blue on blue purple on blue purple on blue

But the problem with rgb colors is that they can't be mapped. Most terminals have the ability to map the default ansi colors to different colors to create better terminal themes. But this is not possible when we use rgb colors.

I would prefer to stick with the default ansi colors, but it would be cool to get some more feedback from the community on what colors we should use to support a wide range of terminal themes.

c4spar avatar Jul 02 '21 09:07 c4spar

Been looking at this. Is there appetite for making Cliffy more generally themeable? I might be able to contribute if so, and it sidesteps some of this problem by letting people specify colors themselves.

eropple avatar Aug 12 '22 16:08 eropple

Hi @eropple, any help is welcome. I think it's fine for me to make the colors configurable. Do you already have an idea how to implement this?

c4spar avatar Aug 14 '22 14:08 c4spar

Hey there, I'm sorry - I lost this in the github notification spam. I work at Render and we're building a CLI on top of Cliffy, and I think we'd like to customize the color scheme we've got. So it might take a bit for me to get to it, but I am provisionally interested. I recall I had some ideas when I was looking last time, but I've lost state, so will have to get it back when I'm able.

eropple avatar Nov 03 '22 15:11 eropple

np, thx!

c4spar avatar Nov 04 '22 13:11 c4spar