Skip to content

Commit 23c1780

Browse files
Bill LeoutsakosBill Leoutsakos
authored andcommitted
fix(knowledge): use standard credential copy icon
1 parent 20a3941 commit 23c1780

2 files changed

Lines changed: 5 additions & 2 deletions

File tree

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/base-tags-modal/base-tags-modal.test.tsx

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,7 @@ vi.mock('@sim/emcn', async () => {
4444
Trigger: Container,
4545
Content: Container,
4646
},
47+
Duplicate: (props: React.SVGProps<SVGSVGElement>) => <svg data-icon='duplicate' {...props} />,
4748
Trash: (props: React.SVGProps<SVGSVGElement>) => <svg data-icon='trash' {...props} />,
4849
useCopyToClipboard: () => {
4950
const [copied, setCopied] = useState(false)
@@ -125,6 +126,7 @@ describe('BaseTagsModal tag ID copy control', () => {
125126

126127
expect(copyButton).toBeTruthy()
127128
expect(deleteButton).toBeTruthy()
129+
expect(copyButton.querySelector('[data-icon="duplicate"]')).toBeTruthy()
128130
expect(
129131
copyButton.compareDocumentPosition(deleteButton) & Node.DOCUMENT_POSITION_FOLLOWING
130132
).toBeTruthy()

apps/sim/app/workspace/[workspaceId]/knowledge/[id]/components/base-tags-modal/base-tags-modal.tsx

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,13 +12,14 @@ import {
1212
ChipModalFooter,
1313
ChipModalHeader,
1414
type ComboboxOption,
15+
Duplicate,
1516
handleKeyboardActivation,
1617
Tooltip,
1718
Trash,
1819
useCopyToClipboard,
1920
} from '@sim/emcn'
2021
import { createLogger } from '@sim/logger'
21-
import { Check, Clipboard } from 'lucide-react'
22+
import { Check } from 'lucide-react'
2223
import type { TagUsageData } from '@/lib/api/contracts/knowledge'
2324
import { SUPPORTED_FIELD_TYPES, TAG_SLOT_CONFIG } from '@/lib/knowledge/constants'
2425
import { getDocumentIcon } from '@/app/workspace/[workspaceId]/knowledge/components'
@@ -317,7 +318,7 @@ export function BaseTagsModal({ open, onOpenChange, knowledgeBaseId }: BaseTagsM
317318
{copied && copiedTagId === tag.id ? (
318319
<Check className='size-3 text-[var(--text-success)]' />
319320
) : (
320-
<Clipboard className='size-3' />
321+
<Duplicate className='size-3' />
321322
)}
322323
</Button>
323324
</Tooltip.Trigger>

0 commit comments

Comments
 (0)