Page MenuHomePhabricator

[native] Remove `renderSearch` in `ChatThreadList` and inline `ChatThreadListSearch` directly
ClosedPublic

Authored by atul on Sep 14 2023, 1:41 PM.
Tags
None
Referenced Files
F2192861: D9208.id31222.diff
Thu, Jul 4, 8:37 PM
Unknown Object (File)
Tue, Jul 2, 9:30 PM
Unknown Object (File)
Tue, Jul 2, 8:16 PM
Unknown Object (File)
Tue, Jul 2, 9:46 AM
Unknown Object (File)
Tue, Jul 2, 3:08 AM
Unknown Object (File)
Sat, Jun 29, 5:09 PM
Unknown Object (File)
Thu, Jun 27, 6:40 PM
Unknown Object (File)
Thu, Jun 27, 6:40 PM
Subscribers

Details

Summary

Instead of passing a function to create ChatThreadListSearch we can inline the component directly.

As part of this we also replace additionalProps w/ more precise innerSearchAutoFocus and innerSearchActive props.


Depends on D9207

Test Plan

Search experience continues to look and work as expected. Set logs in all expected callbacks to ensure that they fired as expected. Made sure that x-icon and "Cancel" button continue to look and behave as before.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

atul published this revision for review.Sep 14 2023, 1:41 PM
tomek added inline comments.
native/chat/chat-thread-list.react.js
361 ↗(On Diff #31154)

We can use a shorthand

364 ↗(On Diff #31154)

We can also consider having an early return

This revision is now accepted and ready to land.Sep 18 2023, 3:58 AM
native/chat/chat-thread-list.react.js
361 ↗(On Diff #31154)

I'd prefer to be explicit here

364 ↗(On Diff #31154)

Will update to early return

address feedback: early-return