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

fix: Don't focus search query when returning to the results list #824

Merged
merged 1 commit into from
Jul 15, 2024

Conversation

nikclayton
Copy link
Contributor

Previous code always focused the search query. This meant that if the user:

  1. Searched for something
  2. Opened a result (post, hashtag, account)
  3. Navigated back to the search results

then because the query was focused the soft-keyboard would open, obscuring the list of results. The user had to press "Back" again to dismiss the keyboard.

New code only focuses the search query view if it is empty. This allows the user to come back to the list of results and immediately open a new result.

Previous code always focused the search query. This meant that if the
user:

1. Searched for something
2. Opened a result (post, hashtag, account)
3. Navigated back to the search results

then because the query was focused the soft-keyboard would open,
obscuring the list of results. The user had to press "Back" again
to dismiss the keyboard.

New code only focuses the search query view if it is empty. This allows
the user to come back to the list of results and immediately open a new
result.
@nikclayton nikclayton merged commit 7ef692c into pachli:main Jul 15, 2024
23 checks passed
@nikclayton nikclayton deleted the search-focus branch July 22, 2024 14:23
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.

None yet

1 participant