Skip to content

fix(vibenet): fix B20 transaction modals exit transitions - #121

Draft
barvian wants to merge 1 commit into
mainfrom
fix/b20-modal-close-flash
Draft

fix(vibenet): fix B20 transaction modals exit transitions#121
barvian wants to merge 1 commit into
mainfrom
fix/b20-modal-close-flash

Conversation

@barvian

@barvian barvian commented Sep 1, 2026

Copy link
Copy Markdown
Collaborator

Before:

replay-2026-09-01T14.25.45-frames-188-309.mov

After:

replay-2026-09-01T14.50.27.mov

…close

reset() ran in the same render that set open=false, so the Base UI exit
animation flashed the empty build step. Form and tx state now live in an
inner tree that only mounts while open.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cb-heimdall

Copy link
Copy Markdown
Collaborator

🟡 Heimdall Review Status

Requirement Status More Info
Reviews 🟡 0/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

@vercel

vercel Bot commented Sep 1, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
omni-ui Ready Ready Preview Sep 1, 2026 10:26pm UTC

Request Review

@barvian barvian changed the title fix(vibenet): unmount B20 transaction modals instead of resetting on close fix(vibenet): fix B20 transaction modals exit transitions Sep 1, 2026
@barvian
barvian marked this pull request as draft September 1, 2026 23:53
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