Card elevation 8 -> 2 / Paper 1 -> 2

1 is default for MUI. To me, there is just not enough shadow on 1, to make the surface distinct from the background. Paper is basically the same thing, so bumping to be consistent with Cards.
SeedCompany · Jun 25, 2024 · 92bb9d9 · 92bb9d9
1 parent 8362b34
commit 92bb9d9
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/src/scenes/ProgressReports/EditForm/ProgressReportStepper.tsx b/src/scenes/ProgressReports/EditForm/ProgressReportStepper.tsx
@@ -67,12 +67,7 @@ export const ProgressReportStepper = () => {
     useProgressReportContext();
 
   return (
-    <Paper
-      elevation={4}
-      sx={{ p: 2 }}
-      component="nav"
-      aria-label="Quarterly Report Steps"
-    >
+    <Paper sx={{ p: 2 }} component="nav" aria-label="Quarterly Report Steps">
       <Typography component="h3" paragraph aria-hidden>
         Steps:
       </Typography>

diff --git a/src/scenes/ProgressReports/EditForm/Steps/VariantAccordion.tsx b/src/scenes/ProgressReports/EditForm/Steps/VariantAccordion.tsx
@@ -18,7 +18,7 @@ export const VariantAccordion = ({
   const [expanded, { toggle }] = useToggle(expandedInput ?? false);
 
   return (
-    <Accordion expanded={expanded} elevation={2} square>
+    <Accordion expanded={expanded} square>
       <AccordionSummary
         aria-controls={`${variant.key}-content`}
         expandIcon={<ExpandMore />}

diff --git a/src/theme/overrides.ts b/src/theme/overrides.ts
@@ -51,9 +51,14 @@ export const appComponents = ({
         size: 'small',
       },
     },
+    MuiPaper: {
+      defaultProps: {
+        elevation: 2,
+      },
+    },
     MuiCard: {
       defaultProps: {
-        elevation: 8,
+        elevation: 2,
       },
     },
     MuiCardActions: {