-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[$1000] Settings - There is a lag when transitioning between Account settings #47716
Comments
Triggered auto assignment to @jliexpensify ( |
@jliexpensify 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 |
I think same as #47041 |
Thanks @codewaseem - I think the root cause might be the same, as these issues are similar. I'll put a hold on this one, and then get Applause to re-test once #47041 is resolved. |
@jliexpensify this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
Waiting on other PR |
@lanitochka17 can you re-test please? #47041 has been deployed to production and I can't reproduce this issue anymore, |
I'm still able to reproduce the issue on my side on 9.0.45-2. There's a notable freeze when navigating to About. slow.mp4 |
@isagoico I'm able to only get 1/10 of this hang on my end, though performance profiling shows ~500ms is spent on setting up Lottie. I can also get rid of this hang completely when removing the animated sections. Could you please record a Chrome profiling session for us? Steps are here:
|
@adhorodyski Tried again today and the freeze is not as notable as when I reproduced back here. I recorded the Profile here Profile.zip. |
I've found this PR that postpones setting the asset so it delays the initiation - we can see the same in the Hermes trace as well. Looks like this was desired to offload this & now we're just experiencing it each time we load this page? useEffect(() => {
setAnimationFile(source.file);
}, [setAnimationFile, source.file]); This block runs each time we mount this component to the screen so it might be that we just re-run the whole internal pipeline for Lottie. My assumption is that we started offloading this to get the gains elsewhere and now we're "paying the price" for it, is that right? |
@jliexpensify the issue we're waiting for is closed now, what action can we take in this one? |
Hi @adhorodyski - hmm so I guess if this issue is still reproducible, we should dig into this further and try and fix the root cause. So we'd need to treat this as a separate issue to #47041. The other consideration is how bad this issue is, and whether it can actually be widely and consistently reproduced. If it's a niche issue, then maybe we can consider not fixing it? But we should probably bring this up publicly before making any decisions around it. |
It can be consistently reproduced, though I'd say it's not worth spending time on it at the moment unless we get more reports of this actually being problematic. I'm experiencing a 115-200ms (mac m1 pro) lag now due to this animation which is not that big of a deal (?) cc @mountiny curious about your opinion on this? I only want to prevent us chasing a 100ms lag on a settings page, but can dive deeper into this if we consider this being important (though we intentionally offloaded this lag to the subpages before). If no one can reproduce the big UI hangs anymore the issue is not about 'freezing the app', but rather 'experiencing a slight lag' which seems like a smaller priority thing. |
Hmm tough one. I would say we can make this external for $1000 and see if anyone can fix it in a root. I agree it might not be worth it having an agency worker on this, if we are only reproducing the lag |
|
Job added to Upwork: https://www.upwork.com/jobs/~021846341339697727091 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @s77rt ( |
Posted here |
Issue not reproducible during KI retests. (First week) |
Not overdue. Looking for proposals |
Same ^ |
📣 It's been a week! Do we have any satisfactory proposals yet? Do we need to adjust the bounty for this issue? 💸 |
@s77rt Let me confirm this error? It seems like we should fix the loading duration of the animations? |
@Anaslancer More specifically we are looking to understand why the app becomes laggy then fix it. |
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.22-5
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
Issue reported by: Applause - Internal Team
Action Performed:
Expected Result:
I should be able to navigate smoothly
Actual Result:
The app freezes after opening Account settings again on specific pages. Affects most of the pages on Account settings. The freeze can last from 2-5 seconds to a point where you have to restart the browser. Sometimes the app returns to normal after you click
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Bug6576947_1724150629250.bandicam_2024-08-20_12-36-16-922.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @s77rtThe text was updated successfully, but these errors were encountered: