Skip to content

fix(ui): move hide-balance icon into balance cards#3453

Merged
CharlVS merged 6 commits intodevfrom
local/hide-balance-icon-relocation
Mar 21, 2026
Merged

fix(ui): move hide-balance icon into balance cards#3453
CharlVS merged 6 commits intodevfrom
local/hide-balance-icon-relocation

Conversation

@CharlVS
Copy link
Copy Markdown
Collaborator

@CharlVS CharlVS commented Mar 21, 2026

Summary

  • Relocated the hide-balance eye icon from a dedicated row above the statistic cards into the balance cards themselves, saving ~48px of vertical space on both mobile and desktop
  • Desktop: icon is now inline with the "Your Balance" caption in the first StatisticCard
  • Mobile: icon is positioned in the top-right corner of the BalanceSummaryWidget card

Test plan

  • Verify the eye icon appears inline with "Your Balance" caption on desktop
  • Verify the eye icon appears in the top-right corner of the balance card on mobile
  • Confirm tapping the icon toggles balance visibility across all screens
  • Confirm the old dedicated row is gone and vertical space is reclaimed
  • Check icon doesn't overlap text on narrow screens or with long locale strings

🤖 Generated with Claude Code

CharlVS and others added 4 commits March 20, 2026 01:39
…pace

Embed the privacy toggle directly in the balance card instead of a
dedicated row, saving ~48px of vertical space on both mobile and desktop.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 21, 2026

Visit the preview URL for this PR (updated for commit d1c9dcd):

https://walletrc--pull-3453-merge-wm2huk0b.web.app

(expires Sat, 28 Mar 2026 17:06:00 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: f66a4ff03faa546f12f0ae5a841bd9eff2714dcc

@CharlVS CharlVS merged commit d424f9c into dev Mar 21, 2026
5 of 12 checks passed
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.

1 participant