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

Improve progress column output #417

Merged
merged 2 commits into from
Mar 7, 2019
Merged

Conversation

ilammy
Copy link
Collaborator

@ilammy ilammy commented Mar 6, 2019

Replace the awk magic which should align output columns with some ANSI code magic that aligns status strings in a separate column to the right. This tends to work better with our current file names. Previous code tends to insert ugly newlines for some really long file names.

Before:

before: ugly empty lines and disarray

After:

after: nicely aligned columns

Note nicely aligned columns and no weird empty lines after long file names.

Replace the awk magic which should align output columns with some ANSI
code magic that aligns status strings in a separate column to the right.
This tends to work better with our current file names. Previous code
tends to insert ugly newlines for some really long file names.
@ilammy
Copy link
Collaborator Author

ilammy commented Mar 6, 2019

I guess it should be possible to align the columns even more nicely near the right edge of the terminal, but my VT-fu is too weak for that at the moment. Or not...

Use more magic to determine a proper right edge of the terminal so that
we support smaller windows. However, don't move too far right. 80x25
terminal windows are still a thing and should be readable enough.
Copy link
Contributor

@vixentael vixentael left a comment

Choose a reason for hiding this comment

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

wow, cool!

@ilammy ilammy merged commit c578cfb into cossacklabs:master Mar 7, 2019
@ilammy ilammy deleted the vt-magic branch March 7, 2019 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants