Skip to content

feat(tui): replace startup banner with braille wordmark - #4929

Open
rumpl wants to merge 1 commit into
docker:mainfrom
rumpl:feat/banner-redesign
Open

rumpl wants to merge 1 commit into
docker:mainfrom
rumpl:feat/banner-redesign

Conversation

@rumpl

@rumpl rumpl commented Nov 13, 2026 •

Copy link
Copy Markdown
Member

Summary

Replace the tall block-letter startup banner with a lowercase, bold italic docker agent wordmark, sampled into braille dots for smooth edges.

  • Shared by both TUIs and colored with the active theme's accent.
  • Reduces the banner from 13 rows to 6; updates the minimum width to 76 columns.
  • Adds coverage for dimensions and braille-only content, and updates the centering assertion.

New banner

       ⣿⡿                 ⣸⣿⠁                                                   ⣤⡶
 ⢠⣶⡿⢷⣶⣿⡇⢀⣴⣾⠿⢷⣦⡀ ⣠⣶⠿⢷⣦⡀⢀⣿⡟⣠⣶⠶⠂⢠⣶⡾⢿⣶⡄ ⣴⣶⣴⡿⠂   ⣠⣶⠿⢷⣦ ⢀⣴⣾⠿⣦⣶⡶ ⣠⣶⠿⢷⣦⡀ ⣶⣶⡶⢾⣷⡆⠠⢾⣿⡷⠆
⢠⣿⡏  ⣿⣿ ⣾⣿⠁ ⢈⣿⡇⢸⣿⠃  ⠉⠁⢸⣿⣿⣿⡇ ⢠⣿⣯⣤⣤⣿⡷⢀⣿⡟⠁    ⢀⣤⣤⣶⣾⣿ ⣼⣿⠁ ⢸⣿⡇⢸⣿⣧⣤⣤⣿⡇⢰⣿⡏ ⢰⣿⡇ ⣾⣿
⠘⣿⣧⣀⣴⣿⡏ ⢻⣿⣄⣠⣾⡿⠁⠸⣿⣄⣠⣾⡖ ⣿⡿⠁⢹⣿⡀⠈⢿⣧⣀⣴⣶⠄⣸⣿⠃     ⢺⣿⣄⣤⣿⡏ ⢻⣿⣤⣤⣾⣿ ⠘⣿⣦⣀⣴⡦ ⣼⣿  ⣼⣿ ⠠⣿⣇⡀
 ⠈⠉⠉⠉⠉⠁  ⠉⠉⠉⠉   ⠈⠉⠉⠁  ⠉⠁ ⠈⠉⠁  ⠉⠉⠉⠁ ⠉⠉       ⠉⠉⠁⠉⠁ ⣤⣍⣉⣡⣿⡇  ⠈⠉⠉⠉  ⠉⠉  ⠉⠉  ⠉⠉⠁
                                                  ⠙⠛⠛⠛⠋

@rumpl
rumpl requested a review from a team as a code owner November 13, 2026 23:43
Copilot AI balanced review requested due to automatic review settings November 13, 2026 23:43

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The compact banner update has no active review issues.

No open issues 🎉

What changed in this PR

Replaces the shared TUI startup banner with a compact, lowercase braille “docker agent” wordmark, retaining theme-controlled coloring.

Changes:

  • Reduces banner height from 13 to 6 rows and increases its width requirement to 76 columns.
  • Adds checks for dimensions, braille-only artwork, and absence of embedded styling or trailing spaces.
  • Updates the chat centering assertion for the new banner.
File Description
pkg/​tui/​page/​chat/​layout_position_test.go Updates the horizontal centering assertion.
pkg/​tui/​banner/​banner.go Replaces the artwork and updates banner dimensions.
pkg/​tui/​banner/​banner_test.go Adds dimension and content validation.

Review effort: Balanced

Signed-off-by: Djordje Lukic <djordje.lukic@docker.com>
@rumpl
rumpl force-pushed the feat/banner-redesign branch from bbefd48 to 1af7870 Compare November 13, 2026 23:50
Copilot AI balanced review requested due to automatic review settings November 13, 2026 23:50

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The default banner wraps in 60–77-column terminals, causing incorrect cursor positioning and repaints because wrapped lines are still counted as single rows.

1 open issue

Review effort: Balanced

Comment thread pkg/leantui/leantui.go
Comment on lines 401 to +404
m.screen.Transcript.AddBlock(func(int) []string {
var banner []string
if !m.hideBanner {
banner = renderBanner(m.banner)
@aheritier aheritier added area/tui For features/issues/fixes related to the TUI kind/feat PR adds a new feature (maps to feat:). Use on PRs only. labels Nov 14, 2026

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/tui For features/issues/fixes related to the TUI kind/feat PR adds a new feature (maps to feat:). Use on PRs only.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants