Conversation
| Text("\(index + 1)") | ||
| .foregroundColor(index == currentStep ? Color.purpleAccent : .white32) | ||
| .font(.custom(Fonts.regular, size: 17)) |
There was a problem hiding this comment.
CLAUDE.md violation: raw Text().foregroundColor().font() chain
The step number label uses a raw Text with chained .foregroundColor() and .font() modifiers, which is explicitly prohibited by CLAUDE.md:
"Use the project's text components (
DisplayText,HeadlineText,TitleText,SubtitleText,BodyMText,BodyMSBText,BodySSBText,BodySText,CaptionMText,CaptionText) instead of rawText().font().foregroundColor()chains."
The font size 17pt with Fonts.regular aligns with BodyMText. The per-step color override can still be applied on top of the text component via .foregroundColor().
| Text("\(index + 1)") | |
| .foregroundColor(index == currentStep ? Color.purpleAccent : .white32) | |
| .font(.custom(Fonts.regular, size: 17)) | |
| BodyMText("\(index + 1)") | |
| .foregroundColor(index == currentStep ? Color.purpleAccent : .white32) |
| ## [Unreleased] | ||
|
|
||
| ### Added | ||
| - Add UI for offline state in various flows #524 |
There was a problem hiding this comment.
CLAUDE.md violation: wrong PR number in CHANGELOG entry
The changelog entry references #524 but this is PR #528. CLAUDE.md requires:
"ALWAYS append
#PR_NUMBERat the end of each changelog entry when the PR number is known"
Looks like #524 was copy-pasted from the entry directly below it ("Add transfer from savings button").
| - Add UI for offline state in various flows #524 | |
| - Add UI for offline state in various flows #528 |
Description
Screenshot / Video
iPhone 17:
Simulator.Screen.Recording.-.iPhone.17.-.2026-04-22.at.18.23.36.mov
iPhone SE:
Simulator.Screen.Recording.-.iPhone.SE.3rd.generation.-.2026-04-22.at.18.25.27.mov
QA Notes
To test use the app while offline / in airplane mode