-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
[$250] Android - Login – Login page blinks when open app #46232
Comments
Triggered auto assignment to @kevinksullivan ( |
We think that this bug might be related to #vip-vsp |
@kevinksullivan FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors |
ProposalPlease re-state the problem that we are trying to solve in this issue.Login page blinks in native. What is the root cause of that problem?The page blinks is caused by the sign in hand animation. In sign in hand animation, if the splash screen isn't hidden yet, we render a plain View with the same size as the lottie animation to prevent any jump because the sign in page is centered vertically.. App/src/pages/signin/SignInPageLayout/SignInHeroImage.tsx Lines 27 to 43 in 8728fc0
It was added in #35185 to improve the performance by delaying the lottie. When the splash screen is hidden, we start rendering the Lottie component. However, in Lottie component, we don't render the lottie animation immediately. App/src/components/Lottie/index.tsx Lines 37 to 47 in 8728fc0
The App/src/components/Lottie/index.tsx Lines 22 to 26 in 8728fc0
It was added in #45211 to also improve the performance by delaying the lottie. But since we render nothing for a brief moment, the page moves down and up. Placeholder view -> null -> Lottie What changes do you think we should make in order to solve the problem?We need to return a placeholder view too when the App/src/components/Lottie/index.tsx Lines 33 to 35 in 8728fc0
so we just need to add I think we can also move the isSplashHidden condition to Lottie. Lmk if you agree! What alternative solutions did you explore? (Optional)Pass a new props to Lottie to set the animationFile immediately for sign in hand animation OR Wrap the Lottie with a View with the same size. App/src/pages/signin/SignInPageLayout/SignInHeroImage.tsx Lines 35 to 43 in 8728fc0
|
@kevinksullivan Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@kevinksullivan Huh... This is 4 days overdue. Who can take care of this? |
Job added to Upwork: https://www.upwork.com/jobs/~014fc22ce9eab9144e |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @ikevin127 ( |
asking about if this fits in newdot quality here https://expensify.slack.com/archives/C05LX9D6E07/p1722554976014829 |
This comment was marked as outdated.
This comment was marked as outdated.
@bernhardoj's proposal looks good to me. The root cause was correctly identified and the main solution fixes the issue.
If you're talking about the 🎀👀🎀 C+ reviewed |
Triggered auto assignment to @MariaHCD, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
cc @MariaHCD for final assignment review when you get a chance, thank you! 🧑💻 |
📣 @ikevin127 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
PR is ready cc: @ikevin127 |
Payment was due yesterday! |
@kevinksullivan I think we need a payment summary here. Requested in ND. |
Reporting for payment summary. |
Summary
|
$250 approved for @bernhardoj |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: 9.0.12-0
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
**Email or phone of affected tester (no customers):**[email protected]
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
App should not blink
Actual Result:
Login page blinks when open app
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6552998_1721934555623.Recording__3569.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @ikevin127The text was updated successfully, but these errors were encountered: