Skip to content

feat: consolidate dashboard-style form fields - #152

Open
barvian wants to merge 7 commits into
mainfrom
cursor/75243cd4
Open

feat: consolidate dashboard-style form fields#152
barvian wants to merge 7 commits into
mainfrom
cursor/75243cd4

Conversation

@barvian

@barvian barvian commented Sep 3, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Add shared Base UI Field, Input, InputGroup, Radio / RadioGroup with dashboard-style inset-ring chrome, and point Vibenet forms (create account, transact, faucet, B20, explorer sessions) at them instead of one-off controls.
  • Align Select and AccountSwitcher with the same inset ring (softer focus color; no ink ring), and convert Snapshots network/preset cards to RadioGroup.
  • Keyboard focus on radios and selects uses a blue outline offset 4px so it stays distinct from the selected ring.

Test plan

  • Create Account: name, type/model radios, salt InputGroup, key rows (select / keyboard focus / delete)
  • Transact modal: From switcher, recipient/amount fields, metadata input, Gas select
  • Faucet recipient field
  • B20 deploy + create policy radio cards and inputs
  • Snapshots: switch network and preset via keyboard and click; command still updates
  • Explorer account → Session keys: Chain / Expiry (and sibling) selects — tab to confirm the offset blue outline
  • Dark mode: inset rings and select focus color (white/40) still read correctly

Made with Cursor

@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 3, 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 3, 2026 9:15pm UTC

Request Review

Keep the faucet recipient autocomplete from main and drop the
old Input conflict so the field uses the shared InputGroup chrome.

Co-authored-by: Cursor <cursoragent@cursor.com>
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