Skip to content

Conversation

@ndeloof
Copy link
Contributor

@ndeloof ndeloof commented Jan 15, 2026

What I did
Rewrote tty progress UI to be more deterministic regarding terminal width and height.
New logic is:

  1. use lineData struct to collect data to be formatted on terminal
  2. compute line width requirements and truncate resource ID / details if needed
  3. compute status and timer padding
  4. render lines

note: sub task rendering (layers details during push/pull) has been removed to keep this beast (relatively) simple.

I tried to make the code bit more structured to get a more deterministic rendering, still this is all based on manual padding computation and string concatenations... We should look into an alternative TUI using https://github.com/charmbracelet/bubbles or comparable

Related issue
fixes #13474

also fixes #10256 by dropping extra lines if terminal height is too low

(not mandatory) A picture of a cute animal, if possible in relation to what you did

@ndeloof ndeloof requested a review from a team as a code owner January 15, 2026 10:37
@ndeloof ndeloof requested a review from glours January 15, 2026 10:37
@KiGamji
Copy link

KiGamji commented Jan 15, 2026

@ndeloof do you mind if I make my own PR with your patch and my approach included, preserving your authorship? I started to work on my approach here

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.

[BUG] Pulling TUI mixes up lines [BUG] TUI spams lines when container count is larger than terminal height

2 participants