fix(trade): lock the order ticket until a funded account exists - #2421
Conversation
With no wallet, no account, or an unfunded account, the direction toggle, size input, leverage slider, and receipt could all be interacted with even though the only real action is the CTA at the bottom (Connect / Start Trading / Deposit to Trade). Wrap the interactive stack in a disabled fieldset (native input+button+keyboard disabling) dimmed to 40% opacity so the ticket reads as locked until the account is ready. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
1 participant
Item 1 of the terminal UX fixes: with no wallet / no account / an unfunded account, the whole order ticket (direction, size, leverage, receipt) is now visibly locked — a disabled
<fieldset>at 40% opacity — instead of accepting input that can only dead-end. The bottom CTA (Connect Wallet / Start Trading / Deposit to Trade) and the inline deposit card remain fully interactive, and mock mode (?mock=1) is unaffected since it supplies a mock account.Verified:
tsc --noEmitclean; component suite green except the 4 pre-existing Privy/ConnectButton env failures.🤖 Generated with Claude Code