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

[SES-540] Add 50dp to isScrolledToBottom #1228

Merged

Conversation

bemusementpark
Copy link

This PR improves the UX by hiding the scroll to bottom button when you are less than 50dp away from the bottom.

@bemusementpark bemusementpark changed the title Add 50dp to isScrolledToBottom [SES-540] Add 50dp to isScrolledToBottom Jun 2, 2023
@burtonemily burtonemily self-assigned this Jun 5, 2023
@burtonemily
Copy link
Collaborator

Looks good ✅

@mpretty-cyro mpretty-cyro requested review from hjubb and removed request for mpretty-cyro June 13, 2023 05:12
@bemusementpark bemusementpark merged commit cc015c4 into oxen-io:dev Jun 14, 2023
@bemusementpark bemusementpark deleted the add-50dp-isScrolledToBottom branch June 14, 2023 01:04
burtonemily pushed a commit to burtonemily/session-android that referenced this pull request Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants