From 3ddc5ae3ec32cbe86f79d778b75103d106aecba1 Mon Sep 17 00:00:00 2001
From: Max Frederiksen <maxfrederiksen@Maxs-MacBook-Air.local>
Date: Thu, 13 Feb 2025 14:40:32 +0100
Subject: [PATCH] Fix history page bug and clean up

---
 .../log/LogDetails/KeyValueTable.jsx          |   2 +-
 .../components/log/LogDetails/LogHeader.jsx   |   1 +
 .../log/EntryEditor/EntryEditor.jsx           |   1 +
 src/components/log/LogHistory/LogHistory.jsx  | 135 ++++++++++--------
 4 files changed, 75 insertions(+), 64 deletions(-)

diff --git a/src/beta/components/log/LogDetails/KeyValueTable.jsx b/src/beta/components/log/LogDetails/KeyValueTable.jsx
index b6a64974..3024770e 100644
--- a/src/beta/components/log/LogDetails/KeyValueTable.jsx
+++ b/src/beta/components/log/LogDetails/KeyValueTable.jsx
@@ -16,13 +16,13 @@ export const KeyValueTable = ({ data }) => (
         <Typography
           sx={{ gridColumn: "1" }}
           fontSize="0.875rem"
-          fontWeight="bold"
         >
           {name}
         </Typography>
         <Typography
           sx={{ gridColumn: "2" }}
           fontSize="0.875rem"
+          fontWeight="bold"
           ml={1}
         >
           {value}
diff --git a/src/beta/components/log/LogDetails/LogHeader.jsx b/src/beta/components/log/LogDetails/LogHeader.jsx
index fb8470de..9a07e2d2 100644
--- a/src/beta/components/log/LogDetails/LogHeader.jsx
+++ b/src/beta/components/log/LogDetails/LogHeader.jsx
@@ -27,6 +27,7 @@ const LogHeader = ({ log, className }) => (
             <FormattedDate
               date={log.createdDate}
               component="span"
+              sx={{ fontSize: ".875rem", fontWeight: "bold" }}
             />
           )
         }
diff --git a/src/components/log/EntryEditor/EntryEditor.jsx b/src/components/log/EntryEditor/EntryEditor.jsx
index de70ba8f..825c9588 100644
--- a/src/components/log/EntryEditor/EntryEditor.jsx
+++ b/src/components/log/EntryEditor/EntryEditor.jsx
@@ -106,6 +106,7 @@ export const EntryEditor = ({
           type="submit"
           variant="contained"
           disabled={submitDisabled}
+          sx={{ marginBottom: 4 }}
         >
           Submit
         </Button>
diff --git a/src/components/log/LogHistory/LogHistory.jsx b/src/components/log/LogHistory/LogHistory.jsx
index 1aed340a..8503810d 100644
--- a/src/components/log/LogHistory/LogHistory.jsx
+++ b/src/components/log/LogHistory/LogHistory.jsx
@@ -1,17 +1,50 @@
 import {
-  Accordion,
-  AccordionDetails,
-  AccordionSummary,
   Box,
   Button,
+  Divider,
   Link,
+  Paper,
   Stack,
   Typography
 } from "@mui/material";
-import ExpandMoreIcon from "@mui/icons-material/ExpandMore";
 import FileDownloadIcon from "@mui/icons-material/FileDownload";
-import LogEntrySingleView from "components/LogDetails/LogEntrySingleView";
 import { FormattedDate } from "components/shared/FormattedDate";
+import LogDetails from "src/beta/components/log/LogDetails/LogDetails";
+import { KeyValueTable } from "src/beta/components/log/LogDetails/KeyValueTable";
+
+const LogHistoryHeader = ({ log }) => (
+  <Box
+    px={3}
+    pt={2}
+    sx={{ "& p, time": { fontSize: "1rem" } }}
+  >
+    <KeyValueTable
+      data={[
+        {
+          name: "Author",
+          value: log.owner
+        },
+        {
+          name: log.modifyDate ? "Edited" : "Created",
+          value: (
+            <FormattedDate
+              date={log?.modifyDate || log?.createdDate}
+              component="span"
+              sx={{ fontSize: ".875rem", fontWeight: "bold" }}
+            />
+          )
+        }
+      ]}
+    />
+    <Divider
+      sx={{
+        marginTop: "15px",
+        borderWidth: "1.25px",
+        borderColor: "#F3F5F7"
+      }}
+    />
+  </Box>
+);
 
 const DownloadButton = ({ currentLog, logHistory }) => {
   const data = {
@@ -38,43 +71,11 @@ const DownloadButton = ({ currentLog, logHistory }) => {
 };
 
 const LogHistory = ({ currentLog, logHistory }) => {
-  const renderTitle = (log, title) => {
-    return (
-      <Typography
-        component="h3"
-        variant="h6"
-      >
-        {title ?? (
-          <>
-            {`${log.modifyDate ? "Edited" : "Created"} `}
-            <FormattedDate date={log.modifyDate ?? log.createdDate} />
-          </>
-        )}
-      </Typography>
-    );
-  };
-
-  const renderLog = (log, title) => {
-    const id = log.modifyDate ?? log.createdBy;
-
-    return (
-      <Accordion defaultExpanded>
-        <AccordionSummary
-          expandIcon={<ExpandMoreIcon />}
-          aria-controls={`${id}-content`}
-          id={`${id}-header`}
-        >
-          {renderTitle(log, title)}
-        </AccordionSummary>
-        <AccordionDetails>
-          <LogEntrySingleView currentLogEntry={log} />
-        </AccordionDetails>
-      </Accordion>
-    );
-  };
-
   return (
-    <Stack padding={1}>
+    <Stack
+      py={2}
+      px={4}
+    >
       <Stack
         flexDirection="row"
         justifyContent="space-between"
@@ -83,34 +84,42 @@ const LogHistory = ({ currentLog, logHistory }) => {
         <Typography
           component="h1"
           variant="h2"
+          mb={2}
         >
           Log History
         </Typography>
         <DownloadButton {...{ currentLog, logHistory }} />
       </Stack>
-      <Stack>
-        <Typography
-          component="h2"
-          variant="h4"
-        >
-          Current Entry
-        </Typography>
-        <Box padding={1}>
-          <LogEntrySingleView currentLogEntry={currentLog} />
-        </Box>
-        <Typography
-          component="h2"
-          variant="h4"
+      <Stack mb={4}>
+        <Paper
+          elevation={3}
+          sx={{ margin: "10px 0" }}
         >
-          Previous Edits
-        </Typography>
-        <Stack>
-          {logHistory?.hitCount === 0 ? (
-            <Typography fontStyle="italic">(No edit history)</Typography>
-          ) : (
-            logHistory?.logs?.map((log) => renderLog(log))
-          )}
-        </Stack>
+          <LogHistoryHeader log={currentLog} />
+          <LogDetails log={currentLog} />
+        </Paper>
+
+        {logHistory?.hitCount === 0 ? (
+          <Typography fontStyle="italic">(No edit history)</Typography>
+        ) : (
+          logHistory?.logs?.map((log) => {
+            const id = log.modifyDate ?? log.createdBy;
+
+            return (
+              <Paper
+                key={id}
+                elevation={3}
+                sx={{
+                  margin: "10px 0",
+                  "&:last-child": { marginBottom: "25px" }
+                }}
+              >
+                <LogHistoryHeader log={log} />
+                <LogDetails log={log} />
+              </Paper>
+            );
+          })
+        )}
       </Stack>
     </Stack>
   );