-
-
Notifications
You must be signed in to change notification settings - Fork 219
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 Width Option #128
Add Width Option #128
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This needs to include completions for fish and zsh as far as I can tell.This needs to conform to conventional commits
I'll help with the summaries, maybe the completions |
TerminalWidth::deduce to respect -w/--width option
Signed-off-by: Christina Sørensen <[email protected]>
Signed-off-by: Christina Sørensen <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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 |
ls
has a-w/--width
option limiting the columns of the grid output.exa/eza
only allows to specify the number of columns via theCOLUMNS
environment variable for some reason.This adds the
-w/--width
option to the existing column limiting, meaning it behaves just like when specifying theCOLUMNS
variable, but with thels
-like command line argument. In case bothCOLUMNS
and-w
, then the cli option takes precedence.The only difference to the
-w
option ofls
is, thatls -w 0
sets the columns to infinity, while here it goes back to automatic width detection just as for theCOLUMNS
variable.Resolves ogham/exa#1168