Add Width Option #128
Merged
Add Width Option #128
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
cafkafk
requested changes
Aug 13, 2023
|
I'll help with the summaries, maybe the completions |
TerminalWidth::deduce to respect -w/--width option
Signed-off-by: Christina Sørensen <christina@cafkafk.com>
Signed-off-by: Christina Sørensen <christina@cafkafk.com>
Ah great thank's :) ... When I skimmed the completion files they seemed kind of outdated, some flags missing, therefore I thought they are not maintained and skipped including the new option ... lazy me :D |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
lshas a-w/--widthoption limiting the columns of the grid output.exa/ezaonly allows to specify the number of columns via theCOLUMNSenvironment variable for some reason.This adds the
-w/--widthoption to the existing column limiting, meaning it behaves just like when specifying theCOLUMNSvariable, but with thels-like command line argument. In case bothCOLUMNSand-w, then the cli option takes precedence.The only difference to the
-woption oflsis, thatls -w 0sets the columns to infinity, while here it goes back to automatic width detection just as for theCOLUMNSvariable.Resolves ogham/exa#1168