Skip to content
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

Fixes scroll TOP/BOTTOM scrolling #337

Conversation

EsmaeelNabil
Copy link
Contributor

@EsmaeelNabil EsmaeelNabil commented Jul 5, 2024

Summary

  • Fixes scroll to the top
  • Fixes scroll to the bottom

Screenshot/Gif

scroll-fix.mp4

@EsmaeelNabil EsmaeelNabil force-pushed the droidcon/esmaeel-fix-scroll-to-top branch 2 times, most recently from 9be617c to 83d7188 Compare July 5, 2024 07:42
@EsmaeelNabil EsmaeelNabil changed the title Fixes scroll to the top of the screen Fixes scroll TOP/BOTTOM scrolling Jul 5, 2024
@alisen
Copy link
Member

alisen commented Jul 5, 2024

Ktlint check is failing, could you check that please?

Fixes scroll to the top by

- using the device actual screen height instead of a hardcoded value
@EsmaeelNabil EsmaeelNabil force-pushed the droidcon/esmaeel-fix-scroll-to-top branch from 83d7188 to fe83018 Compare July 5, 2024 09:17
@louis993546 louis993546 added this pull request to the merge queue Jul 5, 2024
Merged via the queue into gdg-berlin-android:main with commit c1810d2 Jul 5, 2024
5 checks passed
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.

3 participants