Skip to content

[LOOP-5952 & LOOP-5953] Liquid Glass Fixes Part 2 - #946

Open
Camji55 wants to merge 1 commit into
devfrom
cameron/LOOP-5952-liquid-glass-pt2
Open

[LOOP-5952 & LOOP-5953] Liquid Glass Fixes Part 2#946
Camji55 wants to merge 1 commit into
devfrom
cameron/LOOP-5952-liquid-glass-pt2

Conversation

@Camji55

@Camji55 Camji55 commented Aug 16, 2026

Copy link
Copy Markdown
Member

@Camji55
Camji55 requested review from nhamming, ps2 and stachon August 16, 2026 01:11
Text("mg/dL")
.foregroundColor(.secondary)
}
Text("mg/dL")

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Should DisplayGlucosePreference be used for the unit?

private var actionButton: some View {
Button<Text>(
action: {
if self.viewModel.actionButtonAction == .enterBolus {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Can the empty branch be deleted?

var saveFavoriteFoodButtonDisabled: Bool {
get {
if let carbsQuantity, 0...maxCarbEntryQuantity.doubleValue(for: preferredCarbUnit) ~= carbsQuantity, foodType != "", selectedFavoriteFoodIndex == -1 {
if let carbsQuantity, 0...maxCarbEntryQuantity.doubleValue(for: preferredCarbUnit) ~= carbsQuantity, effectiveFoodType != "", selectedFavoriteFoodIndex == -1 {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The selectedDefaultAbsorptionTimeEmoji always has value, so this now allows for empty foodType ?

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