From b886e8eeec8a8b5a9781efa3bdebcd02aea5817e Mon Sep 17 00:00:00 2001
From: Gerard Clos <clos.gerard@gmail.com>
Date: Thu, 26 Sep 2024 18:45:27 +0200
Subject: [PATCH] minor: abstract copy button

---
 .../_components/WorkspaceApiKeys/index.tsx        | 15 +++++++++++----
 packages/web-ui/src/ds/atoms/CodeBlock/index.tsx  |  4 ++--
 .../CopyButton.tsx => CopyButton/index.tsx}       | 14 +++++++++-----
 packages/web-ui/src/ds/atoms/Table/index.tsx      |  6 ++++--
 packages/web-ui/src/ds/atoms/index.ts             |  1 +
 5 files changed, 27 insertions(+), 13 deletions(-)
 rename packages/web-ui/src/ds/atoms/{CodeBlock/CopyButton.tsx => CopyButton/index.tsx} (57%)

diff --git a/apps/web/src/app/(private)/settings/_components/WorkspaceApiKeys/index.tsx b/apps/web/src/app/(private)/settings/_components/WorkspaceApiKeys/index.tsx
index a6be9933b..b09dbc767 100644
--- a/apps/web/src/app/(private)/settings/_components/WorkspaceApiKeys/index.tsx
+++ b/apps/web/src/app/(private)/settings/_components/WorkspaceApiKeys/index.tsx
@@ -2,6 +2,7 @@
 
 import {
   Button,
+  CopyButton,
   Icon,
   Table,
   TableBody,
@@ -34,7 +35,7 @@ export default function WorkspaceApiKeys() {
             </TableHeader>
             <TableBody>
               {apiKeys.map((apiKey) => (
-                <TableRow key={apiKey.id} verticalPadding>
+                <TableRow key={apiKey.id} verticalPadding hoverable={false}>
                   <TableCell>
                     <Text.H4>{apiKey.name || 'Unnamed API Key'}</Text.H4>
                   </TableCell>
@@ -50,9 +51,15 @@ export default function WorkspaceApiKeys() {
                             })
                           }}
                         >
-                          <Text.H4 color='foregroundMuted'>
-                            {apiKey.token}
-                          </Text.H4>
+                          <div className='flex flex-row items-center gap-2'>
+                            <Text.H4 color='foregroundMuted'>
+                              {apiKey.token}
+                            </Text.H4>
+                            <CopyButton
+                              content={apiKey.token}
+                              color='foregroundMuted'
+                            />
+                          </div>
                         </Button>
                       }
                     >
diff --git a/packages/web-ui/src/ds/atoms/CodeBlock/index.tsx b/packages/web-ui/src/ds/atoms/CodeBlock/index.tsx
index 9eb04f014..ad3da083d 100644
--- a/packages/web-ui/src/ds/atoms/CodeBlock/index.tsx
+++ b/packages/web-ui/src/ds/atoms/CodeBlock/index.tsx
@@ -3,7 +3,7 @@ import React from 'react'
 import { Prism as SyntaxHighlighter } from 'react-syntax-highlighter'
 import { oneLight } from 'react-syntax-highlighter/dist/esm/styles/prism'
 
-import { CopyButton } from './CopyButton'
+import { CopyButton } from '../CopyButton'
 
 interface CodeBlockProps {
   language: string
@@ -14,7 +14,7 @@ export function CodeBlock({ language, children }: CodeBlockProps) {
   return (
     <div className='relative max-w-full'>
       <div className='absolute top-4 right-2'>
-        <CopyButton content={children} />
+        <CopyButton content={children} color='foregroundMuted' />
       </div>
       <SyntaxHighlighter
         language={language}
diff --git a/packages/web-ui/src/ds/atoms/CodeBlock/CopyButton.tsx b/packages/web-ui/src/ds/atoms/CopyButton/index.tsx
similarity index 57%
rename from packages/web-ui/src/ds/atoms/CodeBlock/CopyButton.tsx
rename to packages/web-ui/src/ds/atoms/CopyButton/index.tsx
index c1997eb90..6e48ae568 100644
--- a/packages/web-ui/src/ds/atoms/CodeBlock/CopyButton.tsx
+++ b/packages/web-ui/src/ds/atoms/CopyButton/index.tsx
@@ -3,14 +3,18 @@
 import React from 'react'
 
 import { Button } from '../Button'
-import { Icon } from '../Icons'
+import { Icon, IconProps } from '../Icons'
 import { useToast } from '../Toast/useToast'
 
-interface CopyButtonProps {
+type CopyButtonProps = Omit<IconProps, 'name'> & {
   content: string
 }
 
-export function CopyButton({ content }: CopyButtonProps) {
+export function CopyButton({
+  content,
+  color = 'foregroundMuted',
+  ...rest
+}: CopyButtonProps) {
   const { toast } = useToast()
 
   const handleCopy = () => {
@@ -23,8 +27,8 @@ export function CopyButton({ content }: CopyButtonProps) {
   }
 
   return (
-    <Button onClick={handleCopy} variant='ghost' size='small'>
-      <Icon name='clipboard' />
+    <Button onClick={handleCopy} variant='nope' size='small'>
+      <Icon name='clipboard' color={color} {...rest} />
     </Button>
   )
 }
diff --git a/packages/web-ui/src/ds/atoms/Table/index.tsx b/packages/web-ui/src/ds/atoms/Table/index.tsx
index d1727fea4..77152cc36 100644
--- a/packages/web-ui/src/ds/atoms/Table/index.tsx
+++ b/packages/web-ui/src/ds/atoms/Table/index.tsx
@@ -68,14 +68,16 @@ const TableRow = forwardRef<
   HTMLTableRowElement,
   HTMLAttributes<HTMLTableRowElement> & {
     verticalPadding?: boolean
+    hoverable?: boolean
   }
->(({ className, verticalPadding, ...props }, ref) => (
+>(({ className, verticalPadding, hoverable = true, ...props }, ref) => (
   <tr
     ref={ref}
     className={cn(
-      'border-b transition-colors hover:bg-muted/50 data-[state=selected]:bg-muted',
+      'border-b transition-colors data-[state=selected]:bg-muted',
       {
         '[&>td]:py-4': verticalPadding,
+        'hover:bg-muted/50': hoverable,
       },
       className,
     )}
diff --git a/packages/web-ui/src/ds/atoms/index.ts b/packages/web-ui/src/ds/atoms/index.ts
index 3a2178fac..635370bdc 100644
--- a/packages/web-ui/src/ds/atoms/index.ts
+++ b/packages/web-ui/src/ds/atoms/index.ts
@@ -31,3 +31,4 @@ export * from './DropzoneInput'
 export * from './Charts'
 export * from './CircularProgress'
 export * from './CodeBlock'
+export * from './CopyButton'