-
Notifications
You must be signed in to change notification settings - Fork 107
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(IT Wallet): [SIW-1734] Add "back to top" in the wallet home screen #6491
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #6491 +/- ##
==========================================
- Coverage 48.42% 48.30% -0.13%
==========================================
Files 1488 1578 +90
Lines 31617 32265 +648
Branches 7669 7389 -280
==========================================
+ Hits 15311 15585 +274
- Misses 16238 16625 +387
+ Partials 68 55 -13
... and 1715 files with indirect coverage changes Continue to review full report in Codecov by Sentry.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Short description
This PR adds the ability to return to the top of the wallet screen when tapping on the wallet tab navigation icon
List of changes proposed in this pull request
useTabItemPressWhenScreenActive
handler inWalletHomeScreen
How to test
Navigate to the Wallet home screen, scroll to the bottom, tap on the tab navigation icon and check that the scroll view scrolls back to the top
Preview
RPReplay_Final1733305813.MP4