Skip to content

feat: show/hide balances toggle - #1379

Draft
sneurlax wants to merge 2 commits into
stagingfrom
feat/168-hide-balances
Draft

feat: show/hide balances toggle#1379
sneurlax wants to merge 2 commits into
stagingfrom
feat/168-hide-balances

Conversation

@sneurlax

Copy link
Copy Markdown
Member

closes #168

@codecov

codecov Bot commented May 28, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0.37879% with 263 lines in your changes missing coverage. Please review.
✅ Project coverage is 4.83%. Comparing base (8208fe7) to head (7818f14).
⚠️ Report is 63 commits behind head on staging.

Files with missing lines Patch % Lines
.../pages/wallets_view/sub_widgets/favorite_card.dart 0.00% 55 Missing ⚠️
..._menu/appearance_settings/appearance_settings.dart 1.92% 51 Missing ⚠️
.../appearance_settings/appearance_settings_view.dart 0.00% 34 Missing ⚠️
...s/wallet_view/sub_widgets/wallet_summary_info.dart 0.00% 33 Missing ⚠️
lib/widgets/hideable_balance.dart 0.00% 26 Missing ⚠️
...ib/pages/token_view/sub_widgets/token_summary.dart 0.00% 19 Missing ⚠️
...allet_view/sub_widgets/desktop_wallet_summary.dart 0.00% 14 Missing ⚠️
..._view/sub_widgets/firo_desktop_wallet_summary.dart 0.00% 11 Missing ⚠️
..._view/sub_widgets/mweb_desktop_wallet_summary.dart 0.00% 11 Missing ⚠️
lib/utilities/prefs.dart 0.00% 9 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           staging    #1379      +/-   ##
===========================================
- Coverage     4.84%    4.83%   -0.01%     
===========================================
  Files         1092     1093       +1     
  Lines       109141   109250     +109     
===========================================
  Hits          5283     5283              
- Misses      103858   103967     +109     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sneurlax

Copy link
Copy Markdown
Member Author

Converting to draft because I have too many open PRs ready--will take another review pass over this before re-readying it

@sneurlax
sneurlax marked this pull request as draft August 20, 2026 18:57
Adds the persisted `hideBalances` preference with mobile and desktop
appearance toggles, plus a `HideableBalance` wrapper that replaces a
balance with a labelled reveal control until it is tapped.
Wraps the wallet and token summaries, favourite cards, wallet list rows,
the balance toggle sheet, the token list and manage favourites, so the
preference holds on every screen reachable from a masked one.
@sneurlax
sneurlax force-pushed the feat/168-hide-balances branch from 5b33ccd to 0672cf8 Compare August 25, 2026 00:03
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.

[Privacy] A "show/hide" balances feature for added Privacy [enhancement]

1 participant