Skip to content

Open treatment details from a chart pill - #781

Merged
marionbarker merged 1 commit into
devfrom
feature/treatment-popup-navigation
Oct 17, 2026
Merged

marionbarker merged 1 commit into
devfrom
feature/treatment-popup-navigation

Conversation

@bjorkert

@bjorkert bjorkert commented Oct 17, 2026 •

Copy link
Copy Markdown
Member

Tapping a mark on the main chart shows its pill. If the mark is in the Treatments list (carbs, boluses, SMBs, overrides and temp targets) the pill has a chevron, and tapping the pill opens the treatment's details. Double-tap only zooms, so a missed double-tap can't open the wrong entry.

The chart and the Treatments list build their entries with the same code, so the pill opens the entry for the exact Nightscout treatment that drew the mark. Replaces #743.

Carb pill Carb details Bolus pill Note pill

A single tap on a chart mark shows its pill as before. If the mark is in
the Treatments list (carbs, boluses, SMBs, overrides, temp targets) the
pill shows a chevron, and tapping the pill opens the treatment's details.
Double-tap only zooms.

The chart and the Treatments list build their entries with the same
Treatment.historyEntries, so the pill opens the entry from the Nightscout
treatment that drew the mark.
bjorkert added a commit to bjorkert/LoopFollow that referenced this pull request Oct 17, 2026
# Conflicts:
#	LoopFollow/Controllers/Nightscout/Treatments/Overrides.swift
#	LoopFollow/Helpers/DataStructs.swift
@marionbarker

Copy link
Copy Markdown
Collaborator

Test

✅ successful test

Test Details

Use 2 test phones, one observing a Loop URL and one observing a Trio URL.

  • Before building this PR, tapping on a Carb entry on the LoopFollow screen shows details of the entry.
  • After building this PR, tapping on a Carb entry on the LoopFollow screen shows details of the entry along with an indicator for more information
    • tapping on the > takes you to the details about the meal and for a new enough version of Trio, or customized version of Loop, you can edit the meal from this screen

@marionbarker marionbarker left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve by test.

@marionbarker
marionbarker merged commit 4f1c69a into dev Oct 17, 2026
2 checks passed
@marionbarker
marionbarker deleted the feature/treatment-popup-navigation branch October 17, 2026 19:31
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