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

Add clipping char option (-C) #21

Merged
merged 1 commit into from
Feb 21, 2019
Merged

Conversation

gromgit
Copy link
Contributor

@gromgit gromgit commented Feb 16, 2019

This switches to a different character to highlight values that have
exceeded a user-defined max value. The original implementation just
leaves a blank line instead, which is ambiguous but may be desired
in various applications.

@tenox7
Copy link
Owner

tenox7 commented Feb 20, 2019

Excellent. I would make it a default as the current behavior is pretty lame. Could you make it default to 'x' for example?

This switches to a different character to highlight values that have
exceeded the user-defined max value (-m <max_value>).

The original implementation just prints a blank bar; we now default to
highlighting with 'x'.
@gromgit
Copy link
Contributor Author

gromgit commented Feb 21, 2019

Excellent. I would make it a default as the current behavior is pretty lame. Could you make it default to 'x' for example?

Done. I've updated this PR accordingly.

@tenox7
Copy link
Owner

tenox7 commented Feb 21, 2019

Awesome thank you!

@tenox7 tenox7 merged commit ce694fe into tenox7:master Feb 21, 2019
@gromgit gromgit deleted the highlight_hardmax branch December 8, 2023 15:34
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.

None yet

2 participants