Now that loading is async and can take a bit longer, it's good to have a loading indicator visible to the user.
Depends on D14142
Paths
| Differential D14143 Authored by ashoat on Dec 11 2024, 3:51 PM.
Tags None Referenced Files
Subscribers None
Details Summary Now that loading is async and can take a bit longer, it's good to have a loading indicator visible to the user. Depends on D14142 Test Plan I tested this task by playing around with the ChatThreadList on mobile while using a stopwatch to measure how long various operations took. I tested the updated code 3 times against both the most recent Testflight build as well as master. I found that performance was approximately the same as before. I tested scrolling down, selecting threads that were read, selecting threads that were unread, and search.
Diff Detail
Event TimelineHarbormaster completed remote builds in B33044: Diff 46362.Dec 11 2024, 4:52 PM2024-12-11 16:52:59 (UTC-8) Harbormaster completed remote builds in B33051: Diff 46369.Dec 11 2024, 8:53 PM2024-12-11 20:53:50 (UTC-8) This revision is now accepted and ready to land.Dec 13 2024, 5:56 AM2024-12-13 05:56:32 (UTC-8) Closed by commit rCOMM09d091b0edd0: [native] Add loading indicator at the bottom of ChatThreadList (authored by ashoat). · Explain WhyDec 13 2024, 6:13 AM2024-12-13 06:13:58 (UTC-8) This revision was automatically updated to reflect the committed changes.
Revision Contents
Diff 46443 native/chat/chat-thread-list-utils.js
native/chat/chat-thread-list.react.js
|