Skip to content

Show the absolute reset time next to limit countdowns (both 5-hour and weekly) - #263

Merged
chattymin merged 3 commits into
chattymin:mainfrom
Maik93:feat/5-hour-clock
Sep 8, 2026
Merged

Show the absolute reset time next to limit countdowns (both 5-hour and weekly)#263
chattymin merged 3 commits into
chattymin:mainfrom
Maik93:feat/5-hour-clock

Conversation

@Maik93

@Maik93 Maik93 commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Title: Show the absolute reset time next to limit countdowns

Summary

The limits section of the popover only showed a relative countdown to each window's reset (· in 3 hr, 5 min). That answers "how long" but not "at what time", so you can't tell whether a session frees up before your next meeting or whether a weekly cap resets Friday evening or Saturday morning.

This adds the wall-clock reset moment in parentheses after the countdown, for every limit row (Claude 5-hour session, Claude scoped limits[] entries, the Claude "current 5h block" row, Codex primary/secondary windows, the Codex personal spend limit, and Antigravity buckets):

  • Reset today or within 6 hours → time only: · in 3 hr, 5 min (14:52)
  • Reset further out (weekly caps, etc.) → weekday + day + time: · in 2 days, 9 hr (Friday 4, 18:00)

Details:

  • The format is chosen purely from how far away the reset is — no provider-specific branching, per the extension convention in CLAUDE.md.
  • The weekday name and field order are localized to the popover language (companion.language.displayLocale): ko6일 일요일 19:41, deSonntag 6., 19:41, etc. The time is always 24-hour (HHmm template), matching the existing forecastRow time display.
  • Implemented as two small Text-returning helpers (resetClockSuffix, resetLabel) that the row builders now share, replacing the repeated Text("· \(reset, style: .relative)") snippet.

Type of change

  • Bug fix
  • New feature
  • Refactor / cleanup
  • Documentation
  • Other:

UI changes

Limits section of the popover. Percentage, gauge, and colors are unchanged; only the trailing reset text is extended.

Before After
5-hour session — 8% · in 3 hr, 5 min 5-hour session — 8% · in 3 hr, 5 min (14:52)
Weekly — 26% · in 2 days, 9 hr Weekly — 26% · in 2 days, 9 hr (Friday 4, 18:00)

Canonical assets/ screenshots are regenerated at release, so they are not updated here.

Checklist

  • swift build passes locally (swift test could not run in this environment — XCTest module unavailable; CI covers the suite)
  • PR title and description are written in English
  • UI changes are described above (before/after — images optional)
  • No copyrighted assets, secrets, or private tooling references are committed (see CONTRIBUTING)
  • Tests were added or updated for this change — the change is a presentational helper (Text composition + locale template) with no existing popover formatting test harness and only private surface; verified via swift build and a locale-output check across en/ko/ja/de/fr.

@Maik93 Maik93 changed the title feat: enhance weekly reset display with reset date and time aside countdown (with weekday if not near) Show the absolute reset time next to limit countdowns (both 5-hour and weekly) Sep 3, 2026
@chattymin
chattymin merged commit f2fe5ac into chattymin:main Sep 8, 2026
2 checks passed
@chattymin

Copy link
Copy Markdown
Owner

Thank you for your contribution!

cte13 added a commit to cte13/PokeTokenBar-iOS that referenced this pull request Sep 8, 2026
* feat: raise the floating pet size ceiling from 192px to 384px (chattymin#267)

The desktop pet slider topped out at 192px. Double the maximum so the
companion can be sized up to 384px. Sprites render with
`.interpolation(.none)`, so the extra scale stays crisp pixel art instead
of blurring, and `FloatingPetController` already observes
`floatingPetSize`, so the panel resizes with the sprite as the slider
moves.

Also cover the other branch of `max(petSize, bubbleMinWidth)`, where the
pet drives the panel width rather than the bubble column. Only 184 and
192 reached it before; now it is most of the range.

Claude-Session: https://claude.ai/code/session_01X91ErNfhphSw9gC1pqjjjN

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
(cherry picked from commit cd3125a)

* Show the absolute reset time next to limit countdowns (both 5-hour and weekly) (chattymin#263)

* feat: enhance 5-hour reset display with reset time aside countdown

* feat: enhance weekly reset display with reset date and time aside countdown (with weekday if not near)

* chore: update staled comment about weekly relative-only display

(cherry picked from commit f2fe5ac)

* feat: trigger immediate refresh upon network reconnection (chattymin#252)

* feat: trigger immediate refresh upon network reconnection

* fix: add lock synchronization and flapping debounce to NetworkReachabilityMonitor

(cherry picked from commit b34673a)

---------

Co-authored-by: Yongjae Kim <yojkim@lbox.kr>
Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
Co-authored-by: Michael Mugnai <Maik93@users.noreply.github.com>
Co-authored-by: Justin Jeong <justin.jeong@buzzvil.com>
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