-
Notifications
You must be signed in to change notification settings - Fork 202
Open
Labels
bugSomething isn't workingSomething isn't working
Description
Problem
Token icons are missing from swap, limit order, TWAP, and stop-loss transaction cards. Only text symbols show (e.g. "ETH > USDC") with no logos.
tk ticket: sa-midv
Figma: https://www.figma.com/design/GeIQBjsXPrqJRYLVCzcjdV/AI-UX-Flows?node-id=2861-3112
Root Cause
Icons were never implemented in TxStepCardSwapPair. The component only accepts fromSymbol/toSymbol string props — no icon props exist. The vanilla JS migration is unrelated.
The icon data IS available: swapData.sellAsset.icon and swapData.buyAsset.icon carry valid URLs, and AssetIcon component already exists and is used elsewhere (portfolio, tx history, asset search).
Fix
TxStepCard.tsx— AddfromIcon/toIconoptional props toTxStepCardSwapPair, renderAssetIcon- All 4 UI components — Pass icon URLs from tool output:
InitiateSwapUI.tsxLimitOrderUI.tsxStopLossUI.tsxTwapUI.tsx
Acceptance Criteria
- TxStepCardSwapPair renders AssetIcon for both sell and buy assets
- All 4 UI components pass icon URLs
- Graceful fallback to letter initials when URL missing/fails
- Visual appearance matches Figma reference
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working
Type
Projects
Status
Backlog