Skip to content

feat(app): add version number in footer, add app header bar#127

Merged
andrewmelchor merged 1 commit intomainfrom
feat(app)/app-header-bar
Mar 9, 2026
Merged

feat(app): add version number in footer, add app header bar#127
andrewmelchor merged 1 commit intomainfrom
feat(app)/app-header-bar

Conversation

@andrewmelchor
Copy link
Copy Markdown
Member

Summary

Add app version in footer

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Mar 9, 2026

Greptile Summary

This PR restructures the TUI layout by introducing a new HeaderBar component at the top of the screen (displaying the app name and connection status) and moving the version number (v{Installation.VERSION}) into the StatusBar footer. The StatusBar is simplified by removing duplicate "t-req" branding and connection status indicators, which are now owned by HeaderBar.

Key changes:

  • New HeaderBar component — renders the app title on the left and connection status (icon + text) on the right, placed above the main SplitPanel in app.tsx.
  • StatusBar refactor — removes "t-req" label and connection status; consolidates all keybind hints into the left flex group; adds v{Installation.VERSION} on the right.
  • app.tsx — adds import and rendering of HeaderBar at the top of FullScreenLayout.

Issue found: The outer <box> in HeaderBar declares height={2} while containing only a single <box height={1}> child, wasting one terminal row with empty background color.

Confidence Score: 3/5

  • Safe to merge after fixing the HeaderBar outer box height, which currently wastes one terminal row.
  • The changes are straightforward UI restructuring with no data-flow or state-management risk. The layout integration is correct and the version number placement is clean. The only concrete issue is the mismatched height on the HeaderBar outer box (height=2 containing a single height=1 child), which silently wastes one visible terminal row. This is a real visual bug but does not cause crashes or data loss.
  • packages/app/src/tui/components/header-bar.tsx — fix the outer box height declaration to match its single child or add padding content.

Last reviewed commit: 1f54c54

Comment thread packages/app/src/tui/components/header-bar.tsx
@andrewmelchor andrewmelchor merged commit 4073e3c into main Mar 9, 2026
2 checks passed
@andrewmelchor andrewmelchor deleted the feat(app)/app-header-bar branch March 9, 2026 05:29
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.

1 participant