Conversation
Simplifies time navigation UX and adds daily granularity support: **Keybinding Changes:** - Consolidated 'y' (year) and 't' (month) into single 't' key - 't' now cycles: Year → Month → Day → Year - Simpler, more intuitive time navigation **New Feature - DAY Granularity:** - View transactions aggregated by day in TIME view - Navigate between days with arrow keys when drilled in - Drill into specific days to see transactions - Combine with sub-grouping (e.g., "2024-03-15 > by Merchant") **Implementation:** - Added TimeGranularity.DAY enum value - Updated state.py to track selected_time_day - Enhanced data_manager.py for day-level aggregation with gap filling - Updated formatters.py to display ISO dates (2024-03-15) - Updated breadcrumb to show day in correct drill-down order - Added navigate_time_period support for day-to-day navigation **Documentation:** - Updated all keyboard shortcut references - Added screenshot generation for Days view - Updated README, quickstart, and navigation guides **Testing:** - All 912 tests pass - Added test for DAY → YEAR cycle - Type checking clean - Code formatting and linting pass - Markdown formatting validated 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…formatting Fixes three issues from commit 74fa329 and adds compact UI formatting: **Bug Fixes:** 1. Time period formatting now correctly displays Month/Day granularities - Was showing only "2024" for all granularities - Now shows "Mar 2024" for months, "2024-03-15" for days - Fixed by passing day parameter to format_time_period() 2. Hint bar now shows correct time toggle cycle - Was showing inconsistent "t=Month" then "y=Year" - Now correctly shows: "t=By Month" → "t=By Day" → "t=By Year" - Always uses 't' key (removed old 'y' reference) 3. Drilling down into time periods no longer crashes - Fixed by #1 above - missing day parameter caused formatting errors **UX Improvements:** 4. Abbreviated breadcrumb labels for better space usage - Changed "Merchant: Amazon" → "M: Amazon" - Abbreviations: M:, C:, G:, A:, T: (or first letter of custom labels) - Saves ~40-50 characters in multi-level navigation - Example: "T: 2024-03-15 > M: Amazon" vs "Time: 2024-03-15 > Merchant: Amazon" 5. Compact stats formatting with abbreviated labels - Changed "Income: | Expenses: | Savings:" → "In: | Out: | Net:" - Always shows all three values in compact format - Fits on narrow terminals (e.g., 120 chars) along with breadcrumb - Example: "1,234 txns | In: +$5,000 | Out: -$3,500 | Net: +$1,500" **Testing:** - All 912 tests pass - Updated 12 test assertions for new formats (11 breadcrumb, 1 stats) - Type checking clean (pyright) - Code formatting and linting pass (ruff) 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Ongoing UI refinement with the time grouping feature.