Page MenuHomePhabricator
Differential D9208

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

Authored by atul on Sep 14 2023, 1:41 PM.
Tags
None
Referenced Files
F3359534: D9208.id31213.diff
Sun, Nov 24, 9:35 AM
F3359493: D9208.id31214.diff
Sun, Nov 24, 9:27 AM
F3359247: D9208.diff
Sun, Nov 24, 8:04 AM
Unknown Object (File)
Wed, Nov 20, 3:43 PM
Unknown Object (File)
Wed, Nov 20, 8:13 AM
Unknown Object (File)
Tue, Nov 19, 6:02 PM
Unknown Object (File)
Sun, Oct 27, 12:55 AM
Unknown Object (File)
Sun, Oct 27, 12:55 AM
View All Files
Subscribers
ashoat

Details

Reviewers
ginsu
tomek
rohan
Commits
rCOMMa0967233d046: [native] Remove `renderSearch` in `ChatThreadList` and inline…
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
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

atul created this revision.Sep 14 2023, 1:41 PM
Herald added a subscriber: ashoat. · View Herald TranscriptSep 14 2023, 1:41 PM
atul published this revision for review.Sep 14 2023, 1:41 PM
atul updated this revision to Diff 31154.Sep 14 2023, 1:43 PM

some memoization

Harbormaster completed remote builds in B22642: Diff 31153.Sep 14 2023, 1:59 PM
Harbormaster completed remote builds in B22643: Diff 31154.Sep 14 2023, 2:03 PM
atul added a child revision: D9209: [native] Forward `searchInputRef` from `ChatThreadList` to `ChatThreadListSearch`.Sep 14 2023, 2:05 PM
tomek accepted this revision.Sep 18 2023, 3:58 AM
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
atul updated this revision to Diff 31212.Sep 18 2023, 10:22 AM

rebase

atul added inline comments.Sep 18 2023, 10:25 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

atul updated this revision to Diff 31213.Sep 18 2023, 10:25 AM

address feedback: early-return

atul updated this revision to Diff 31214.Sep 18 2023, 10:28 AM

rebase

Harbormaster completed remote builds in B22684: Diff 31212.Sep 18 2023, 10:40 AM
Harbormaster completed remote builds in B22685: Diff 31213.Sep 18 2023, 10:42 AM
Harbormaster completed remote builds in B22686: Diff 31214.Sep 18 2023, 10:45 AM
Closed by commit rCOMMa0967233d046: [native] Remove `renderSearch` in `ChatThreadList` and inline… (authored by atul). · Explain WhySep 18 2023, 11:00 AM
This revision was automatically updated to reflect the committed changes.
atul added a commit: rCOMMa0967233d046: [native] Remove `renderSearch` in `ChatThreadList` and inline….

Revision Contents
Changeset List

Diff 31153

View Options

native/chat/chat-thread-list-search.react.js

Loading...
View Options

native/chat/chat-thread-list.react.js

Loading...