Page MenuHomePhabricator

[native] Add loading indicator at the bottom of ChatThreadList
ClosedPublic

Authored by ashoat on Dec 11 2024, 3:51 PM.
Tags
None
Referenced Files
F6089112: D14143.id46369.diff
Mon, Apr 21, 1:01 AM
F6080665: D14143.id46443.diff
Sun, Apr 20, 7:52 PM
F6061227: D14143.diff
Sun, Apr 20, 9:59 AM
Unknown Object (File)
Mon, Apr 7, 11:57 PM
Unknown Object (File)
Mar 11 2025, 4:17 PM
Unknown Object (File)
Mar 11 2025, 4:12 PM
Unknown Object (File)
Feb 26 2025, 4:45 AM
Unknown Object (File)
Feb 26 2025, 4:45 AM
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

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable