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

Back button doesn't work if search tab is opened #18

Open
martyniukkk opened this issue Dec 28, 2024 · 1 comment
Open

Back button doesn't work if search tab is opened #18

martyniukkk opened this issue Dec 28, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@martyniukkk
Copy link
Contributor

When I open search tab and click back button - nothing happens

Screen_recording_20241228_233137.mp4
@shorthouse
Copy link
Owner

Hi @martyniukkk, thanks for raising this issue. I've taken a look at this and it turns out the SearchBar() composable I've used in this screen that is provided in the Compose standard library overrides the back button using a BackHandler. I've tried overriding this with an onNavigateUp() call but it seems that the bottom navigation bar somehow blocks this solution from working.

The combination of a SearchBar inside of a bottom navigation bar seems to be an unhandled interaction that has this bug. Aside from rewriting the search screen to no longer use the SearchBar() composable I'm unsure how else this could be solved 🤔

@shorthouse shorthouse added the bug Something isn't working label Jan 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants