X-Git-Url: https://git.arvados.org/arvados.git/blobdiff_plain/235d9456eb6679611af96383f9dfcadc3462a7da..cbe483113381e8c22b28a042479f5b7374a84744:/src/components/search-input/search-input.tsx diff --git a/src/components/search-input/search-input.tsx b/src/components/search-input/search-input.tsx index 2835f156d8..8d86307ab7 100644 --- a/src/components/search-input/search-input.tsx +++ b/src/components/search-input/search-input.tsx @@ -2,7 +2,7 @@ // // SPDX-License-Identifier: AGPL-3.0 -import React from 'react'; +import React, {useState, useEffect} from 'react'; import { IconButton, StyleRulesCallback, withStyles, WithStyles, FormControl, InputLabel, Input, InputAdornment, Tooltip } from '@material-ui/core'; import SearchIcon from '@material-ui/icons/Search'; @@ -45,84 +45,75 @@ interface SearchInputActionProps { type SearchInputProps = SearchInputDataProps & SearchInputActionProps & WithStyles; -interface SearchInputState { - value: string; - label: string; -} - -let selfClearPropState = ''; - export const DEFAULT_SEARCH_DEBOUNCE = 1000; -export const SearchInput = withStyles(styles)( - class extends React.Component { - state: SearchInputState = { - value: "", - label: "" - }; +const SearchInputComponent = (props: SearchInputProps) => { + const [timeout, setTimeout] = useState(0); + const [value, setValue] = useState(""); + const [label, setLabel] = useState("Search"); + const [selfClearProp, setSelfClearProp] = useState(""); - timeout: number; - - render() { - return
- - {this.state.label} - - - - - - - - } /> - -
; + useEffect(() => { + if (props.value) { + setValue(props.value); } - - componentDidMount() { - this.setState({ - value: this.props.value, - label: this.props.label || 'Search' - }); + if (props.label) { + setLabel(props.label); } - componentWillReceiveProps(nextProps: SearchInputProps) { - if (nextProps.value !== this.props.value) { - this.setState({ value: nextProps.value }); - } - if (this.state.value !== '' && nextProps.selfClearProp && nextProps.selfClearProp !== selfClearPropState) { - if (selfClearPropState !== '') { - this.props.onSearch(''); - } - selfClearPropState = nextProps.selfClearProp; - } - } + return () => { + setValue(""); + clearTimeout(timeout); + }; + }, [props.value, props.label]); // eslint-disable-line react-hooks/exhaustive-deps - componentWillUnmount() { - clearTimeout(this.timeout); + useEffect(() => { + if (selfClearProp !== props.selfClearProp) { + setValue(""); + setSelfClearProp(props.selfClearProp); + handleChange({ target: { value: "" } } as any); } + }, [props.selfClearProp]); // eslint-disable-line react-hooks/exhaustive-deps - handleSubmit = (event: React.FormEvent) => { - event.preventDefault(); - clearTimeout(this.timeout); - this.props.onSearch(this.state.value); - } + const handleSubmit = (event: React.FormEvent) => { + event.preventDefault(); + clearTimeout(timeout); + props.onSearch(value); + }; - handleChange = (event: React.ChangeEvent) => { - clearTimeout(this.timeout); - this.setState({ value: event.target.value }); - this.timeout = window.setTimeout( - () => this.props.onSearch(this.state.value), - this.props.debounce || DEFAULT_SEARCH_DEBOUNCE - ); + const handleChange = (event: React.ChangeEvent) => { + const { target: { value: eventValue } } = event; + clearTimeout(timeout); + setValue(eventValue); + + setTimeout(window.setTimeout( + () => { + props.onSearch(eventValue); + }, + props.debounce || DEFAULT_SEARCH_DEBOUNCE + )); + }; - } - } -); + return
+ + {label} + + + + + + + + } /> + +
; +} + +export const SearchInput = withStyles(styles)(SearchInputComponent); \ No newline at end of file