Austenem/CAT-913 remove file accordion (#3546)

* remove extra files accordion * add changelog * add accordion prop for publications
hubmapconsortium · Sep 23, 2024 · 3efd23b · 3efd23b
1 parent febf1b8
commit 3efd23b
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 9 deletions.
diff --git a/CHANGELOG-remove-file-accordion.md b/CHANGELOG-remove-file-accordion.md
@@ -0,0 +1 @@
+- Remove extra files accordion from processed datasets summary sections. 
diff --git a/context/app/static/js/components/detailPage/files/Files/Files.tsx b/context/app/static/js/components/detailPage/files/Files/Files.tsx
@@ -10,18 +10,25 @@ import { UnprocessedFile } from '../types';
 
 interface FilesProps {
   files: UnprocessedFile[];
+  includeAccordion?: boolean;
 }
 
-function Files({ files }: FilesProps) {
+function Files({ files, includeAccordion }: FilesProps) {
+  const fileContent = files.length > 0 && (
+    <Box mb={2}>
+      <FileBrowser files={files} />
+    </Box>
+  );
+
   return (
     <FilesContextProvider>
-      <CollapsibleDetailPageSection id="files" title="Files">
-        {files.length > 0 && (
-          <Box mb={2}>
-            <FileBrowser files={files} />
-          </Box>
-        )}
-      </CollapsibleDetailPageSection>
+      {includeAccordion ? (
+        <CollapsibleDetailPageSection id="files" title="Files">
+          {fileContent}
+        </CollapsibleDetailPageSection>
+      ) : (
+        fileContent
+      )}
     </FilesContextProvider>
   );
 }

diff --git a/context/app/static/js/pages/Publication/Publication.jsx b/context/app/static/js/pages/Publication/Publication.jsx
@@ -59,7 +59,7 @@ function Publication({ publication, vignette_json }) {
         {shouldDisplaySection.visualizations && (
           <PublicationsVisualizationSection vignette_json={vignette_json} uuid={uuid} />
         )}
-        {shouldDisplaySection.files && <Files files={files} uuid={uuid} hubmap_id={hubmap_id} />}
+        {shouldDisplaySection.files && <Files files={files} includeAccordion />}
         {shouldDisplaySection['bulk-data-transfer'] && <PublicationBulkDataTransfer uuid={uuid} label={hubmap_id} />}
         <ContributorsTable contributors={contributors} contacts={contacts} title="Authors" />
         {shouldDisplaySection.provenance && <ProvSection uuid={uuid} assayMetadata={publication} />}