Skip to content

Print one path per line without trailing padding in ls and search - #391

Closed
dlevi309 wants to merge 4 commits into
dropbox:masterfrom
dlevi309:graphical-fixes
Closed

dlevi309 wants to merge 4 commits into
dropbox:masterfrom
dlevi309:graphical-fixes

Conversation

@dlevi309

Copy link
Copy Markdown

In short mode, ls deliberately printed entries in rows of four columns, but names with spaces made those columns look jumbled leaving a broken output. This is also the case when the shell session's window is narrower than a row of four paths: ls/search always put four entries per row regardless of terminal width, so long rows wrapped onto the next line and the columns no longer lined up.

Instead it prints a single entry per row, so the output stays readable at any window width and is easy to pipe into other tools like grep or xargs. it now also sorts entry results alphabetically.

dlevi309 and others added 3 commits October 18, 2026 19:46
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011KiQ74PLNJ2h77HwcPwbGr
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011KiQ74PLNJ2h77HwcPwbGr
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011KiQ74PLNJ2h77HwcPwbGr
@AndreyVMarkelov

Copy link
Copy Markdown
Contributor

Preserve server ordering in search unless --sort is set

@CLAassistant

CLAassistant commented Oct 19, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@dlevi309

Copy link
Copy Markdown
Author

Preserve server ordering in search unless --sort is set

Just reverted the sorting changes for now since it was only one line

@AndreyVMarkelov

Copy link
Copy Markdown
Contributor

Thanks for raising the formatting issue and for quickly reverting the unintended search sorting change. We decided to keep the existing four-column ls layout for compatibility. #398 has now been merged; it removes trailing padding from ls and search while preserving that layout.

Since the narrower fix is in place, I'm closing this PR. I appreciate the contribution and the discussion.

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.

3 participants