Page MenuHomePhabricator

[native] Lift `FlatList` to `ConnectedChatThreadList`
ClosedPublic

Authored by atul on Sep 13 2023, 9:53 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jan 17, 1:54 AM
Unknown Object (File)
Sat, Jan 11, 8:55 PM
Unknown Object (File)
Fri, Jan 10, 7:29 PM
Unknown Object (File)
Thu, Jan 9, 1:50 PM
Unknown Object (File)
Fri, Dec 27, 7:06 PM
Unknown Object (File)
Fri, Dec 27, 7:06 PM
Unknown Object (File)
Fri, Dec 27, 7:06 PM
Unknown Object (File)
Fri, Dec 27, 7:05 PM
Subscribers

Details

Summary

This diff is just one step in the process of converting ChatThreadList into a functional component. When this work is done, we will be able to avoid a lot of re-rendering and hopefully improve performance quite a bit.

This should be the second to last diff in converting ChatThreadList to a functional component. After that I'll do some cleanup/refactoring to make the component more readable (group hooks, break search into its own component, etc).


Depends on D9183

Test Plan

ChatThreadList + search experience continue to work as expected.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul published this revision for review.Sep 13 2023, 9:58 AM

One question inline

native/chat/chat-thread-list.react.js
236–238

Should this be deleted?

This revision is now accepted and ready to land.Sep 13 2023, 1:23 PM
native/chat/chat-thread-list.react.js
236–238

It gets deleted in subsequent diff