Skip to content

Keep the guardrail alert visible on the carb and meal edit screens - #786

Open
bjorkert wants to merge 1 commit into
devfrom
fix/edit-sheet-guardrail-alert
Open

bjorkert wants to merge 1 commit into
devfrom
fix/edit-sheet-guardrail-alert

Conversation

@bjorkert

Copy link
Copy Markdown
Member

When a value over the max carbs, fat or protein setting is entered on the Loop carb edit or Trio meal edit screen and Update is tapped, the guardrail alert with the minimum and maximum now stays up until dismissed. The follow-up validation is skipped while an alert is showing, so it can't replace the guardrail message with the empty-amount alert or with a confirmation showing 0 g carbs.

Tapping Update while the carb field is focused ran the guardrail check and then the form validation right after it. The guardrail alert reset the field to 0, so the validation replaced it with the empty-amount alert, or for a Trio meal with fat or protein, with the confirmation showing 0 g carbs. The validation now skips when an alert is already showing.
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