[native] Lift `onScroll` to `ConnectedChatThreadList`
ClosedPublic
Actions

Authored by atul on Sep 12 2023, 1:57 PM.

Details

Reviewers

ginsu
tomek
rohan

Commits

rCOMM3a4e9c686ee5: [native] Lift `onScroll` to `ConnectedChatThreadList`

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 D9136

Test Plan

ChatThreadList + search experience continue to work as expected.

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Sep 12 2023, 1:57 PM

Herald added a subscriber: ashoat. · View Herald TranscriptSep 12 2023, 1:57 PM

atul added a child revision: D9164: [native] Lift `onSwipeableWillOpen` and `composeThread` to `ConnectedChatThreadList`.Sep 12 2023, 2:06 PM

atul published this revision for review.Sep 12 2023, 2:07 PM

Harbormaster completed remote builds in B22549: Diff 30995.Sep 12 2023, 2:22 PM

please see inline comment before landing

native/chat/chat-thread-list.react.js
129	I think this should be `mixed`

This revision is now accepted and ready to land.Sep 12 2023, 9:37 PM

atul added inline comments.Sep 13 2023, 7:36 AM

native/chat/chat-thread-list.react.js
129	That's fair. In a couple diffs though I completely remove these props after merging everything into a single component, so I think it's fine to leave as-is for now since this is temporary?