From 0eb57e7774afc0ecad719b053c9e29dceab568c3 Mon Sep 17 00:00:00 2001
From: alvaromartmart <alvaro.mart.mart+dev@gmail.com>
Date: Tue, 25 Jun 2024 23:23:00 +0200
Subject: [PATCH] tweak: use Expandable for attachments

---
 .../trace-viewer/src/ui/attachmentsTab.tsx    | 19 ++++++++++++++-----
 1 file changed, 14 insertions(+), 5 deletions(-)

diff --git a/packages/trace-viewer/src/ui/attachmentsTab.tsx b/packages/trace-viewer/src/ui/attachmentsTab.tsx
index 6ba9e1303a..3f1908f3d3 100644
--- a/packages/trace-viewer/src/ui/attachmentsTab.tsx
+++ b/packages/trace-viewer/src/ui/attachmentsTab.tsx
@@ -22,6 +22,7 @@ import { PlaceholderPanel } from './placeholderPanel';
 import type { AfterActionTraceEventAttachment } from '@trace/trace';
 import { CodeMirrorWrapper } from '@web/components/codeMirrorWrapper';
 import { isTextualMimeType } from '@isomorphic/mimeType';
+import { Expandable } from '@web/components/expandable';
 
 
 type Attachment = AfterActionTraceEventAttachment & { traceUrl: string };
@@ -53,6 +54,18 @@ const TextAttachment: React.FunctionComponent<TextAttachmentProps> = ({ attachme
   </div>;
 };
 
+const ExpandableAttachment: React.FunctionComponent<{ attachment: Attachment }> = ({ attachment }) => {
+  const [expanded, setExpanded] = React.useState(false);
+  return <Expandable title={
+    <a href={attachmentURL(attachment) + '&download'}>{attachment.name}</a>
+  } expanded={expanded} setExpanded={exp => setExpanded(exp)}>
+    {isTextualMimeType(attachment.contentType) ?
+      <TextAttachment attachment={attachment} label={attachment.name} /> :
+      <div><i>no preview available</i></div>
+    }
+  </Expandable>;
+};
+
 export const AttachmentsTab: React.FunctionComponent<{
   model: MultiTraceModel | undefined,
 }> = ({ model }) => {
@@ -114,11 +127,7 @@ export const AttachmentsTab: React.FunctionComponent<{
       return <>
         { i > 0 && <hr /> }
         <div className='attachment-item' key={`attachment-${i}`}>
-          <a href={attachmentURL(a) + '&download'}>{a.name}</a>
-          { isTextualMimeType(a.contentType) ?
-            <TextAttachment attachment={a} label={a.name}/> :
-            <div><i>no preview available</i></div>
-          }
+          <ExpandableAttachment attachment={a} />
         </div>
       </>;
     })}