Skip to content

Conversation

@NivinCNC
Copy link
Contributor

@NivinCNC NivinCNC commented Dec 3, 2025

Implemented search suggestions using Google's autocomplete API. Added SearchSuggestionAdapter, SearchSuggestionApi, and related UI components to display and interact with suggestions in both mobile and TV search fragments. Suggestions are fetched with debounce and can be selected to search or fill the search box.

Implemented search suggestions using Google's autocomplete API. Added SearchSuggestionAdapter, SearchSuggestionApi, and related UI components to display and interact with suggestions in both mobile and TV search fragments. Suggestions are fetched with debounce and can be selected to search or fill the search box.
Changed root layout from LinearLayout to FrameLayout in both search and search_tv fragments. Moved search suggestions RecyclerView to overlay above content for better UI layering and increased elevation for improved visibility.
@diogob003
Copy link
Contributor

I personally would like an option to disable it.

And I found a problem with encoding
Screenshot_20251203-143719~2

Copy link
Collaborator

@fire-light42 fire-light42 left a comment

Choose a reason for hiding this comment

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

Very cool pull request!

I only have minor code and UI+UX considerations.

Introduces a new setting to enable or disable search suggestions while typing. Updates UI logic to respect this setting, improves suggestion parsing, and adjusts search suggestion recycler margin for better layout.
@NivinCNC
Copy link
Contributor Author

NivinCNC commented Dec 4, 2025

Resolved.

@NivinCNC NivinCNC requested a review from fire-light42 December 4, 2025 12:04
Updated the text size of search suggestions from 16sp to 18sp for improved readability in the UI.
Copy link
Collaborator

@fire-light42 fire-light42 left a comment

Choose a reason for hiding this comment

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

This is virtually unusable on TV due to the focus loss. I am unsure if we only want the focus loss on phone layout or if we disable this feature on TV.

Introduces a clear suggestions button to the search suggestions overlay, visible on TV layouts. Refactors the suggestions overlay into a FrameLayout to support the button, updates focus handling for TV, and adds a new string resource for the button label.
@NivinCNC
Copy link
Contributor Author

NivinCNC commented Dec 5, 2025

This is virtually unusable on TV due to the focus loss. I am unsure if we only want the focus loss on phone layout or if we disable this feature on TV.

Removed focus loss for tv layout, alternatively added a clear suggestion button for tv layout to hide the suggestions and even enter or ok button also hides the suggestions in all layouts

@NivinCNC NivinCNC requested a review from fire-light42 December 5, 2025 10:59
@NivinCNC
Copy link
Contributor Author

NivinCNC commented Dec 5, 2025

Removed focus loss for tv layout, alternatively added a clear suggestion button for tv layout to hide the suggestions and even enter or ok button also hides the suggestions in all layouts

same done for emulator layout

@NivinCNC NivinCNC closed this Dec 5, 2025
@NivinCNC NivinCNC reopened this Dec 5, 2025
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