Skip to content

fix: sync nav timer with timer mode and checklist mode#79

Merged
hyldmo merged 1 commit intomainfrom
hyldmo/fix-timer-sync
Apr 5, 2026
Merged

fix: sync nav timer with timer mode and checklist mode#79
hyldmo merged 1 commit intomainfrom
hyldmo/fix-timer-sync

Conversation

@hyldmo
Copy link
Copy Markdown
Owner

@hyldmo hyldmo commented Apr 5, 2026

Summary

  • Nav RestTimer now shows set elapsed when a set timer is active, matching TimerMode's display
  • Session page passes startedAt to the store so session elapsed timer works in checklist mode (not just timer mode)
  • init() preserves existing rest timers when entering timer mode for the first time

Test plan

  • In timer mode: start a set, exit → nav shows same set elapsed time
  • In timer mode: rest countdown running, exit → nav shows same countdown
  • In checklist mode: confirm a set → rest countdown appears in nav
  • In checklist mode: rest expires → nav shows session elapsed (not just dumbbell)
  • Start rest in checklist mode, then open timer mode → rest countdown persists

🤖 Generated with Claude Code

Nav RestTimer now shows set elapsed when a set timer is active,
matching what TimerMode displays. Session page passes startedAt
to the store so session elapsed works in checklist mode too.
Store init() preserves existing rest timers.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

Preview deployment

URL https://hyldmo-fix-timer-sync.macromaxxing.pages.dev
Branch hyldmo/fix-timer-sync
Commit 296340a

@hyldmo hyldmo merged commit 036d24d into main Apr 5, 2026
3 checks passed
@hyldmo hyldmo deleted the hyldmo/fix-timer-sync branch April 5, 2026 19:09
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