Skip to content

Comments

Render TUI panels in two‑column layout#14

Merged
theokeist merged 2 commits intomainfrom
codex/update-documentation-with-types-and-arguments-1g1szq
Jan 27, 2026
Merged

Render TUI panels in two‑column layout#14
theokeist merged 2 commits intomainfrom
codex/update-documentation-with-types-and-arguments-1g1szq

Conversation

@theokeist
Copy link
Owner

Motivation

  • Make the serve command TUI use the full terminal width by arranging panels side‑by‑side so related info (status, traffic, performance, logs) appears as a dashboard rather than a long vertical list.
  • Address alignment/overflow issues so paired boxes stay visually consistent across different panel heights.

Description

  • Reworked src/Console/Commands/ServeCommand.php to render panels in two‑column rows using a new renderTuiPanelRow(...) helper instead of many independent renderTuiPanel(...) calls, and adjusted the panel ordering to improve readability.
  • Introduced buildTuiPanelLines(...), padPanelLines(...) and a padded renderTuiPanel(...) implementation so each box is produced as a list of lines and can be aligned and merged into rows reliably.
  • Kept existing coloring and width/padding logic but centralized string building and color stripping via padStrip(...) and stripColors(...) to avoid misalignment from ANSI codes.
  • Minor layout and copy tweaks (panel titles, placeholders) to match the new two‑column presentation and keep the existing TUI controls/usage hints.

Testing

  • No automated tests were executed for this change.

Codex Task

@chatgpt-codex-connector
Copy link

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@theokeist theokeist merged commit 63cb674 into main Jan 27, 2026
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant