hard to read on light background about onefetch HOT 10 CLOSED

Since we've made a few PRs to address this, should this be closed?
Yes, it is actually perfect now. No configuration, in my case, needed.

Thanks a lot for your time!

from onefetch.

Themes definitely sounds like a fun idea! I would consider it a new feature instead of a resolution to this issue. Mainly because I consider this issue resolved already due to the below points.

• onefetch supports defining your own colors via command-line options
• isn't this more of an issue of terminal configuration than onefetch output? Out of curiosity I tried onefetch --true-color never with Gruvbox Light, and it looked fine to me

Since bat and delta's themes are for syntax highlighting, I think our themes would be slightly different. No ASCII has more than 9 colors currently (swift.ascii has 9, all shades of red though). I think it would be better if this could be accomplished without changing the language macro rule and calls, which are quite long.

So the way I imagine themes working is that colors would be defined as primary, secondary, tertiary, etc., and these would replace color {0}, color {1}, {2}, etc. We probably don't want to define 9 colors since most langs use less than half of that, so maybe 5 colors at maximum, and {5}, {6}, etc. would either use their original color or the primary color.

Basically, themes could be built-in versions of what would happen if you used onefetch --ascii-colors x y z

from onefetch.

isn't this more of an issue of terminal configuration than onefetch output? Out of curiosity I tried onefetch --true-color never with Gruvbox Light, and it looked fine to me

This is a fair point, but I do not want to figure out which colors I need to change on every terminal program I use. Picking a theme per program isn't as onerous, especially if I can just pick the ansi theme for everything (and ideally do that in a config file so I don't have to call it with command line flags every time).

onefetch --true-color never doesn't seem to make a difference for me, the text is still hard to read:

from onefetch.

Note that neofetch has no problem with this ;-)

Without looking at their source code, I think I can guess why: they're probably using unstyled text for the info fields, while we default to using white (showing a bias for dark terminals).

Made #593 based on this, so that you can at least disable some of the coloring for now and use your theme's foreground color 🙂

Also, I needed to add one more 0 to that workaround 😅
onefetch -t 0 0 0 0 0 0

from onefetch.

Yeah, unfortunately as dark colors don't show well on black background, light colors are hard to read on white background.

I don't see how i could please everyone...

Thx for your feedback @bugabinga

from onefetch.

I don't see how i could please everyone...

That would be hard indeed :)

However, how about these alternatives:
Offer flags for dark/light colors

$ onefetch --color=auto|light|dark|none

Maybe there is a way to use the default colors of the terminal emulator, though I admit, I don't know.

from onefetch.

One possible solution is to have a theme that uses ansi colors, as bat does:

• sharkdp/bat#490
• sharkdp/bat#1104
• sharkdp/bat#1412

Delta uses more or less the same solution because it uses bat themes: dandavison/delta#447 (comment)

Is creating an ansi theme or using bat's ansi theme possible for onefetch?

from onefetch.

Oh yeah, that white-on-white is pretty hard to read (I actually prefer low contrast output like that, but I know that's not the majority opinion 😆).

So I think I've been thinking about this the wrong way. I was focused on the logo, but really it seems like the text is the issue. As a very quick workaround for you, onefetch -t 0 0 0 0 0 would set all text colors to be black. Our themes should probably act as aliases for onefetch --text-colors a b c....

Looks like your theme keeps black as dark and white as light, but other light themes, like Gruvbox Light, make black the light color and white the dark color (this is why I thought it was resolved). So, if we add themes, we'll at minimum probably need 2 light themes to handle terminal themes that do reverse black and white and themes that don't reverse them.

from onefetch.

As a very quick workaround for you, onefetch -t 0 0 0 0 0 would set all text colors to be black.

Disappointingly, this doesn't seem to do it either:

The text is still too light to read.

Here's some more info about my system:

My theme is Adwaita.

P.S. Note that neofetch has no problem with this ;-)

from onefetch.

Since we've made a few PRs to address this, should this be closed? I think anything more would have to be done on a logo-by-logo basis.

from onefetch.