Skip to content

Commit 6522b8e

Browse files
committed
add autocompletes on search field
1 parent b4fe4b7 commit 6522b8e

File tree

3 files changed

+273
-126
lines changed

3 files changed

+273
-126
lines changed

web/src/components/toolbar/filters/autocomplete-filter.tsx

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -14,11 +14,9 @@ import { CaretDownIcon } from '@patternfly/react-icons';
1414
import * as _ from 'lodash';
1515
import * as React from 'react';
1616
import { createFilterValue, FilterDefinition, FilterOption, FilterValue } from '../../../model/filters';
17-
import { autoCompleteCache } from '../../../utils/autocomplete-cache';
1817
import { getHTTPErrorDetails } from '../../../utils/errors';
1918
import { undefinedValue } from '../../../utils/filter-definitions';
2019
import { Indicator } from '../../../utils/filters-helper';
21-
import { usePrevious } from '../../../utils/previous-hook';
2220
import './autocomplete-filter.css';
2321

2422
const optionsMenuID = 'options-menu-list';
@@ -49,16 +47,9 @@ export const AutocompleteFilter: React.FC<AutocompleteFilterProps> = ({
4947
const searchInputRef = React.useRef<HTMLInputElement | null>(null);
5048
const optionsRef = React.useRef<HTMLDivElement | null>(null);
5149
const [options, setOptions] = React.useState<FilterOption[]>([]);
52-
const previousFilterDefinition = usePrevious(filterDefinition);
5350

5451
React.useEffect(() => {
55-
if (filterDefinition !== previousFilterDefinition) {
56-
//reset filter value if definition has changed
57-
resetFilterValue();
58-
searchInputRef.current?.focus();
59-
searchInputRef.current?.setAttribute('autocomplete', 'off');
60-
autoCompleteCache.clear();
61-
} else if (_.isEmpty(currentValue)) {
52+
if (_.isEmpty(currentValue)) {
6253
setIndicator(ValidatedOptions.default);
6354
} else {
6455
//update validation icon on field on value change

web/src/components/toolbar/filters/filter-search-input.css

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
#filter-search-input, #filter-search-form {
1+
#filter-search-input, #filter-popper {
22
min-width: 400px !important;
33
}
44

0 commit comments

Comments
 (0)