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

BUG: Flicker on returning from Dashboard to main screen (Android) #475

Open
diarmidmackenzie opened this issue Apr 12, 2020 · 8 comments
Open
Labels
good first issue Good for newcomers UI/UX UI or UX related

Comments

@diarmidmackenzie
Copy link

When I move from the Dashboard to the main screen using the back button (either in the app, or the Android back button, the part of the screen at the bottom with the link to May Clinic flickers white.

Not a big deal, but doesn't look great. I don't know if everyone sees this or just my phone, but for me it happens every time.

v0.9.1

OS: androif v28;360x760.

Motorola G7 power.

I don't have any screen video recording caopability, and can't time a screenshot correctly. If you can't see this on your Android phone, please let me know what diags you'd like from a repro.

@tremblerz tremblerz added the UI/UX UI or UX related label Apr 12, 2020
@NethravathiPuttraju
Copy link
Contributor

I can look into this issue .

@troach-sf
Copy link
Contributor

@NethravathiPuttraju I've looked a bit into this working on a separate issue #496. The flicker appears to be due to StatusBar going from transulcent = false on the settings screen, to translucent = true on the main screen. The flicker goes away if we use translucent = true in the NavigationBarWrapper. If we set to true here, we need to then create a paddingTop, as setting to translucent moves content behind the status bar.

@NethravathiPuttraju
Copy link
Contributor

@troach-salesforce Okay.. So, have you resolved this issue and raised PR?

@troach-sf
Copy link
Contributor

I have not resolved it. If we go with the solution I outlined above, we would need to calculate the proper paddingTop size to set. 24 would work in most cases, but we need to be mindful of notched Android devices where the status bar size may be different. I can try to do some testing today. While the flicker doesn't look great, it only lasts for a split second and probably shouldn't block more pertinent issues.

@diarmidmackenzie
Copy link
Author

Agree - the flicker makes the app look a little unpolished, but if it's not an easy fix, there are definitely other things that we should prioritize over this.

@penrods penrods added the Low Priority Still gathering info; low impact; unclear label Apr 18, 2020
@summetj
Copy link
Collaborator

summetj commented May 10, 2020

@diarmidmackenzie - Is this still an issue or has it been fixed already?

@diarmidmackenzie
Copy link
Author

This is still happening, even in the latest v1.1 debug builds.

@summetj summetj added the good first issue Good for newcomers label May 11, 2020
@Patrick-Erichsen Patrick-Erichsen removed the Low Priority Still gathering info; low impact; unclear label May 22, 2020
@Patrick-Erichsen
Copy link
Contributor

Yup - this is still a good first issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers UI/UX UI or UX related
Projects
None yet
Development

No branches or pull requests

7 participants