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

Improving Linux accessibility #692

Open
ross-spencer opened this issue Oct 24, 2024 · 2 comments
Open

Improving Linux accessibility #692

ross-spencer opened this issue Oct 24, 2024 · 2 comments

Comments

@ross-spencer
Copy link

Really a question, and I'm guessing it's unlikely to be a bug. I'm seeing the current output on Linux:

image

I thought the app I was using was just creating tabbed line and didn't include a datetime. It turns out it's just exactly the color of my console's background.

Is anyone seeing the same? Or does anyone have tips to workaround this?

The example here is just cribbed from the README.md.

Output in a text file:

{"level":"debug","time":1729779448,"message":"hello world"}
ESC[90m4:17PMESC[0m ESC[32mINFESC[0m ESC[1mHello worldESC[0m ESC[36mfoo=ESC[0mbar
@dharmab
Copy link

dharmab commented Oct 25, 2024

I can see it, although it's low contrast. You should probably change your terminal colorscheme so that your selected foreground colors have adequate contrast with your selected background colors. e.g. https://github.com/tinted-theming/tinted-shell has many such colorschemes.

@ross-spencer
Copy link
Author

I appreciate the response @dharmab

You should probably change your terminal colorscheme

That would be one approach, but at this point, I would only be changing it because someone might be using this library and I have been quite happy with my terminal choices for a while.

The question is more whether it impacts others too, and whether there are some usability guidelines for logging across terminals that is more balanced for all cases.

NB. I have been able to turn off color for now. It's not embedded in my own library and so it's not a long-term solution.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants