File list is totally vertical if the list doesn't fit into one line unless run with `-a`

[Un1q32]

• os: macOS 14.4
• lsd --version: lsd 1.0.0 (it's the 1.1.0 binary, see [Bug]: lsd-v1.1.0-aarch64-unknown-linux-musl reports 1.0.0 as version #1012)
• echo $TERM: xterm-kitty
• echo $LS_COLORS: (none)

Expected behavior

Wrapping behavior should be the same as 1.0.0, or the same as when run with -a

Actual behavior

Zooming in shows a normal run can also have this problem if one line isnt enough to fit

[wwwjfy]

It seems -a doesn't matter (much) here.
My observation is if non-. items cannot fit in one line, it'll show one item per line.

In the above case non-. items are not many, so even though with both it's wider than one line, it still behaves as expected.

I'm using iTerm2. This is new in the last release.

[nogweii]

• lsd --version: lsd 1.1.1
• echo $TERM: xterm-kitty
• Arch Linux

Running into the same issue.

I've aliased ls to /usr/bin/lsd --icon=auto --color=auto -F, and indeed adding -a will make files wrap as expected.

[zwpaper]

it was introduced because we upgraded the term_grid = "0.1.*" dependency to the newer version, let me have a look at it.

[oneElectron]

I am also having this issue.

• lsd --version: lsd 1.1.1
• echo $TERM: alacritty
• MacOS 14.4

[FischLu]

I have the same issue

• lsd --version: lsd 1.1.1
• echo $TERM: xterm-256color
• MacOS 14.4

[nogweii]

Hm, I think I've found the root cause - term_grid doesn't ignore ANSI codes when calculating the width of each entry. That could be tricky to solve...

EDIT: Ah, actually there's a more maintained fork of the library at https://github.com/uutils/uutils-term-grid ! And it does properly ignore ANSI codes.

[CarloCattano]

came here looking for others having the same.
lsd 1.1.1

ls returns a vertical list of files if the size cant hold them all in one line

[zwpaper]

I decided to rollback the term_grid dependency and we can later discuss which version or libs we should upgrade to.

#1019, after this one is merged, I will release a v1.1.2 to do a quick fix.

sorry for the bothering, I was kinda busy recently, but this should not be an excuse

[zwpaper]

as now https://github.com/lsd-rs/lsd/releases/tag/v1.1.2 released, this would be close
/close

[muniu-bot]

@zwpaper: Closing this issue.

Details

In response to this:

as now https://github.com/lsd-rs/lsd/releases/tag/v1.1.2 released, this would be close
/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.