diff --git a/CHANGELOG.md b/CHANGELOG.md index 655d73dec..d65a6bf9c 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -3,6 +3,7 @@ ## [10.1.0] IN PROGRESS +* Make Inventory search and browse query boxes expandable. Refs UIIN-2493. * Added support for `containsAny` match option in Advanced search. Refs UIIN-2486. ## [10.0.1] IN PROGRESS diff --git a/src/components/InstancesList/InstancesList.js b/src/components/InstancesList/InstancesList.js index 49a468b63..2626f3d83 100644 --- a/src/components/InstancesList/InstancesList.js +++ b/src/components/InstancesList/InstancesList.js @@ -1178,6 +1178,7 @@ class InstancesList extends React.Component { searchableIndexesPlaceholder={null} initialResultCount={INITIAL_RESULT_COUNT} initiallySelectedRecord={getItem(`${namespace}.${segment}.lastOpenRecord`)} + inputType="textarea" resultCountIncrement={RESULT_COUNT_INCREMENT} viewRecordComponent={ViewInstanceWrapper} editRecordComponent={InstanceForm} diff --git a/src/components/InstancesList/InstancesList.test.js b/src/components/InstancesList/InstancesList.test.js index 2c88470d9..2680d5c2f 100644 --- a/src/components/InstancesList/InstancesList.test.js +++ b/src/components/InstancesList/InstancesList.test.js @@ -458,10 +458,10 @@ describe('InstancesList', () => { it('should have query in search input', () => { renderInstancesList({ segment: 'instances' }); - fireEvent.change(screen.getByRole('searchbox', { name: 'Search' }), { target: { value: 'search query' } }); + fireEvent.change(screen.getByRole('textbox', { name: 'Search' }), { target: { value: 'search query' } }); fireEvent.click(screen.getAllByRole('button', { name: 'Search' })[1]); - expect(screen.getByRole('searchbox', { name: 'Search' })).toHaveValue('search query'); + expect(screen.getByRole('textbox', { name: 'Search' })).toHaveValue('search query'); }); describe('when the search option is changed', () => { @@ -549,7 +549,7 @@ describe('InstancesList', () => { await act(async () => fireEvent.change(screen.getByLabelText('Search field index'), { target: { value: qindex } })); - fireEvent.change(screen.getByRole('searchbox', { name: 'Search' }), { target: { value: _query } }); + fireEvent.change(screen.getByRole('textbox', { name: 'Search' }), { target: { value: _query } }); fireEvent.click(screen.getAllByRole('button', { name: 'Search' })[1]); const row = screen.getAllByText('ABA Journal')[0]; diff --git a/src/views/BrowseInventory/BrowseInventory.js b/src/views/BrowseInventory/BrowseInventory.js index 35dafcc1c..ec2339cad 100644 --- a/src/views/BrowseInventory/BrowseInventory.js +++ b/src/views/BrowseInventory/BrowseInventory.js @@ -159,6 +159,7 @@ const BrowseInventory = () => { changeSearchIndex={onChangeSearchIndex} selectedIndex={searchIndex} searchableIndexesPlaceholder={searchableIndexesPlaceholder} + inputType="textarea" /> { it('should call "changeSearch" when search query was changed', async () => { const { container } = renderBrowseInventory(); - await act(async () => userEvent.type(screen.getByRole('searchbox'), 'newQuery')); + await act(async () => userEvent.type(screen.getByRole('textbox'), 'newQuery')); await act(async () => userEvent.click(container.querySelector('[data-test-single-search-form-submit="true"]'))); expect(applySearch).toHaveBeenCalled();