Clarify hidden balance UI - #6191
Conversation
📸 Test evidencetotal balance visible
unhide balance toast
balance restored
Captured by the agent's in-app test run (build-and-test). |
2d9881d to
0e0ecd2
Compare
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Rename the balance card title to "Unhide Balance" while balances are hidden, and show a toast when they become hidden so users know how to bring them back instead of contacting support.
Adds balanceCardToggle and balanceCardTitle so maestro can drive the show/hide balance control; the card groups its children for accessibility, so text selectors do not resolve.
0e0ecd2 to
6cf399c
Compare
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
Claude Code Review
Claude Code Review is paused for this repository. To reconnect it, an admin of this repository's GitHub organization (or the account owner, for personal repositories) who can also manage your Claude organization's Code Review settings needs to re-link GitHub in Code Review settings. This is a one-time step.
Tip: disable this comment in your organization's Code Review settings.
📸 Test evidence (after fix)total balance visible
unhide balance toast
balance restored
Captured by the agent's in-app test run (build-and-test). |
|
Approved as minor UI, force-merging. |
1 participant
Description
Users hide their account balance, usually by accident, and then see
$ ●●●● USDunder a "Total Balance" title with nothing on screen telling them how to get the
number back. That has been a steady source of "all my money is gone" support
tickets.
back to "Total Balance" when they are visible. The title keeps its normal
color; the eye icon, the dots and the tap target are unchanged.
to show them again." It fires from
toggleAccountBalanceVisibility(), so itcovers every entry point (home/assets balance card, wallet detail header,
request scene), and it doubles as the notice that the feature just activated,
which is what the ticket users missed.
Scope follows the operator's latest direction on the task: rename the title,
keep the toast, and leave the tap area alone (the earlier
reduce-the-tap-area proposal was withdrawn).
Also in this branch: pre-existing lint fixes in
BalanceCard.tsx(its owncommit), and
testIDs on the balance toggle so maestro can drive it (the cardgroups its children for accessibility, so text selectors do not resolve).
Asana: https://app.asana.com/0/1215088146871429/1209717892385291
CHANGELOG
Does this branch warrant an entry to the CHANGELOG?
Dependencies
none
Requirements
If you have made any visual changes to the GUI. Make sure you have: