Skip to content

Redesign the history list (month groups, swipe to delete) - #93

Closed
SamiPr0 wants to merge 3 commits into
feat/history-heatmapfrom
feat/history-list-redesign
Closed

SamiPr0 wants to merge 3 commits into
feat/history-heatmapfrom
feat/history-list-redesign

Conversation

@SamiPr0

@SamiPr0 SamiPr0 commented Oct 8, 2026

Copy link
Copy Markdown
Owner

Stacked on #92 (base is feat/history-heatmap; retarget to main once #92 merges).

What

  • Entries are grouped under sticky month headers ("October · 5 visits"), then a light day label, instead of one card per entry.
  • Flat compact rows with a divider and a chevron; tap still opens the entry details.
  • Swipe left to delete with an Undo snackbar, replacing the red delete button and confirmation dialog on every row. Undo restores the entry with its swim duration/distance and subscription.
  • Accessibility: rows expose a "Delete entry" custom action, since a swipe isn't available to screen readers.

Tests

Unit tests for the month/visit labels and onRestoreEntry; screen tests for swipe delete, Undo, empty state after deleting the last entry, tap not deleting, and month headers. ./gradlew check ktfmtCheck is green locally. No emulator run and no device check (connectedDebugAndroidTest not run).

Note

Deleting no longer asks for confirmation; the Undo snackbar is the safety net.

🤖 Generated with Claude Code

SamiPr0 and others added 2 commits October 8, 2026 11:46
Lets the UI undo a delete with the entry's swim data intact.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Group entries under sticky month headers with visit counts, use flat
compact rows, and delete by swiping left with an Undo snackbar instead
of a red button and a confirmation dialog on every row. Screen readers
get a Delete entry action in place of the swipe.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Entries sit under sticky week headers (This week, Last week, then the
dates) with a visit count, and each row carries its own date block
instead of a per-day header, since there is about one visit a day.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
@SamiPr0

SamiPr0 commented Oct 9, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #96, which combined this stack and is merged into main (cbab128). Closing; the branch is left in place.

@SamiPr0 SamiPr0 closed this Oct 9, 2026
@SamiPr0
SamiPr0 deleted the feat/history-list-redesign branch October 9, 2026 22: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