add isClearableCallback

NYPL · Nov 1, 2023 · d57618c · d57618c
1 parent 7ee598a
commit d57618c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/src/components/SearchForm/SearchForm.test.tsx b/src/components/SearchForm/SearchForm.test.tsx
@@ -20,6 +20,8 @@ describe("SearchForm", () => {
     const input = screen.getByRole("textbox")
     await userEvent.clear(input)
   })
+  it.todo("searches on an empty keyword after clearing the form")
+  it.todo("searches for {TBD} on an empty query")
   it("submits a keyword query by default", async () => {
     render(<SearchForm />)
     const input = screen.getByRole("textbox")

diff --git a/src/components/SearchForm/SearchForm.tsx b/src/components/SearchForm/SearchForm.tsx
@@ -66,6 +66,7 @@ const SearchForm = () => {
             textInputProps={{
               isClearable: true,
               onChange: (e) => handleChange(e, setSearchTerm),
+              isClearableCallback: () => setSearchTerm(""),
               value: searchTerm,
               labelText:
                 "Search by keyword, title, journal title, or author/contributor",