Implement minor UI adjustments to align with the updated layout

bluewave-labs · Aug 14, 2024 · 02708cd · 02708cd
1 parent b72fbc4
commit 02708cd
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 12 deletions.
diff --git a/Client/src/Pages/Monitors/Details/index.css b/Client/src/Pages/Monitors/Details/index.css
@@ -3,8 +3,7 @@
   color: var(--env-var-color-1);
   font-weight: 600;
 }
-.monitor-details h2.MuiTypography-root,
-.monitor-details h6.MuiTypography-root {
+.monitor-details h2.MuiTypography-root {
   font-size: var(--env-var-font-size-large);
 }
 .monitor-details h2.MuiTypography-root,
@@ -39,11 +38,18 @@
   min-width: 100px;
   border: solid 1px;
   border-radius: var(--env-var-radius-1);
-  padding: var(--env-var-spacing-1) 18px;
+  padding: var(--env-var-spacing-1) 16px;
   background-color: var(--env-var-color-8);
 }
-.monitor-details .stat-box h6.MuiTypography-root {
-  margin-bottom: var(--env-var-spacing-1);
+.monitor-details h6.MuiTypography-root {
+  font-size: var(--env-var-font-size-medium-plus);
+}
+.monitor-details h6.MuiTypography-root span.MuiTypography-root {
+  text-transform: none;
+  font-style: italic;
+  font-size: var(--env-var-font-size-medium);
+  color: var(--env-var-color-5);
+  opacity: 0.8;
 }
 .monitor-details button.MuiButtonBase-root,
 .monitor-details .stat-box {

diff --git a/Client/src/Pages/Monitors/Details/index.jsx b/Client/src/Pages/Monitors/Details/index.jsx
@@ -22,7 +22,11 @@ import MonitorDetails60MinChart from "../../../Components/Charts/MonitorDetails6
 const StatBox = ({ title, value }) => {
   return (
     <Box className="stat-box">
-      <Typography variant="h6" sx={{ fontWeight: 400 }}>
+      <Typography
+        variant="h6"
+        mb="5px"
+        sx={{ fontWeight: 500, textTransform: "uppercase" }}
+      >
         {title}
       </Typography>
       <Typography variant="h4">{value}</Typography>
@@ -31,7 +35,7 @@ const StatBox = ({ title, value }) => {
 };
 
 StatBox.propTypes = {
-  title: PropTypes.string,
+  title: PropTypes.oneOfType([PropTypes.string, PropTypes.element]),
   value: PropTypes.oneOfType([PropTypes.string, PropTypes.number]),
 };
 
@@ -176,7 +180,7 @@ const DetailsPage = () => {
               },
             }}
           />
-          <Stack gap={theme.gap.large}>
+          <Stack gap={theme.gap.xl}>
             <Stack
               direction="row"
               gap={theme.gap.small}
@@ -244,7 +248,7 @@ const DetailsPage = () => {
               />
               <StatBox
                 title="Last check"
-                value={`${formatDuration(monitor?.lastChecked)} ago`}
+                value={`${formatDurationRounded(monitor?.lastChecked)} ago`}
               />
               <StatBox title="Incidents" value={monitor?.incidents} />
               <StatBox title="Certificate Expiry" value={certificateExpiry} />
@@ -253,19 +257,32 @@ const DetailsPage = () => {
                 value={monitor?.latestResponseTime}
               />
               <StatBox
-                title="Average Repsonse Time (24 hours)"
+                title={
+                  <>
+                    Avg Response Time{" "}
+                    <Typography component="span">(24-hr)</Typography>
+                  </>
+                }
                 value={parseFloat(monitor?.avgResponseTime24hours)
                   .toFixed(2)
                   .replace(/\.?0+$/, "")}
               />
               <StatBox
-                title="Uptime (24 hours)"
+                title={
+                  <>
+                    Uptime <Typography component="span">(24-hr)</Typography>
+                  </>
+                }
                 value={`${parseFloat(monitor?.uptime24Hours)
                   .toFixed(2)
                   .replace(/\.?0+$/, "")}%`}
               />
               <StatBox
-                title="Uptime (30 days)"
+                title={
+                  <>
+                    Uptime <Typography component="span">(30-day)</Typography>
+                  </>
+                }
                 value={`${parseFloat(monitor?.uptime30Days)
                   .toFixed(2)
                   .replace(/\.?0+$/, "")}%`}