[native] Lift `onSwipeableWillOpen` and `composeThread` to `ConnectedChatThreadList`
ClosedPublic
Actions

Authored by atul on Tue, Sep 12, 2:06 PM.

Details

Reviewers

ginsu
tomek
rohan

Commits

rCOMM5facba63469a: [native] Lift `onSwipeableWillOpen` and `composeThread` to…

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 D9161

Test Plan

ChatThreadList + search experience continue to work as expected.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

atul created this revision.Tue, Sep 12, 2:06 PM

Herald added a subscriber: ashoat. · View Herald TranscriptTue, Sep 12, 2:06 PM

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

Harbormaster completed remote builds in B22554: Diff 31001.Tue, Sep 12, 2:30 PM

atul added a child revision: D9166: [native] Lift `renderSearch` to `ConnectedChatThreadList`.Tue, Sep 12, 3:06 PM

please see inline comments before landing

native/chat/chat-thread-list.react.js
130 ↗	(On Diff #31001)	`mixed`
131 ↗	(On Diff #31001)	`mixed`

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

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

native/chat/chat-thread-list.react.js
130 ↗	(On Diff #31001)	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?