From 8a3207a1e7b2015ed252836ef142ca72ba190e47 Mon Sep 17 00:00:00 2001
From: Christophe <christophe.misc+git@protonmail.ch>
Date: Thu, 14 Sep 2023 03:08:24 +0200
Subject: [PATCH] refactor: resolve IDE warnings

---
 components/Paginator.test.tsx                 |  1 +
 components/ResultEditModal.tsx                | 11 ++----
 components/ResultReportModal.tsx              |  5 +--
 components/ResultsPerPageSelection.test.tsx   | 14 ++++----
 components/ResultsPerPageSelection.tsx        |  2 +-
 components/TagSelector/index.tsx              |  6 ++--
 components/TermsOfServiceCheck.test.tsx       | 12 ++-----
 components/forms/ErrorMessage.tsx             |  1 +
 components/loadingOverlay.tsx                 |  8 ++++-
 components/report-view/BenchmarkInfo.tsx      |  2 +-
 .../{claimInfo.tsx => ClaimInfo.tsx}          | 34 ++++++++-----------
 components/report-view/ClaimView.tsx          |  2 +-
 components/report-view/FlavorInfo.tsx         |  4 +--
 components/report-view/ResultInfo.tsx         |  2 +-
 components/report-view/SiteInfo.tsx           |  2 +-
 components/result-search/DiagramCard.tsx      |  3 +-
 components/result-search/ResultTable.tsx      |  2 +-
 .../result-search/columns/BenchmarkColumn.tsx |  5 +--
 .../columns/ExecutionDateColumn.tsx           |  6 ++--
 .../result-search/columns/SiteColumn.tsx      |  5 ++-
 .../columns/SiteFlavorColumn.tsx              |  5 ++-
 components/result-search/diagrams/chartjs.tsx |  9 ++---
 components/result-search/diagrams/echarts.tsx | 14 ++++----
 .../searchSelectors/BenchmarkSearchSelect.tsx |  6 ++--
 .../searchSelectors/FlavorSearchSelect.tsx    |  6 ++--
 .../searchSelectors/SiteSearchPopover.tsx     |  6 ++--
 components/site-editor/FlavorEditor.tsx       |  6 ++--
 .../FlavorSubmissionModal.tsx                 |  2 +-
 pages/search/result.tsx                       |  1 -
 29 files changed, 86 insertions(+), 96 deletions(-)
 rename components/report-view/{claimInfo.tsx => ClaimInfo.tsx} (50%)

diff --git a/components/Paginator.test.tsx b/components/Paginator.test.tsx
index 552ec4d..c6cf4cf 100644
--- a/components/Paginator.test.tsx
+++ b/components/Paginator.test.tsx
@@ -9,6 +9,7 @@ describe('pagination', () => {
         pages: 7,
         prev_num: 3,
         total: 7,
+        page: 4,
     };
 
     test('first', () => {
diff --git a/components/ResultEditModal.tsx b/components/ResultEditModal.tsx
index 1f47411..fcd00af 100644
--- a/components/ResultEditModal.tsx
+++ b/components/ResultEditModal.tsx
@@ -20,7 +20,7 @@ const ResultEditModal: FC<ResultEditModalProps> = ({ result, show, closeModal })
     const [selectedTags, setSelectedTags] = useState<Tag[]>([]);
 
     useEffect(() => {
-        setSelectedTags(result.tags ?? []);
+        setSelectedTags(result.tags);
     }, [result]);
 
     const { mutate } = useMutation((data: TagsIds) => api.results.updateResult(result.id, data), {
@@ -43,13 +43,8 @@ const ResultEditModal: FC<ResultEditModalProps> = ({ result, show, closeModal })
             <Modal.Body>
                 <h4>Tags</h4>
                 <TagSelector selected={selectedTags} setSelected={setSelectedTags} />
-                {result !== null && (
-                    <>
-                        <h4>Data</h4>
-                        <JsonHighlight>{JSON.stringify(result.json, null, 4)}</JsonHighlight>
-                    </>
-                )}
-                {result == null && <div className="text-muted">Loading...</div>}
+                <h4>Data</h4>
+                <JsonHighlight>{JSON.stringify(result.json, null, 4)}</JsonHighlight>
             </Modal.Body>
             <Modal.Footer>
                 <Button variant="success" onClick={submitEdit}>
diff --git a/components/ResultReportModal.tsx b/components/ResultReportModal.tsx
index 731660a..054672e 100644
--- a/components/ResultReportModal.tsx
+++ b/components/ResultReportModal.tsx
@@ -45,10 +45,7 @@ const ResultReportModal: FC<ResultReportModalProps> = ({ result, show, closeModa
                         />
                     </Form.Group>
                 </Form>
-                {result !== null && (
-                    <JsonHighlight>{JSON.stringify(result.json, null, 4)}</JsonHighlight>
-                )}
-                {result == null && <div className="text-muted">Loading...</div>}
+                <JsonHighlight>{JSON.stringify(result.json, null, 4)}</JsonHighlight>
             </Modal.Body>
             <Modal.Footer>
                 <Button variant="danger" onClick={submitReport}>
diff --git a/components/ResultsPerPageSelection.test.tsx b/components/ResultsPerPageSelection.test.tsx
index 2b746a7..17ad94f 100644
--- a/components/ResultsPerPageSelection.test.tsx
+++ b/components/ResultsPerPageSelection.test.tsx
@@ -5,19 +5,17 @@ describe('results per page selection', () => {
     test('valid option', () => {
         const onChange = jest.fn();
         const { container } = render(
-            <>
-                <ResultsPerPageSelection onChange={onChange} currentSelection={50} />
-            </>
+            <ResultsPerPageSelection onChange={onChange} currentSelection={50} />
         );
-        expect((container.querySelector('option:checked') as HTMLOptionElement).value).toBe('50');
+        const option = container.querySelector('option:checked') as HTMLOptionElement | null;
+        expect(option?.value).toBe('50');
     });
     test('invalid option', () => {
         const onChange = jest.fn();
         const { container } = render(
-            <>
-                <ResultsPerPageSelection onChange={onChange} currentSelection={621} />
-            </>
+            <ResultsPerPageSelection onChange={onChange} currentSelection={621} />
         );
-        expect((container.querySelector('option:checked') as HTMLOptionElement).value).toBe('10');
+        const option = container.querySelector('option:checked') as HTMLOptionElement | null;
+        expect(option?.value).toBe('10');
     });
 });
diff --git a/components/ResultsPerPageSelection.tsx b/components/ResultsPerPageSelection.tsx
index 160e62d..52e66c6 100644
--- a/components/ResultsPerPageSelection.tsx
+++ b/components/ResultsPerPageSelection.tsx
@@ -1,4 +1,4 @@
-import { FC } from 'react';
+import { type FC } from 'react';
 import { Col, Form, Row } from 'react-bootstrap';
 
 type ResultsPerPageSelectionProps = {
diff --git a/components/TagSelector/index.tsx b/components/TagSelector/index.tsx
index db2556d..dca404b 100644
--- a/components/TagSelector/index.tsx
+++ b/components/TagSelector/index.tsx
@@ -79,7 +79,7 @@ const TagSelector: FC<TagSelectorProps> = ({
                                 {tags.data.data.items
                                     .filter(
                                         (tag) =>
-                                            !selected.some((selected) => selected.id === tag.id)
+                                            !selected.some((otherTag) => otherTag.id === tag.id)
                                     )
                                     .map((tag) => (
                                         <PlaceholderTag key={tag.id} />
@@ -88,13 +88,13 @@ const TagSelector: FC<TagSelectorProps> = ({
                         )}
                         {(tags.isFetching || tags.isLoading || tags.isRefetching) &&
                             !tags.isPreviousData && <LoadingOverlay />}
-                        {tags.isSuccess && !tags.isPreviousData && tags.data && (
+                        {tags.isSuccess && !tags.isPreviousData && (
                             <>
                                 <div className="d-flex">
                                     {tags.data.data.items
                                         .filter(
                                             (tag) =>
-                                                !selected.some((selected) => selected.id === tag.id)
+                                                !selected.some((otherTag) => otherTag.id === tag.id)
                                         )
                                         .map((tag) => (
                                             <UnselectedTag tag={tag} select={select} key={tag.id} />
diff --git a/components/TermsOfServiceCheck.test.tsx b/components/TermsOfServiceCheck.test.tsx
index 3fe1a25..5a75af2 100644
--- a/components/TermsOfServiceCheck.test.tsx
+++ b/components/TermsOfServiceCheck.test.tsx
@@ -4,11 +4,7 @@ import { act, render, screen } from '@testing-library/react';
 describe('terms of service check', () => {
     test('check', () => {
         const onClick = jest.fn();
-        render(
-            <>
-                <TermsOfServiceCheck accepted={false} setAccepted={onClick} />
-            </>
-        );
+        render(<TermsOfServiceCheck accepted={false} setAccepted={onClick} />);
         act(() => {
             screen.getByRole('checkbox').click();
         });
@@ -16,11 +12,7 @@ describe('terms of service check', () => {
     });
     test('uncheck', () => {
         const onClick = jest.fn();
-        render(
-            <>
-                <TermsOfServiceCheck accepted={true} setAccepted={onClick} />
-            </>
-        );
+        render(<TermsOfServiceCheck accepted setAccepted={onClick} />);
         act(() => {
             screen.getByRole('checkbox').click();
         });
diff --git a/components/forms/ErrorMessage.tsx b/components/forms/ErrorMessage.tsx
index 15377c5..5b118e8 100644
--- a/components/forms/ErrorMessage.tsx
+++ b/components/forms/ErrorMessage.tsx
@@ -36,6 +36,7 @@ const ErrorMessage: FC<ErrorMessageProps> = ({ error }) => {
         } else if (error.request) {
             return <>No response from the server</>;
         } else {
+            // eslint-disable-next-line react/jsx-no-useless-fragment
             return <>{error.message}</>;
         }
     }
diff --git a/components/loadingOverlay.tsx b/components/loadingOverlay.tsx
index fbee752..506312c 100644
--- a/components/loadingOverlay.tsx
+++ b/components/loadingOverlay.tsx
@@ -22,4 +22,10 @@ type LoadingWrapperProps = {
 export const LoadingWrapper: FC<PropsWithChildren<LoadingWrapperProps>> = ({
     isLoading,
     children,
-}) => (isLoading ? <LoadingOverlay /> : <>{children}</>);
+}) => {
+    if (isLoading) {
+        return <LoadingOverlay />;
+    }
+    // eslint-disable-next-line react/jsx-no-useless-fragment
+    return <>{children}</>;
+};
diff --git a/components/report-view/BenchmarkInfo.tsx b/components/report-view/BenchmarkInfo.tsx
index 522d0ae..10c4b78 100644
--- a/components/report-view/BenchmarkInfo.tsx
+++ b/components/report-view/BenchmarkInfo.tsx
@@ -13,7 +13,7 @@ const BenchmarkInfo: FC<BenchmarkInfoProps> = ({ id }) => {
     return (
         <>
             {benchmark.isLoading && <LoadingOverlay />}
-            {benchmark.isSuccess && benchmark.data && (
+            {benchmark.isSuccess && (
                 <>
                     Image: {benchmarkLinkDisplay(benchmark.data.data)}
                     <br />
diff --git a/components/report-view/claimInfo.tsx b/components/report-view/ClaimInfo.tsx
similarity index 50%
rename from components/report-view/claimInfo.tsx
rename to components/report-view/ClaimInfo.tsx
index d8d1bda..bb23b03 100644
--- a/components/report-view/claimInfo.tsx
+++ b/components/report-view/ClaimInfo.tsx
@@ -1,48 +1,42 @@
-import { type ReactElement } from 'react';
+import { type FC } from 'react';
 import { useQuery } from 'react-query';
 import { LoadingOverlay } from 'components/loadingOverlay';
 import ResultInfo from './ResultInfo';
 import { truthyOrNoneTag } from 'components/utility';
-import { Badge } from 'react-bootstrap';
 import useApi from 'lib/useApi';
 import { type Claim } from '@eosc-perf/eosc-perf-client';
 import useUser from 'lib/useUser';
 
 type ClaimInfoProps = { id: string; claim: undefined } | { id: undefined; claim: Claim };
 
-export const ClaimInfo: React.FC<ClaimInfoProps> = (props): ReactElement => {
+const ClaimInfo: FC<ClaimInfoProps> = ({ id, claim: claimCache }) => {
     const auth = useUser();
     const api = useApi(auth.token);
 
-    const claim = useQuery(
-        ['claim', props.id],
-        () => api.reports.getClaim(props.id ?? props.claim.id),
-        {
-            enabled: props.claim === undefined,
-        }
-    );
+    const claim = useQuery(['claim', id], () => api.reports.getClaim(id ?? claimCache.id), {
+        enabled: claimCache === undefined,
+    });
 
     // use local copy if available
-    let claimData: Claim | undefined = props.claim;
-    if (props.claim === undefined && props.id !== undefined && claim.isSuccess && claim.data) {
-        claimData = claim.data.data;
-    }
+    const claimData =
+        claimCache !== undefined ? claimCache : claim.isSuccess ? claim.data.data : undefined;
 
     return (
         <>
-            {props.id === undefined && props.claim === undefined && (
-                <Badge bg="danger">No claim data available</Badge>
-            )}
             {claim.isLoading && <LoadingOverlay />}
-            {claim.isSuccess && claimData && (
+            {claimData && (
                 <>
                     Message: {truthyOrNoneTag(claimData.message)} <br />
                     Claimant: {claimData.uploader.email} <br />
-                    {claimData.resource_type === 'result' && (
+                    {
+                        //claimData.resource_type === 'result' && (
                         <ResultInfo id={claimData.resource_id} />
-                    )}
+                        //)
+                    }
                 </>
             )}
         </>
     );
 };
+
+export default ClaimInfo;
diff --git a/components/report-view/ClaimView.tsx b/components/report-view/ClaimView.tsx
index 6633ead..9404c39 100644
--- a/components/report-view/ClaimView.tsx
+++ b/components/report-view/ClaimView.tsx
@@ -1,7 +1,7 @@
 import { type FC, useState } from 'react';
 import { ListGroup } from 'react-bootstrap';
 import actionable from 'styles/actionable.module.css';
-import { ClaimInfo } from './claimInfo';
+import ClaimInfo from './ClaimInfo';
 import ClaimInteraction from './ClaimInteraction';
 import { type Claim } from '@eosc-perf/eosc-perf-client';
 
diff --git a/components/report-view/FlavorInfo.tsx b/components/report-view/FlavorInfo.tsx
index ecbcf94..bbe9ba1 100644
--- a/components/report-view/FlavorInfo.tsx
+++ b/components/report-view/FlavorInfo.tsx
@@ -20,7 +20,7 @@ const FlavorInfo: FC<FlavorInfoProps> = ({ id }) => {
     return (
         <>
             {flavor.isLoading && <LoadingOverlay />}
-            {flavor.isSuccess && flavor.data && (
+            {flavor.isSuccess && (
                 <>
                     Name: {flavor.data.data.name}
                     <br />
@@ -30,7 +30,7 @@ const FlavorInfo: FC<FlavorInfoProps> = ({ id }) => {
                     <hr />
                     Site:
                     <br />
-                    {site.isSuccess && site.data && <SiteInfo id={site.data.data.id} />}
+                    {site.isSuccess && <SiteInfo id={site.data.data.id} />}
                 </>
             )}
         </>
diff --git a/components/report-view/ResultInfo.tsx b/components/report-view/ResultInfo.tsx
index 24b3a41..17e7750 100644
--- a/components/report-view/ResultInfo.tsx
+++ b/components/report-view/ResultInfo.tsx
@@ -17,7 +17,7 @@ const ResultInfo: FC<ResultInfoProps> = ({ id }) => {
     return (
         <>
             {result.isLoading && <LoadingOverlay />}
-            {result.isSuccess && result.data && (
+            {result.isSuccess && (
                 <>
                     Site: {result.data.data.site.name}
                     <br />
diff --git a/components/report-view/SiteInfo.tsx b/components/report-view/SiteInfo.tsx
index ea6f9d0..43c2e64 100644
--- a/components/report-view/SiteInfo.tsx
+++ b/components/report-view/SiteInfo.tsx
@@ -13,7 +13,7 @@ const SiteInfo: FC<SiteInfoProps> = ({ id }) => {
     return (
         <>
             {site.isLoading && <LoadingOverlay />}
-            {site.isSuccess && site.data && (
+            {site.isSuccess && (
                 <>
                     Name: {site.data.data.name}
                     <br />
diff --git a/components/result-search/DiagramCard.tsx b/components/result-search/DiagramCard.tsx
index 28582c0..cfe166e 100644
--- a/components/result-search/DiagramCard.tsx
+++ b/components/result-search/DiagramCard.tsx
@@ -11,10 +11,9 @@ type DiagramCardProps = {
     results: Result[];
     benchmark?: Benchmark;
     suggestions?: Suggestion[];
-    className?: string;
 };
 
-const DiagramCard: FC<DiagramCardProps> = ({ results, benchmark, suggestions, className }) => {
+const DiagramCard: FC<DiagramCardProps> = ({ results, benchmark, suggestions }) => {
     const [selectedDiagram, setSelectedDiagram] = useState(charts.EChartsMeta.id);
 
     return (
diff --git a/components/result-search/ResultTable.tsx b/components/result-search/ResultTable.tsx
index 815e7ec..ef7cf0f 100644
--- a/components/result-search/ResultTable.tsx
+++ b/components/result-search/ResultTable.tsx
@@ -150,7 +150,7 @@ const ResultTable: FC<ResultTableProps> = ({
                             />
                         ))}
                         <th>
-                            {/* TODO: move this to hotbar above table? */}
+                            {/* TODO: move this to hot-bar above table? */}
                             <Button
                                 size="sm"
                                 variant="secondary"
diff --git a/components/result-search/columns/BenchmarkColumn.tsx b/components/result-search/columns/BenchmarkColumn.tsx
index e02dcb0..ce1212c 100644
--- a/components/result-search/columns/BenchmarkColumn.tsx
+++ b/components/result-search/columns/BenchmarkColumn.tsx
@@ -1,5 +1,5 @@
-import { FC } from 'react';
-import { Result } from '@eosc-perf/eosc-perf-client';
+import { type FC } from 'react';
+import { type Result } from '@eosc-perf/eosc-perf-client';
 
 type BenchmarkColumnProps = { result: Result };
 
@@ -10,6 +10,7 @@ type BenchmarkColumnProps = { result: Result };
  * @param props.result result data
  */
 const BenchmarkColumn: FC<BenchmarkColumnProps> = ({ result }) => (
+    // eslint-disable-next-line react/jsx-no-useless-fragment
     <>{`${result.benchmark.docker_image}:${result.benchmark.docker_tag}`}</>
 );
 
diff --git a/components/result-search/columns/ExecutionDateColumn.tsx b/components/result-search/columns/ExecutionDateColumn.tsx
index b95141a..5851e42 100644
--- a/components/result-search/columns/ExecutionDateColumn.tsx
+++ b/components/result-search/columns/ExecutionDateColumn.tsx
@@ -1,5 +1,5 @@
-import { FC, useMemo } from 'react';
-import { Result } from '@eosc-perf/eosc-perf-client';
+import { type FC, useMemo } from 'react';
+import { type Result } from '@eosc-perf/eosc-perf-client';
 import { formatDistance } from 'date-fns';
 import { OverlayTrigger, Tooltip } from 'react-bootstrap';
 import styles from './ExecutionDateColumn.module.scss';
@@ -24,7 +24,7 @@ const ExecutionDateColumn: FC<ExecutionDateColumnProps> = ({ result }) => {
         <span className={styles['underline']}>
             <OverlayTrigger
                 overlay={
-                    <Tooltip id={result.id + '-tooltip'}>
+                    <Tooltip id={`${result.id}-tooltip`}>
                         {executionDate.toLocaleString(undefined)}
                     </Tooltip>
                 }
diff --git a/components/result-search/columns/SiteColumn.tsx b/components/result-search/columns/SiteColumn.tsx
index 0765d99..26a89f7 100644
--- a/components/result-search/columns/SiteColumn.tsx
+++ b/components/result-search/columns/SiteColumn.tsx
@@ -9,6 +9,9 @@ type SiteColumnProps = { result: Result };
  * @param props
  * @param props.result
  */
-const SiteColumn: FC<SiteColumnProps> = ({ result }) => <>{result.site.name}</>;
+const SiteColumn: FC<SiteColumnProps> = ({ result }) => (
+    // eslint-disable-next-line react/jsx-no-useless-fragment
+    <>{result.site.name}</>
+);
 
 export default SiteColumn;
diff --git a/components/result-search/columns/SiteFlavorColumn.tsx b/components/result-search/columns/SiteFlavorColumn.tsx
index adee949..c18295f 100644
--- a/components/result-search/columns/SiteFlavorColumn.tsx
+++ b/components/result-search/columns/SiteFlavorColumn.tsx
@@ -9,6 +9,9 @@ type SiteFlavorColumnProps = { result: Result };
  * @param props
  * @param props.result
  */
-const SiteFlavorColumn: FC<SiteFlavorColumnProps> = ({ result }) => <>{result.flavor.name}</>;
+const SiteFlavorColumn: FC<SiteFlavorColumnProps> = ({ result }) => (
+    // eslint-disable-next-line react/jsx-no-useless-fragment
+    <>{result.flavor.name}</>
+);
 
 export default SiteFlavorColumn;
diff --git a/components/result-search/diagrams/chartjs.tsx b/components/result-search/diagrams/chartjs.tsx
index 4dd0446..a57448a 100644
--- a/components/result-search/diagrams/chartjs.tsx
+++ b/components/result-search/diagrams/chartjs.tsx
@@ -95,7 +95,7 @@ const ChartJSDiagram: FC<ChartJSDiagramProps> = ({ results, suggestions }) => {
     // generate datasets
     const datasets: ChartDataset<'line'>[] = [];
     let colorIndex = 0;
-    dataPoints.forEach(function (dataMeta, siteId, _) {
+    dataPoints.forEach((dataMeta) => {
         datasets.push({
             label: dataMeta.site.name,
             backgroundColor: BACKGROUND_COLORS[colorIndex],
@@ -128,9 +128,10 @@ const ChartJSDiagram: FC<ChartJSDiagramProps> = ({ results, suggestions }) => {
             {rejected.length > 0 && (
                 <div className="my-1">
                     {datasets.length > 0 &&
-                        rejected.map((rejected) => (
-                            <Alert variant="warning" key={rejected.result.id}>
-                                Result {rejected.result.id} not displayed due to: {rejected.reason}
+                        rejected.map((rejectedResult) => (
+                            <Alert variant="warning" key={rejectedResult.result.id}>
+                                Result {rejectedResult.result.id} not displayed due to:{' '}
+                                {rejectedResult.reason}
                             </Alert>
                         ))}
                     {datasets.length === 0 && (
diff --git a/components/result-search/diagrams/echarts.tsx b/components/result-search/diagrams/echarts.tsx
index 32ead8b..0134e4f 100644
--- a/components/result-search/diagrams/echarts.tsx
+++ b/components/result-search/diagrams/echarts.tsx
@@ -1,5 +1,5 @@
-import { type ChangeEvent, type FC, useState } from 'react';
-import { Alert, Badge, Button, Col, Form, Row } from 'react-bootstrap';
+import { type FC, useState } from 'react';
+import { Alert, Button, Col, Form, Row } from 'react-bootstrap';
 import { type Suggestion } from 'components/result-search/jsonSchema';
 import {
     type DataPoint,
@@ -54,7 +54,7 @@ enum Regression {
 
 type RegressionSelectProps = {
     setRegressionMode: (mode: Regression) => void;
-    regressionMode: string;
+    regressionMode: Regression;
     disabled: boolean;
     polyRegressionOrder: number;
     setPolyRegressionOrder: (num: number) => void;
@@ -93,7 +93,6 @@ const RegressionSelect: FC<RegressionSelectProps> = ({
                 onChange={(e) => setPolyRegressionOrder(parseInt(e.target.value))}
                 size="sm"
                 value={polyRegressionOrder}
-                disabled={regressionMode !== Regression.Polynomial}
             >
                 {/* 1 - 10 */}
                 {[...Array(10).keys()].map((n) => (
@@ -332,9 +331,10 @@ const EChartsDiagram: FC<EChartsDiagramProps> = ({ results, suggestions }) => {
             {rejected.length > 0 && (
                 <div className="my-1">
                     {datasets.length > 0 &&
-                        rejected.map((rejected) => (
-                            <Alert variant="warning" key={rejected.result.id}>
-                                Result {rejected.result.id} not displayed due to: {rejected.reason}
+                        rejected.map((rejectedResult) => (
+                            <Alert variant="warning" key={rejectedResult.result.id}>
+                                Result {rejectedResult.result.id} not displayed due to:{' '}
+                                {rejectedResult.reason}
                             </Alert>
                         ))}
                     {datasets.length === 0 && (
diff --git a/components/searchSelectors/BenchmarkSearchSelect.tsx b/components/searchSelectors/BenchmarkSearchSelect.tsx
index 445905e..17f41fe 100644
--- a/components/searchSelectors/BenchmarkSearchSelect.tsx
+++ b/components/searchSelectors/BenchmarkSearchSelect.tsx
@@ -14,10 +14,10 @@ type BenchmarkSearchSelectProps = {
 const BenchmarkSearchSelect: FC<BenchmarkSearchSelectProps> = ({ benchmark, setBenchmark }) => {
     const api = useApi();
 
-    const displayRow = (benchmark: Benchmark) => (
+    const displayRow = (rowBenchmark: Benchmark) => (
         <>
-            {`${benchmark.docker_image}:${benchmark.docker_tag}`}
-            <p>{benchmark.description}</p>
+            {`${rowBenchmark.docker_image}:${rowBenchmark.docker_tag}`}
+            <p>{rowBenchmark.description}</p>
         </>
     );
 
diff --git a/components/searchSelectors/FlavorSearchSelect.tsx b/components/searchSelectors/FlavorSearchSelect.tsx
index 5d0a19f..4d32dcc 100644
--- a/components/searchSelectors/FlavorSearchSelect.tsx
+++ b/components/searchSelectors/FlavorSearchSelect.tsx
@@ -19,10 +19,10 @@ const FlavorSearchSelect: FC<FlavorSearchSelectProps> = ({
 }): ReactElement => {
     const api = useApi();
 
-    const displayRow = (flavor: Flavor) => (
+    const displayRow = (rowFlavor: Flavor) => (
         <>
-            {flavor.name}
-            <p>{flavor.description}</p>
+            {rowFlavor.name}
+            <p>{rowFlavor.description}</p>
         </>
     );
 
diff --git a/components/searchSelectors/SiteSearchPopover.tsx b/components/searchSelectors/SiteSearchPopover.tsx
index 94b8f23..11a34d2 100644
--- a/components/searchSelectors/SiteSearchPopover.tsx
+++ b/components/searchSelectors/SiteSearchPopover.tsx
@@ -14,10 +14,10 @@ type SiteSearchPopoverProps = {
 const SiteSearchPopover: FC<SiteSearchPopoverProps> = ({ site, setSite }): ReactElement => {
     const api = useApi();
 
-    const displayRow = (site: Site) => (
+    const displayRow = (rowSite: Site) => (
         <>
-            {site.name}
-            <p>{site.description}</p>
+            {rowSite.name}
+            <p>{rowSite.description}</p>
         </>
     );
 
diff --git a/components/site-editor/FlavorEditor.tsx b/components/site-editor/FlavorEditor.tsx
index 9be1371..77bc229 100644
--- a/components/site-editor/FlavorEditor.tsx
+++ b/components/site-editor/FlavorEditor.tsx
@@ -17,12 +17,12 @@ const FlavorEditor: FC<FlavorEditorProps> = ({ flavor, refetch }) => {
 
     const [editing, setEditing] = useState(false);
 
-    const updateEditing = (editing: boolean) => {
-        if (editing) {
+    const updateEditing = (newEditing: boolean) => {
+        if (newEditing) {
             setName(flavor.name);
             setDesc(flavor.description ? flavor.description : '');
         }
-        setEditing(editing);
+        setEditing(newEditing);
     };
 
     const { mutate } = useMutation((data: Flavor) => api.flavors.updateFlavor(flavor.id, data), {
diff --git a/components/submissionModals/FlavorSubmissionModal.tsx b/components/submissionModals/FlavorSubmissionModal.tsx
index c103bf3..40d5104 100644
--- a/components/submissionModals/FlavorSubmissionModal.tsx
+++ b/components/submissionModals/FlavorSubmissionModal.tsx
@@ -1,4 +1,4 @@
-import { type FC, ReactElement, useState } from 'react';
+import { type FC, useState } from 'react';
 import { Modal, Toast } from 'react-bootstrap';
 import FlavorSubmitForm from 'components/forms/FlavorSubmitForm';
 import { type Site } from '@eosc-perf/eosc-perf-client';
diff --git a/pages/search/result.tsx b/pages/search/result.tsx
index acee56f..0ed4a04 100644
--- a/pages/search/result.tsx
+++ b/pages/search/result.tsx
@@ -501,7 +501,6 @@ const ResultSearch: FC<PageProps> = (props: PageProps) => {
                         results={selectedResults}
                         benchmark={benchmark}
                         suggestions={suggestedFields}
-                        className="my-2"
                     />
                     <hr />
                     <Stack gap={2}>