Skip to content

Polish terminal UI theming and fix count display - #10

Open
superxlbot wants to merge 1 commit into
mainfrom
improve-tui-styling
Open

Polish terminal UI theming and fix count display#10
superxlbot wants to merge 1 commit into
mainfrom
improve-tui-styling

Conversation

@superxlbot

Copy link
Copy Markdown
Collaborator

Summary

Optimizes the terminal UI (Rich console output) for the annual expense report, giving it a cohesive, on-brand look and fixing a small display bug.

Changes

  • Unified brand theming — all statistics tables now share a single styled builder using the project's brand blue (#00468c): rounded borders, colored headers, and subtle zebra rows for readability.
  • Stronger visual hierarchy — the yearly total is highlighted in a prominent banner panel, and spend amounts are accented in a warm color throughout.
  • Friendlier framing — the greeting now shows the campus-card balance (already fetched, previously unused), and the closing report message is wrapped in a themed panel that also points to where the charts were saved.
  • Bug fix — transaction counts were rendering as floats (e.g. 2.0, 15.0) in the consumption-type and time-slot tables because the aggregated DataFrame was upcast to float. They are now cast to int.

Testing

Rendered print_statistics against a representative sample dataset and verified all tables display correctly with the new styling and integer counts. Both modules pass py_compile and import cleanly.

🤖 Generated with Claude Code

- Add a unified, brand-blue (#00468c) themed table style shared across all
  statistics tables (rounded borders, colored headers, zebra rows).
- Highlight the yearly total with a prominent banner panel and accent
  spend amounts throughout for stronger visual hierarchy.
- Show the campus-card balance in the greeting and wrap the closing
  report message in a themed panel.
- Fix transaction counts rendering as floats (e.g. "2.0") in the type
  and time-slot tables by casting to int.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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