Page MenuHomePhabricator

[web] Introduce onClearText to Search
ClosedPublic

Authored by inka on Jun 21 2023, 5:17 AM.
Tags
None
Referenced Files
F1669504: D8273.id28155.diff
Sat, Apr 27, 10:33 AM
F1669500: D8273.id27925.diff
Sat, Apr 27, 10:33 AM
F1669480: D8273.id.diff
Sat, Apr 27, 10:31 AM
F1669456: D8273.diff
Sat, Apr 27, 10:17 AM
Unknown Object (File)
Mar 28 2024, 6:18 AM
Unknown Object (File)
Mar 28 2024, 6:18 AM
Unknown Object (File)
Mar 28 2024, 6:17 AM
Unknown Object (File)
Mar 28 2024, 6:13 AM
Subscribers

Details

Summary

issue: https://linear.app/comm/issue/ENG-3466/add-some-state-for-keeping-the-search-query
In message search when a user clears the query we will need to remove the query from context. For this I need to ba able to pass a callback to the Search component. The flex: 1 was added because in
message search modal we want to have a button next to it, and have the two of them take up the full width of the modal.

Test Plan

Tested that a provided callback is called on clear

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

inka requested review of this revision.Jun 21 2023, 5:34 AM
inka retitled this revision from [web] Intirduce onClearText to Search to [web] Introduce onClearText to Search.
This revision is now accepted and ready to land.Jun 23 2023, 2:06 AM