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 bottom nav showing up in app-reports on orientation changes #453

Merged
merged 1 commit into from
May 3, 2024
Merged

Conversation

starry-shivam
Copy link
Contributor

@starry-shivam starry-shivam commented May 3, 2024

Closes #452

@Jean-BaptisteC
Copy link
Contributor

Is it the right fix ?
Bottom bar continue to be visible and disappear after 1 second

@starry-shivam
Copy link
Contributor Author

starry-shivam commented May 3, 2024

Is it the right fix ? Bottom bar continue to be visible and disappear after 1 second

It's because the visibility is set inside the withEndAction() callback in hideBottomNavigation(), which means it waits for the animation to finish and then changes the visibility asynchronously. This results in a split-second delay. This could be fixed by setting the visibility before starting the animation, but doing so would make the animation feel choppy. Personally, I don't think it's a significant issue, as not many people constantly rotate their device while using a certain app, and even if they do, it's not like the bottom navigation bar keeps showing up. Therefore, I think it's not worth breaking the animation just to fix this.

If I come up with a better solution later, maybe I'll create a PR. But for now, I don't know any better solution than this. One reason might be that I haven't worked much with the XML view system, as I primarily work with Jetpack Compose. So, I may lack enough experience to handle this properly.

Copy link
Contributor

@Jean-BaptisteC Jean-BaptisteC left a comment

Choose a reason for hiding this comment

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

Thanks
Maybe one day we migrate to Jetpack Compose ;)

@Jean-BaptisteC Jean-BaptisteC merged commit 2fc1be1 into Exodus-Privacy:master May 3, 2024
1 check 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.

Bottom navigation is present in app reports
2 participants