Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Shows and hides keyboard when scrolling the app drawer #162

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

hayribakici
Copy link

@hayribakici hayribakici commented Oct 21, 2023

This PR is a UX enhancement when scrolling through the app drawer.

If the show keyboard option is set, it will minimize the keyboard once users start scrolling down the app drawer. Vice versa, it displays the keyboard when the user is scrolling up again.

Screen_recording_20231021_132138.webm

In order to be smooth, the MainActivity's android:windowSoftInputMode had to be set to adjustNothing. This however conflicts with PR #143 , since it would cover the search field if it set to the bottom of the screen. There are fixes for this as seen in an official android guide.

Copy link
Collaborator

@TobiTenno TobiTenno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this looks really awesome, could you rebase to make sure we don't squash some part of your changes when we rebase it ourselves?

@hayribakici
Copy link
Author

This PR is now outdated. As mentionend in the description, therefore reverting to draft. I propably won't be able to continue/fix the issue, and I would be grateful, if someone would pick this up.

@hayribakici hayribakici marked this pull request as draft December 21, 2023 23:10
@TobiTenno
Copy link
Collaborator

we're still talking over this, i don't know if it's quite gonna fit, especially with the search moving to the bottom, optionally

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants