1
0
mirror of https://github.com/nushell/nushell.git synced 2025-03-23 12:07:32 +01:00
Commit Graph

2 Commits

Author SHA1 Message Date
Maxim Zhiburt
183be911d0
Patch after fix after fix 7380 ()
> I'm not sure how i feel about that. I mean if there are a lot of
columns, it should probably have a max width so 1 column doesn't take
the entire width of your screen. Ideally it would work closely like
table worked before we migrated to tabled, as far as how column widths
were allocated.

I believe it still not completely matched.
*To be honest I am not against the  approach.

The PR makes a switch between logics on a premise of `termwidth`.
So if `termwidth > 120` we start prioritizing amount of columns we can
show (We try to show as many columns as we can).
Otherwise we do what I've described in  (We show the least columns
but with least truncation involvement).

In case it's OK,
I guess we could make the value configurable.

cc @fdncred 
ref 

Signed-off-by: Maxim Zhiburt <zhiburt@gmail.com>
2022-12-17 16:16:32 -06:00
Maxim Zhiburt
33aea56ccd
Try to fix ()
fix https://github.com/nushell/nushell/issues/7380
2022-12-15 08:47:04 -06:00