Skip to content

fix: honor relative reset toggle and local tz in tray/pop-out cards - #39

Merged
Finesssee merged 2 commits into
nesszer:mainfrom
nov30th:fix/menucard-reset-time-local-tz
Apr 26, 2026
Merged

Finesssee merged 2 commits into
nesszer:mainfrom
nov30th:fix/menucard-reset-time-local-tz

Conversation

@nov30th

@nov30th nov30th commented Apr 25, 2026

Copy link
Copy Markdown

Summary

  • fix: render Claude reset time in local timezone, honor relative-reset toggle #37 fixed reset-time formatting in Settings → Providers, but the tray icon popup and pop-out window render through MenuCard, which still displayed snap.resetDescription verbatim (backend pre-formats Claude's DateTime<Utc> as UTC wall time) and ignored the Display → "Relative Reset Time" toggle.
  • Thread resetTimeRelative through TrayPanel and PopOutPanel into MenuCard, and route both metric resets and cost.resetsAt through the existing useFormattedResetTime hook so the same local-tz / countdown logic applies in every surface.

Test plan

  • Toggle Display → "Relative Reset Time" off; tray popup shows reset time in local timezone (verified with Claude account on a UTC+8 machine).
  • Toggle on; tray popup shows live "Resets in Xh Ym" countdown.
  • Pop-out window mirrors the tray behavior.
  • Cost section's "Resets" line reformats correctly.
  • tsc --noEmit and tauri build pass.

nov30th and others added 2 commits April 25, 2026 17:07
PR nesszer#37 fixed reset-time formatting in Settings → Providers, but the tray
icon popup and pop-out window render through MenuCard, which still
displayed snap.resetDescription verbatim (backend pre-formats Claude's
DateTime<Utc> as UTC wall time) and ignored the Display →
"Relative Reset Time" toggle.

Thread resetTimeRelative through TrayPanel and PopOutPanel into
MenuCard, and route both metric resets and cost.resetsAt through the
existing useFormattedResetTime hook so the same local-tz / countdown
logic applies in every surface.
@Finesssee
Finesssee merged commit 59abeef into nesszer:main Apr 26, 2026
0 of 10 checks passed
@Finesssee

Copy link
Copy Markdown
Collaborator

Thanks for the PR, this was the missing follow-through from #37 for the tray popup and pop-out surfaces. I added one small guard before merging so fallback reset text from providers that use it for non-time details stays unlabelled, then validated locally with frontend build/tests and a Tauri debug build. Merged into main.

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.

2 participants