Add classic and spacious TUI layouts - #354
Conversation
995f96d to
4c8ae9a
Compare
There was a problem hiding this comment.
Pull request overview
Adds selectable classic and spacious TUI layouts with runtime switching and responsive spacing.
Changes:
- Adds layout selection and Ctrl+G toggling.
- Introduces spacious content, row, header, and footer spacing.
- Documents and tests layout behavior.
Tip
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.
Reviewed changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
README.md |
Documents layout selection and switching. |
internal/cmd/tui.go |
Parses and validates layout arguments. |
internal/cmd/root_test.go |
Tests CLI layout selection. |
internal/tui/layout.go |
Defines layouts and responsive metrics. |
internal/tui/layout_test.go |
Tests layout rendering and toggling. |
internal/tui/tui.go |
Integrates layout sizing and switching. |
internal/tui/tui_test.go |
Adds Ctrl+G test input support. |
internal/tui/content.go |
Adds spacing between mail rows and sections. |
internal/tui/mail.go |
Applies layout gaps to mail lists. |
internal/tui/toast.go |
Sizes toasts against content width. |
internal/tui/section_view.go |
Shares layout metrics with views. |
internal/tui/accounts.go |
Sizes account modals to content width. |
internal/tui/accounts_test.go |
Updates model construction calls. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
…pin the --remote layout refusal The Screener captures input for its own keys, which swallowed the layout toggle the way it once swallowed help; Ctrl+G is now dispatched ahead of captured input and allowed there, as ? already is. A list shorter than five rows cannot show one sectioned posting with its gaps, so they collapse until it can. The --remote plus layout usage error gets its command-level case beside the other destination checks.
047d051 to
6cf8eba
Compare
|
Rebased onto main at 6cf8eba (the README paragraph moved to Where this stands: whether HEY's TUI takes on a second layout is a product call that is still pending, so the PR is parked on that decision rather than on anything in the diff. It is current and green in the meantime. |
Summary
classiclayouthey tui spaciouswith vertical breathing room around content, between mail rows, and around section headers--remote, which opens an already-running TUIDemo
Testing
make fmt-check vet tidy-check check-surface check-release-lockstepgolangci-lint run ./...make test