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

PR #229 • Add option to print color table #235

Merged
merged 3 commits into from
Dec 27, 2024

Conversation

sahinfalcon
Copy link
Contributor

@sahinfalcon sahinfalcon commented Dec 13, 2024

This PR adds the option to print a colour table, fixes #229

Copy link
Owner

@sharkdp sharkdp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you very much!

@sharkdp sharkdp merged commit 45a557e into sharkdp:master Dec 27, 2024
15 checks passed
@sahinfalcon sahinfalcon deleted the print-colour-table branch December 27, 2024 13:44
@thernstig
Copy link

@sharkdp @sahinfalcon I am using the default colors (a default install) and see this:

image

Is there a reason whitespace and control characters does not default to two different colors?

@sharkdp
Copy link
Owner

sharkdp commented Mar 2, 2025

Is there a reason whitespace and control characters does not default to two different colors?

No particular reason. I guess I didn't see the need to distinguish these categories so far, and apparently noone else did either.

@thernstig
Copy link

Is there a reason whitespace and control characters does not default to two different colors?

No particular reason. I guess I didn't see the need to distinguish these categories so far, and apparently noone else did either.

Ok, I though that since they had different icons, also color would be a way to distinguish them. Why I think so I do not even know? Maybe something with cognitive association or some other psychological reason 😆

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

Successfully merging this pull request may close these issues.

Add option to print the color table
3 participants