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

Unreadable colors #71

Closed
aleksator opened this issue Aug 8, 2020 · 8 comments · Fixed by #73
Closed

Unreadable colors #71

aleksator opened this issue Aug 8, 2020 · 8 comments · Fixed by #73
Labels
bug Something isn't working

Comments

@aleksator
Copy link

I have a problem with default coloring scheme. It contains both white and dark gray colored words, so half of it is very hard to read with whitish background theme, while another half with dark background theme (see screenshot).
hors_colors

Hors version is 0.6.4, run under KDE, Yakuake, Sweet console theme.
I tried other themes and it was still problematic.

@dzervas
Copy link
Contributor

dzervas commented Aug 8, 2020

This will be solved with #61 I think, where the color scheme will be configurable.

Apart from that though, I think that you've changed the default colors (terminal themes tend to do that), so for example red could appear as blue. To fix that, I think the best solution is to use more "safe" colors as default, so that it appears "good enough" on most terminal themes. We can read how highlight or https://github.com/sharkdp/bat do things and how they picked colors.

@aleksator
Copy link
Author

I don't think the issue is the current theme that I am using, as I noted I tried different ones.
Here's the example with default KDE terminal theme Breeze (I changed global KDE theme to default Breeze Dark as well just in case):

hors_colors_breeze
With terminal theme set to "White on Black":
hors_colors_black_and_white

In fact, hors output colors don't change with the theme at all as far as I noticed, only the colors of the text I typed and the background change.

@dzervas
Copy link
Contributor

dzervas commented Aug 8, 2020

Oh, I hear you...
image

@WindSoilder
Copy link
Owner

Hi, thanks for your report, I can reproduce this issue too.

It seems that hors does not take the most approprate tag to colorize output. For this link, it just take json scheme to colorize....

Maybe I need to take a little time to think how to fix it appropriately.

@WindSoilder
Copy link
Owner

Oh, maybe there is a better way to select language scheme to colorize code.
图片
I can make use of this attribute :-)

@WindSoilder WindSoilder added the bug Something isn't working label Aug 10, 2020
@dzervas
Copy link
Contributor

dzervas commented Aug 10, 2020

oh ok, if it's a problem on that specific answer (marked as java while it's Rust) I don't consider it a bug

@WindSoilder
Copy link
Owner

@aleksator Hi, I have publish a new version(0.6.6) to handle this, feel free to check it out :-)

@aleksator
Copy link
Author

@WindSoilder Looks great! Thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
3 participants