Skip to content

fix: align chain and token list spacing in transact select screens#3038

Merged
ReflectiveChimp merged 2 commits intomainfrom
fix/transact-list-spacing-consistency
Apr 14, 2026
Merged

fix: align chain and token list spacing in transact select screens#3038
ReflectiveChimp merged 2 commits intomainfrom
fix/transact-list-spacing-consistency

Conversation

@dotpablocom
Copy link
Copy Markdown
Collaborator

@dotpablocom dotpablocom commented Apr 9, 2026

Summary

  • Standardize chain select list to use the same noGap + fixed 44px item height approach as the token select list
  • Reduce search-to-list gap from 16px/24px to 12px to match Figma spec
  • Remove redundant sm margin override in SelectListSearch

Test plan

  • Open a vault with cross-chain deposit support
  • Verify "Select chain to deposit from" list spacing matches "Select token to deposit"
  • Verify 12px gap between search box and first list item on both screens
  • Check both mobile and desktop breakpoints

🤖 Generated with Claude Code

Standardize chain list to use the same noGap + fixed 44px item height
approach as the token list, and reduce search-to-list gap to 12px to
match Figma.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@netlify
Copy link
Copy Markdown

netlify bot commented Apr 9, 2026

Deploy Preview for thunderous-cow-9b0405 ready!

Name Link
🔨 Latest commit caa6c05
🔍 Latest deploy log https://app.netlify.com/projects/thunderous-cow-9b0405/deploys/69d7f16b0a7d1300085a9181
😎 Deploy Preview https://deploy-preview-3038--thunderous-cow-9b0405.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Override lineHeight to 1 on ListItemBalanceUsd to reduce the visual
gap between token amount and its USD value, making them read as a
grouped pair.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@dotpablocom dotpablocom marked this pull request as ready for review April 9, 2026 19:04
@ReflectiveChimp ReflectiveChimp merged commit 37ca20c into main Apr 14, 2026
4 checks passed
@dotpablocom dotpablocom deleted the fix/transact-list-spacing-consistency branch April 15, 2026 15:37
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