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

android: add SearchView (#584) #590

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

nickkhyl
Copy link
Contributor

-Material3 search bar opens up search suggestions/results view under the bar, but we want it to open up a full page, so create SearchView and use placeholder search bar on MainView that navigates to SearchView -Tapping on suggestions/results should open up PeerDetails, so fix PeerDetails navigation to use backstack instead of always going back to Main view

Next up: ensuring search filtering adheres to MDM requirements, and UI polish

Updates tailscale/corp#18973

Signed-off-by: kari-ts [email protected]
(cherry picked from commit 45ddef1)

@nickkhyl nickkhyl force-pushed the release-branch/1.78 branch 2 times, most recently from 768f584 to 4bc7e78 Compare December 11, 2024 20:13
-Material3 search bar opens up search suggestions/results view under the bar, but we want it to open up a full page, so create SearchView and use placeholder search bar on MainView that navigates to SearchView
-Tapping on suggestions/results should open up PeerDetails, so fix PeerDetails navigation to use backstack instead of always going back to Main view

Next up: ensuring search filtering adheres to MDM requirements, and UI polish

Updates tailscale/corp#18973

Signed-off-by: kari-ts <[email protected]>
Signed-off-by: Nick Khyl <[email protected]>
(cherry picked from commit 45ddef1)
@nickkhyl nickkhyl force-pushed the nickkhyl/searchview-1.78 branch from 97f237d to a99758b Compare December 11, 2024 20:14
@nickkhyl nickkhyl merged commit 7027d7b into release-branch/1.78 Dec 11, 2024
2 checks passed
@nickkhyl nickkhyl deleted the nickkhyl/searchview-1.78 branch December 11, 2024 20:26
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.

2 participants