Skip to content

Fix combined search results losing keyboard focus when typing#1524

Merged
damontecres merged 2 commits into
damontecres:mainfrom
zoro11031:fix/combined-search-results
Jun 5, 2026
Merged

Fix combined search results losing keyboard focus when typing#1524
damontecres merged 2 commits into
damontecres:mainfrom
zoro11031:fix/combined-search-results

Conversation

@zoro11031
Copy link
Copy Markdown
Contributor

@zoro11031 zoro11031 commented Jun 5, 2026

Description

Minor tweak to prevent the keyboard from losing focus when typing search queries when "combine search results" is enabled

Related issues

Fixes #1523

Testing

Emulator only

Screenshots

N/A

AI or LLM usage

None

Copy link
Copy Markdown
Owner

@damontecres damontecres left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

I also pushed another focus related fix. Once the user clicks search to trigger an "immediate" search, searchClicked is set to true, but doesn't ever reset. So if the user then changes the query, focus is moved down like you described in your issue. So the change just resets searchClicked to false whenever the query is changed.

@damontecres
Copy link
Copy Markdown
Owner

Also, focus bugs are the worst!

@zoro11031
Copy link
Copy Markdown
Contributor Author

Thanks for the assist! Tell me about it man, its like playing whack a mole

@damontecres damontecres merged commit 0842c35 into damontecres:main Jun 5, 2026
2 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.

Keyboard loses focus as results load when typing and "combine search results" is enabled

2 participants