Skip to content

Show calibration advice only once, split into why and how - #113

Open
ETolboom wants to merge 2 commits into
LoopKit:next-devfrom
ETolboom:feat/calibration-advice-redesign
Open

ETolboom wants to merge 2 commits into
LoopKit:next-devfrom
ETolboom:feat/calibration-advice-redesign

Conversation

@ETolboom

@ETolboom ETolboom commented Nov 4, 2026 •

Copy link
Copy Markdown

Summary

  • The "Before You Calibrate" advice now only leads the calibration flow the first time it's shown; later calibrations skip straight to the meter-entry page.
  • The advice is split into two focused pages: whether calibrating is a good idea at all, then how to do it properly. The how-to stays reachable later via a "How to Calibrate" pill on the entry page.
  • "Seen" is set on an explicit exit from the advice (Cancel, or continuing past the how-to page) rather than on mere appearance, so a sheet nobody acted on doesn't count as shown — and so VoiceOver/Voice Control users (who don't trigger tap gestures) aren't skipped incorrectly either.
  • Adds a #if DEBUG-only "Reset Calibration Advice" control in settings for re-testing the first-time flow without a fresh pairing.

Screenshots

Before

Advice Sheet Calibration Sheet
IMG_1666 IMG_1667

After

Advice Sheet (Why) Advice Sheet (How) Calibration Sheet
IMG_1672 IMG_1673 IMG_1674

The calibration sheet always led with a long "Before You Calibrate"
page, every single time, which made routine calibrations tedious.

The advice now only leads the flow the first time it is shown; later
calibrations skip straight to the meter-entry page, with the
how-to reachable again from a "How to Calibrate" pill on that page.

The advice itself is split into two pages: whether calibrating is a
good idea at all, then how to do it properly. The seen flag flips on
an explicit exit from the advice flow (cancelling, or continuing past
the how-to page) rather than on mere appearance, so a sheet nobody
acted on doesn't count as shown.
@ETolboom
ETolboom marked this pull request as draft November 4, 2026 20:09
#if DEBUG alone would still ship it to real phones, since Trio's own
builds are commonly Debug configuration.
@ETolboom
ETolboom marked this pull request as ready for review November 4, 2026 20:37
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