diff --git a/client/modules/IDE/components/Searchbar/Searchbar.jsx b/client/modules/IDE/components/Searchbar/Searchbar.jsx index a7c79d1f46..fdb52d8a16 100644 --- a/client/modules/IDE/components/Searchbar/Searchbar.jsx +++ b/client/modules/IDE/components/Searchbar/Searchbar.jsx @@ -1,71 +1,66 @@ import PropTypes from 'prop-types'; -import React from 'react'; +import React, { useState, useEffect, useCallback } from 'react'; import { throttle } from 'lodash'; import { withTranslation } from 'react-i18next'; import i18next from 'i18next'; import SearchIcon from '../../../../images/magnifyingglass.svg'; -class Searchbar extends React.Component { - constructor(props) { - super(props); - this.state = { - searchValue: this.props.searchTerm - }; - this.throttledSearchChange = throttle(this.searchChange, 500); - } +const Searchbar = ({ + searchTerm, + setSearchTerm, + resetSearchTerm, + searchLabel = i18next.t('Searchbar.SearchSketch'), + t +}) => { + const [searchValue, setSearchValue] = useState(searchTerm); - componentWillUnmount() { - this.props.resetSearchTerm(); - } + const throttledSearchChange = useCallback( + throttle((value) => setSearchTerm(value.trim()), 500), + [setSearchTerm] + ); - handleResetSearch = () => { - this.setState({ searchValue: '' }, () => { - this.props.resetSearchTerm(); - }); - }; + useEffect( + () => () => { + resetSearchTerm(); + }, + [resetSearchTerm] + ); - searchChange = () => { - this.props.setSearchTerm(this.state.searchValue.trim()); + const handleResetSearch = () => { + setSearchValue(''); + resetSearchTerm(); }; - handleSearchChange = (e) => { - this.setState({ searchValue: e.target.value }, () => { - this.throttledSearchChange(this.state.searchValue.trim()); - }); + const handleSearchChange = (e) => { + const { value } = e.target; + setSearchValue(value); + throttledSearchChange(value.trim()); }; - render() { - const { searchValue } = this.state; - return ( -
-
-
- +
+
- ); - } -} + + +
+ ); +}; Searchbar.propTypes = { searchTerm: PropTypes.string.isRequired,