-
Notifications
You must be signed in to change notification settings - Fork 97
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
Hide/Show pin banner based on scroll direction #3080
Conversation
Generated by 🚫 Danger Swift against b147428 |
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.
Nice!
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #3080 +/- ##
===========================================
- Coverage 78.01% 78.00% -0.02%
===========================================
Files 703 703
Lines 54362 54420 +58
===========================================
+ Hits 42410 42449 +39
- Misses 11952 11971 +19
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
but the transition does not work properly
and improved the hide/show behaviour based on the scroll direction
and fixed a bug with the indicator and improved the tests for it
4b4cd43
to
b147428
Compare
Quality Gate passedIssues Measures |
Simulator.Screen.Recording.-.iPhone.15.-.2024-07-23.at.17.47.46.mp4