Skip to content

Commit 49e2ffc

Browse files
feat(discoveryDetailPanelTags): More general code cleanup
1 parent 706e4f4 commit 49e2ffc

File tree

3 files changed

+10
-18
lines changed

3 files changed

+10
-18
lines changed

packages/frontend/src/features/Discovery/DiscoveryDropdownTagViewer.tsx

Lines changed: 9 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -63,14 +63,10 @@ const MultiSelectContainer = ({ category }: MultiSelectContainerProps) => {
6363
const handleChange = (value: string[]) => {
6464
const previousValues = prevSelectionsRef.current;
6565
const removedTags = previousValues.filter((tag) => !value.includes(tag));
66-
console.log('value', value);
67-
6866
const addedTags = value.filter((tag) => !previousValues.includes(tag));
69-
7067
setSelectedTags((prev) => {
7168
const updated = { ...prev };
7269
removedTags.forEach((tag) => delete updated[tag]);
73-
7470
addedTags.forEach((tag) => {
7571
updated[tag] = true;
7672
});
@@ -117,16 +113,16 @@ const DiscoveryDropdownTagViewer = ({
117113
if (!tagCategoryData || tagCategoryData?.length === 0) return null;
118114

119115
return (
120-
<div
121-
className={`grid sm:grid-cols-1
116+
<div
117+
className={`grid sm:grid-cols-1
122118
${tagCategoryData.length > 1 && ' md:grid-cols-2 gap-4'}`}
123-
>
124-
{tagCategoryData.map((category, i) => (
125-
<div key={i}>
126-
<MultiSelectContainer category={category} />
127-
</div>
128-
))}
129-
</div>
119+
>
120+
{tagCategoryData.map((category, i) => (
121+
<div key={i}>
122+
<MultiSelectContainer category={category} />
123+
</div>
124+
))}
125+
</div>
130126
);
131127
};
132128

packages/frontend/src/features/Discovery/DiscoveryIndexPanel.tsx

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
import React, { ReactNode, useMemo, useRef, useState } from 'react';
2-
import { DiscoveryIndexConfig } from './types';
32
import DiscoveryTable from './DiscoveryTable/DiscoveryTable';
43
import { Button, Grid, Text } from '@mantine/core';
54
import AdvancedSearchPanel from './Search/AdvancedSearchPanel';
@@ -65,9 +64,7 @@ const DiscoveryIndexPanel = ({ indexSelector }: DiscoveryIndexPanelProps) => {
6564
);
6665
const [selections, setSelections] = useState<string[]>([]); // table selections
6766
const [sorting, setSorting] = useState<MRT_SortingState>([]);
68-
const [selectedTags, setSelectedTags] = useState<{ [key: string]: boolean }>(
69-
{},
70-
);
67+
7168
const [advancedSearchTerms, setAdvancedSearchTerms] =
7269
useState<AdvancedSearchTerms>({
7370
operation: SearchCombination.and,

packages/frontend/src/features/Discovery/DiscoveryTable/TableRenderers/RowDetailPanelTags.tsx

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@ interface RowDetailPanelProps {
99
}
1010

1111
const RowDetailPanelTags = ({ rowTags }: RowDetailPanelProps) => {
12-
rowTags as TagData[];
1312
const {
1413
discoveryConfig: config,
1514
selectedTags,

0 commit comments

Comments
 (0)