Repository navigation
Conversation
A flat list of week headers and entries with weekly totals, finding an entry by day, the distance-first row texts and the distance format. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
One summary line (visits this week, week streak) and the current week as seven days; a Calendar button expands to the month calendar with its year zoom. Tapping a day no longer filters, it highlights. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Rows lead with the distance, week headers show the week's total distance, and tapping a day in the overview scrolls to its entry and tints it, or says there was no visit that day. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Days with one visit start at 70% opacity instead of 30%, since most days have at most one visit; the year view keeps its wider gradient. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Render four to six weeks as the month needs instead of always six, so the card no longer has a blank band under the last week. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
The summary is just the visits this week. The year-long grid and its streak summary only existed for that line, so they go too. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Fills are 40-65% tints for days and 15-60% for months instead of the full primary color, with the normal text color on top. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Owner
Author
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
1 participant
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.
Stacked on #94 (retarget as the earlier PRs merge).
What
Structure
HistoryListModel.ktholds the pure logic (week rows, finding an entry's index, row texts, distance format);WeekStrip.ktis the compact week;HistoryOverviewswitches between the strip and the calendar.Tests
Unit tests for the row model, texts, totals, week strip and summary text; screen tests for the compact card, expanding, day taps and the totals.
./gradlew check ktfmtCheckis green locally. Not run on a device/emulator by me: scroll-to-day and the highlight timing need a look.🤖 Generated with Claude Code