[bordalix]

No description provided.

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (1)

• next-version

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: ac5c2734-8938-4ca0-8cf1-55ade1cb5fee

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch fix_reminder

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[cloudflare-workers-and-pages]

Deploying wallet-mutinynet with    Cloudflare Pages

Latest commit:	f2d7fba
Status:	✅  Deploy successful!
Preview URL:	https://ff3a02aa.arkade-wallet.pages.dev
Branch Preview URL:	https://fix-reminder.arkade-wallet.pages.dev

View logs

[cloudflare-workers-and-pages]

Deploying wallet-bitcoin with    Cloudflare Pages

Latest commit:	f2d7fba
Status:	✅  Deploy successful!
Preview URL:	https://c7c718b7.wallet-bitcoin.pages.dev
Branch Preview URL:	https://fix-reminder.wallet-bitcoin.pages.dev

View logs

[cloudflare-workers-and-pages]

Deploying tmp-boltz-upstream-mainnet-arkade-wallet with    Cloudflare Pages

Latest commit:	f2d7fba
Status:	✅  Deploy successful!
Preview URL:	https://9e2d5ed5.tmp-boltz-upstream-mainnet-arkade-wallet.pages.dev
Branch Preview URL:	https://fix-reminder.tmp-boltz-upstream-mainnet-arkade-wallet.pages.dev

View logs

[ghost]

Code Review — clean Reminder

Verdict: Looks good overall — clean refactor replacing inline styles and custom CSS with shared Button and Text components. One minor issue to flag.

🟡 Minor: Subtitle color lost

src/components/Reminder.tsx — The old code applied color: 'var(--neutral-500)' to the date subtitle text. The new <Text small> (line ~52 in the new code) doesn't pass a color prop, so prettyDate(startTime) will render in the default text color instead of the muted grey. Likely should be:

<Text small color='neutral-500'>{prettyDate(startTime)}</Text>

ℹ️ Note: SheetModal padding is global

src/components/SheetModal.tsx:21 — Adding pt-4 to the drawer content wrapper affects all SheetModal consumers (Init.tsx, QrCode.tsx ×2, Send/Form.tsx), not just Reminder. Those screens already have their own inner padding so it's probably fine, but worth a quick visual check on each.

✅ Everything else

• .reminder-button CSS removal is safe — only used in Reminder.tsx
• Button outline and Text bold correctly map to the design system variants
• No protocol-critical code touched

LGTM pending the color fix above. 🤙

---

Reviewed by Arkana 🔍

[ghost]

Code Review — clean Reminder

Verdict: Looks good overall — clean refactor replacing inline styles and custom CSS with shared Button and Text components. One minor issue to flag.

🟡 Minor: Subtitle color lost

src/components/Reminder.tsx — The old code applied color: 'var(--neutral-500)' to the date subtitle text. The new <Text small> (line ~52 in the new code) doesn't pass a color prop, so prettyDate(startTime) will render in the default text color instead of the muted grey. Likely should be:

<Text small color='neutral-500'>{prettyDate(startTime)}</Text>

ℹ️ Note: SheetModal padding is global

src/components/SheetModal.tsx:21 — Adding pt-4 to the drawer content wrapper affects all SheetModal consumers (Init.tsx, QrCode.tsx ×2, Send/Form.tsx), not just Reminder. Those screens already have their own inner padding so it's probably fine, but worth a quick visual check on each.

✅ Everything else

• .reminder-button CSS removal is safe — only used in Reminder.tsx
• Button outline and Text bold correctly map to the design system variants
• No protocol-critical code touched

LGTM pending the color fix above. 🤙

---

Reviewed by Arkana 🔍

[ghost]

Re-review — new commit f2d7fba

Color fix looks good — <Text small color='neutral-500'> correctly restores the muted subtitle style from the old inline CSS. Previous feedback addressed. ✅

No further issues. LGTM.

---

Reviewed by Arkana 🔍