Page MenuHomePhabricator

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

Authored by atul on Sep 13 2023, 10:09 AM.
Tags
None
Referenced Files
F3349586: D9188.diff
Fri, Nov 22, 6:15 PM
F3348339: D9188.diff
Fri, Nov 22, 2:29 PM
Unknown Object (File)
Thu, Nov 14, 11:50 PM
Unknown Object (File)
Sun, Nov 3, 4:44 AM
Unknown Object (File)
Sun, Oct 27, 5:33 PM
Unknown Object (File)
Oct 19 2024, 2:14 AM
Unknown Object (File)
Oct 19 2024, 2:14 AM
Unknown Object (File)
Oct 19 2024, 2:14 AM
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