-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[$500] When scrolling backwards the messages with longer chats stuck and need to scroll forward #30895
Comments
Job added to Upwork: https://www.upwork.com/jobs/~01dc47a9e2814684af |
Triggered auto assignment to @mallenexpensify ( |
Bug0 Triage Checklist (Main S/O)
|
Triggered auto assignment to Contributor-plus team member for initial proposal review - @jjcoffee ( |
📣 @MADahab! 📣
|
Contributor details I had a similar issue but it was related to posts not chats but it might be the same idea if the reason is as u state that it loads one previous page at a time, |
✅ Contributor details stored successfully. Thank you for contributing to Expensify! |
👋🏻 |
📣 @jjcoffee 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
As the assigned BZ, I'm going to 'hang tight' as this has been addressed and worked on before. Per the slack chat, @perunt has experience/knowledge here so I'll wait til they post and follow along. |
@mallenexpensify, @jjcoffee, @perunt Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
1 similar comment
@mallenexpensify, @jjcoffee, @perunt Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@perunt , can you provide an update please? |
Hey there! Sorry for getting back to you late. |
@mallenexpensify, @jjcoffee, @perunt Still overdue 6 days?! Let's take care of this! |
Unassigning since I don't think I'm needed here! |
@mallenexpensify @perunt this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ and will need to go internal. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks! |
It was resolved with #31288 |
I tested, it appears to be resolved. There are times where, when scrolling up fast, it lags for a moment before the chats populate, I think that's expected behaviour. @quinthar if you continue to encounter, please comment and reopen the GH. |
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: 1.3.95-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:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @quinthar
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1699164509427949
Action Performed:
Expected Result:
Messages should load without stuck
Actual Result:
If I scroll backwards very quickly, it will get "stuck" -- and I need to scroll forward and then backwards again to get it to load the new page. I think it might be that it loads one previous page at a time, and if I scroll quickly enough to get to the top of that page's "skeleton" before it displays, then it doesn't register me being at the top of it. I need to scroll down to the "next" page, and then scroll back to the "previous" before it gets the one before. I would expect it should never get stuck while scrolling backwards.
Workaround:
unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Add any screenshot/video evidence
Recording.202.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: