Skip to content

UI fixes - #146

Merged
julian-CStack merged 6 commits into
stagingfrom
ui-fixes
Oct 18, 2022
Merged

julian-CStack merged 6 commits into
stagingfrom
ui-fixes

Conversation

@julian-CStack

Copy link
Copy Markdown
Collaborator
  • read from provider instead of prefs singleton for easier testing
  • hide price values in favorites and wallet summary cards when incognito enabled
  • hide price values in transaction list item card and tx details view when incognito enabled

@codecov-commenter

codecov-commenter commented Oct 17, 2022 •

Copy link
Copy Markdown

Codecov Report

Merging #146 (e1e9cb7) into staging (58fb625) will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           staging     #146      +/-   ##
===========================================
- Coverage    37.40%   37.40%   -0.01%     
===========================================
  Files          128      128              
  Lines        17932    17933       +1     
===========================================
  Hits          6707     6707              
- Misses       11225    11226       +1     
Impacted Files Coverage Δ
lib/services/notifications_service.dart 6.29% <0.00%> (-0.05%) ⬇️
lib/utilities/prefs.dart 34.78% <ø> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@julian-CStack
julian-CStack merged commit 90a80ce into staging Oct 18, 2022
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