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

Default colors hard to read on Windows terminals #72

Open
erezsh opened this issue May 25, 2022 · 7 comments
Open

Default colors hard to read on Windows terminals #72

erezsh opened this issue May 25, 2022 · 7 comments
Labels

Comments

@erezsh
Copy link

erezsh commented May 25, 2022

For example, this is a screenshot on Windows, using "Windows Terminal".

image

And this one is taken from a regular cmd:

image

(which is better but still hard to read when it's small)

@erezsh erezsh changed the title Default colors hard to read on some terminals Default colors hard to read on Windows terminals May 25, 2022
@ewels
Copy link
Owner

ewels commented May 25, 2022

Yeah, I saw in a comment recently that the colours aren't ideal in light colour themes either. Picking colours that work everywhere is difficult! I want to use colours as much as possible to make content visually distinct (that's kind of the whole point of the package).

Any suggestions for improvements?

@ewels
Copy link
Owner

ewels commented May 25, 2022

NB: Remember that you can overwrite the default styles / colours for any element with rich-click. See the end of the readme. But good defaults is obviously the best solution here.

@erezsh
Copy link
Author

erezsh commented May 25, 2022

I think playing it safe is the best approach for default themes. It's better to risk something being less distinct, than it being hard to read even when your attention is on it.

For dark, I would go with lighter colors (maybe pastel-oriented). I don't know about the light theme, but I guess it needs stronger darker colors.

p.s. note that the first one supports true-color (or close enough), so you can play with shades. The second one is 16 colors, so nothing will make it pretty, but it needs to be cyan or light gray, not blue.

Anyway, just me 2 cents. I know it's hard to get it right for every terminal.

@ewels
Copy link
Owner

ewels commented May 25, 2022

Yup, but what would be playing it safe other than uncoloured text?

Light terminal themes use the same rich-click theme as everything else, so the colours have to work on both light and dark terminal themes. So using lighter paster-colours would be worse for those users.

@erezsh
Copy link
Author

erezsh commented May 25, 2022

Oh, I see the problem now. I assumed there was some way to detect it.

This sounds like a fundamental problem. Maybe rich should implement something like this https://github.com/dalance/termbg

@ewels
Copy link
Owner

ewels commented May 25, 2022

Hmm 🤔 Maybe it does already - I haven't looked for it. Wouldn't be the first time 😅

@erezsh
Copy link
Author

erezsh commented May 25, 2022

Seems that it doesn't right now. But it never hurts to ask - Textualize/rich#1170

@ewels ewels added the style label Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants