EPMRPP-96394 || Code Review fix - 1

reportportal · Oct 22, 2024 · de7e8f6 · de7e8f6
1 parent 0ff7557
commit de7e8f6
Show file tree

Hide file tree

Showing 5 changed files with 11 additions and 11 deletions.
diff --git a/app/src/components/fields/searchField/searchField.jsx b/app/src/components/fields/searchField/searchField.jsx
@@ -17,7 +17,7 @@
 import { useEffect, useState } from 'react';
 import PropTypes from 'prop-types';
 import { useTracking } from 'react-tracking';
-import { FieldText, SearchIcon } from '@reportportal/ui-kit';
+import { BaseIconButton, FieldText, SearchIcon } from '@reportportal/ui-kit';
 import classNames from 'classnames/bind';
 import styles from './searchField.scss';
 
@@ -79,7 +79,11 @@ export const SearchField = ({
       onBlur={handleBlur}
       placeholder={placeholder}
       loading={isSearchActive && isLoading}
-      startIcon={<SearchIcon />}
+      startIcon={
+        <BaseIconButton>
+          <SearchIcon />
+        </BaseIconButton>
+      }
       className={cx('search-field')}
       maxLength={256}
       collapsible

diff --git a/app/src/controllers/filter/withFilter.jsx b/app/src/controllers/filter/withFilter.jsx
@@ -46,7 +46,7 @@ export const withFilter = ({ filterKey = FILTER_KEY, namespace } = {}) => (Wrapp
 
       handleFilterChange = debounce((value) => {
         this.props.updateFilter(value || undefined);
-      }, 300);
+      }, 1000);
 
       render() {
         const { filter, updateFilter, ...rest } = this.props;

diff --git a/app/src/pages/instance/organizationsPage/organizationsPageHeader/organizationsPageHeader.jsx b/app/src/pages/instance/organizationsPage/organizationsPageHeader/organizationsPageHeader.jsx
@@ -31,7 +31,7 @@ import styles from './organizationsPageHeader.scss';
 
 const cx = classNames.bind(styles);
 
-const SearchFieldWitFilter = withFilter({ filterKey: SEARCH_KEY })(SearchField);
+const SearchFieldWithFilter = withFilter({ filterKey: SEARCH_KEY })(SearchField);
 
 export const OrganizationsPageHeader = ({ isEmpty, searchValue, setSearchValue }) => {
   const { formatMessage } = useIntl();
@@ -44,7 +44,7 @@ export const OrganizationsPageHeader = ({ isEmpty, searchValue, setSearchValue }
         <div className={cx('actions')}>
           {!isEmpty && (
             <div className={cx('icons')}>
-              <SearchFieldWitFilter
+              <SearchFieldWithFilter
                 isLoading={projectsLoading}
                 searchValue={searchValue}
                 setSearchValue={setSearchValue}

diff --git a/...src/pages/instance/organizationsPage/organizationsPageHeader/organizationsPageHeader.scss b/...src/pages/instance/organizationsPage/organizationsPageHeader/organizationsPageHeader.scss
@@ -53,10 +53,6 @@
     svg {
       width: 16px;
       height: 16px;
-
-      path {
-        fill: $COLOR--e-300;
-      }
     }
 
     i {

diff --git a/...src/pages/organization/organizationProjectsPage/projectsPageHeader/projectsPageHeader.jsx b/...src/pages/organization/organizationProjectsPage/projectsPageHeader/projectsPageHeader.jsx
@@ -35,7 +35,7 @@ import userIcon from './img/user-inline.svg';
 
 const cx = classNames.bind(styles);
 
-const SearchFieldWitFilter = withFilter({ filterKey: SEARCH_KEY })(SearchField);
+const SearchFieldWithFilter = withFilter({ filterKey: SEARCH_KEY })(SearchField);
 
 export const ProjectsPageHeader = ({
   hasPermission,
@@ -87,7 +87,7 @@ export const ProjectsPageHeader = ({
         <div className={cx('actions')}>
           {isNotEmpty && (
             <div className={cx('icons')}>
-              <SearchFieldWitFilter
+              <SearchFieldWithFilter
                 isLoading={projectsLoading}
                 searchValue={searchValue}
                 setSearchValue={setSearchValue}