Skip to content

feat(account-demo): unify Features tiles and toast send results - #92

Merged
barvian merged 2 commits into
mainfrom
feat/account-demo-feature-cards
Aug 25, 2026
Merged

feat(account-demo): unify Features tiles and toast send results#92
barvian merged 2 commits into
mainfrom
feat/account-demo-feature-cards

Conversation

@barvian

@barvian barvian commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Share one FeatureGridCard chrome across the account demo Features grid so idle tiles (including Subscribe / Open Account) left-align the same sm Button and drop the footer divider.
  • Connected subscription and spending-account rows keep the status footer (badge/address + divider); Unsubscribe / Delete wrap left with the content above.
  • Toast send success and errors with Sonner so the outcome is visible even if the transact modal is dismissed. Estimate-blocked and sequence-recovery banners stay inline because they still need actions.

Test plan

  • Open /vibenet/demos/account with an account selected: Features tiles should match (left-aligned buttons, no divider), Features heading closer to the grid.
  • Subscribe, then confirm the status row (badge, expiry, Unsubscribe) has the divider and left-aligned wrap.
  • Open a spending account, then confirm address + Delete Account wrap left the same way.
  • Send a transaction: success/error toast appears; dismissing the modal still shows the toast.
  • Estimate-blocked and out-of-sequence banners still render inline with their actions.

@cb-heimdall

cb-heimdall commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

✅ Heimdall Review Status

Requirement Status More Info
Reviews 1/1
Denominator calculation
Show calculation
1 if user is bot 0
1 if user is external 0
2 if repo is sensitive 0
From .codeflow.yml 1
Additional review requirements
Show calculation
Max 0
0
From CODEOWNERS 0
Global minimum 0
Max 1
1
1 if commit is unverified 0
Sum 1

⚠️ Ignored Reviews (2)

Reviewer Reason
danyalprout Old or outdated review
danyalprout Review missing MFA

@vercel

vercel Bot commented Aug 25, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
omni-ui Ready Ready Preview Aug 25, 2026 8:59pm

Request Review

Share one card chrome so action buttons stay left-aligned, and surface
transaction outcomes with Sonner instead of only the inline banner.

Co-Authored-By: Cursor <cursoragent@cursor.com>
@barvian barvian closed this Aug 25, 2026
@barvian barvian reopened this Aug 25, 2026
@barvian
barvian merged commit 56d0b31 into main Aug 25, 2026
12 checks passed
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.

3 participants