Page MenuHomePhabricator

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

Authored by atul on Sep 13 2023, 10:09 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Nov 3, 4:44 AM
Unknown Object (File)
Sun, Oct 27, 5:33 PM
Unknown Object (File)
Sat, Oct 19, 2:14 AM
Unknown Object (File)
Sat, Oct 19, 2:14 AM
Unknown Object (File)
Sat, Oct 19, 2:14 AM
Unknown Object (File)
Sat, Oct 19, 2:07 AM
Unknown Object (File)
Sun, Oct 13, 1:27 AM
Unknown Object (File)
Sat, Oct 12, 7:43 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.


Depends on D9184

Test Plan

ChatThreadList + search experience continue to work as expected.

NOTE: Specifically tried tapping the tab and ensuring the scrollToTop functionality worked as expected.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage