Skip to content

fix(tipping): show the below-minimum tip prompt as info, not an alert - #1361

Merged
bmc08gt merged 1 commit into
code/cashfrom
fix/min-tip-info-modal
Aug 28, 2026
Merged

fix(tipping): show the below-minimum tip prompt as info, not an alert#1361
bmc08gt merged 1 commit into
code/cashfrom
fix/min-tip-info-modal

Conversation

@bmc08gt

@bmc08gt bmc08gt commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

Follow-up to #1360.

Entering a minimum tip under the floor put up a destructive red banner. The design calls for the info style — nothing has failed and nothing is being destroyed, the entry is just too low and needs raising. Saving still fails loudly, since that one is a real failure.

Also drops the Figma node ids from these files' KDoc. They don't survive file edits, so they go stale without anyone noticing.

Entering a minimum tip under the floor put up a destructive red banner. The
design calls for the info style: nothing has failed and nothing is being
destroyed, the entry is just too low and needs raising. Saving still fails
loudly.

Also drops the Figma node ids from these files' KDoc — they don't survive
file edits, so they go stale without anyone noticing.
@bmc08gt bmc08gt self-assigned this Aug 28, 2026
@github-actions github-actions Bot added the type: fix Bug fix label Aug 28, 2026
@bmc08gt
bmc08gt merged commit f812fa6 into code/cash Aug 28, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: fix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant