@okta/odyssey-react-mui 1.13.6 → 1.13.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@okta/odyssey-react-mui",
-  "version": "1.13.6",
+  "version": "1.13.8",
   "description": "React MUI components for Odyssey, Okta's design system",
   "author": "Okta, Inc.",
   "license": "Apache-2.0",
@@ -51,7 +51,7 @@
     "@mui/system": "^5.14.9",
     "@mui/utils": "^5.11.2",
     "@mui/x-date-pickers": "^5.0.15",
-    "@okta/odyssey-design-tokens": "1.13.6",
+    "@okta/odyssey-design-tokens": "1.13.8",
     "date-fns": "^2.30.0",
     "i18next": "^23.5.1",
     "material-react-table": "^2.0.2",
@@ -63,5 +63,5 @@
     "react": ">=17 <19",
     "react-dom": ">=17 <19"
   },
-  "gitHead": "cab3fb1def0c2b9cfbc1699b54230489f626a758"
+  "gitHead": "50e22cb3fb5c47d28ea5d3a0f8e4d5ed5c71e077"
 }

package/src/Accordion.tsx

@@ -35,10 +35,6 @@ export type AccordionProps = {
    * The label text for the AccordionSummary
    */
   label: string;
-  /**
-   * If true, the Accordion item will have a shadow.
-   */
-  hasShadow?: boolean;
   /**
    * Whether the item is expanded by default
    */
@@ -70,7 +66,6 @@ export type AccordionProps = {
 const Accordion = ({
   children,
   label,
-  hasShadow = true,
   id: idOverride,
   isDefaultExpanded,
   isDisabled,
@@ -88,7 +83,6 @@ const Accordion = ({
       disableGutters
       expanded={isExpanded}
       onChange={onChange}
-      className={hasShadow ? `hasShadow` : undefined}
     >
       <MuiAccordionSummary
         aria-controls={contentId}

package/src/Button.tsx

@@ -32,7 +32,6 @@ export const buttonTypeValues = ["button", "submit", "reset"] as const;
 export const buttonVariantValues = [
   "primary",
   "secondary",
-  "tertiary",
   "danger",
   "floating",
 ] as const;
@@ -112,7 +111,7 @@ export type ButtonProps = {
   /**
    * The variant of the Button
    */
-  variant: (typeof buttonVariantValues)[number];
+  variant: (typeof buttonVariantValues)[number] | "tertiary";
 } & (
   | {
       endIcon?: ReactElement;
@@ -150,9 +149,13 @@ const Button = ({
   tooltipText,
   translate,
   type = "button",
-  variant,
+  variant: variantProp,
 }: ButtonProps) => {
   const muiProps = useMuiProps();
+
+  // We're deprecating the "tertiary" variant, so map it to
+  // "secondary" in lieu of making a breaking change
+  const variant = variantProp === "tertiary" ? "secondary" : variantProp;
   const localButtonRef = useRef<HTMLButtonElement>(null);
 
   useImperativeHandle(

package/src/theme/components.tsx

@@ -62,15 +62,13 @@ export const components = ({
       styleOverrides: {
         root: () => ({
           backgroundColor: odysseyTokens.HueNeutralWhite,
-          borderWidth: 1,
-          borderInlineStyle: "solid",
-          borderColor: odysseyTokens.HueNeutral100,
+          border: 0,
+          borderBottomColor: odysseyTokens.BorderColorDisplay,
+          borderBottomStyle: "solid",
+          borderBottomWidth: odysseyTokens.BorderWidthMain,
+          borderRadius: 0,
           boxShadow: "none",
 
-          "&.hasShadow": {
-            boxShadow: odysseyTokens.DepthLow,
-          },
-
           "&.Mui-disabled": {
             backgroundColor: odysseyTokens.HueNeutralWhite,
             color: odysseyTokens.TypographyColorDisabled,
@@ -81,21 +79,18 @@ export const components = ({
             },
           },
 
-          "&::before": {
-            backgroundColor: odysseyTokens.BorderColorDisplay,
-            opacity: "1 !important",
-          },
-
           "&:first-of-type": {
-            borderTopLeftRadius: odysseyTokens.BorderRadiusMain,
-            borderTopRightRadius: odysseyTokens.BorderRadiusMain,
-            borderBlockStartStyle: "solid",
+            borderRadius: 0,
+            borderTopColor: odysseyTokens.BorderColorDisplay,
+            borderTopStyle: "solid",
+            borderTopWidth: odysseyTokens.BorderWidthMain,
           },
 
           "&:last-of-type": {
-            borderBottomLeftRadius: odysseyTokens.BorderRadiusMain,
-            borderBottomRightRadius: odysseyTokens.BorderRadiusMain,
-            borderBlockEndStyle: "solid",
+            borderRadius: 0,
+            borderBottomColor: odysseyTokens.BorderColorDisplay,
+            borderBottomStyle: "solid",
+            borderBottomWidth: odysseyTokens.BorderWidthMain,
           },
         }),
       },
@@ -106,26 +101,12 @@ export const components = ({
           paddingBlock: odysseyTokens.Spacing4,
           paddingInline: odysseyTokens.Spacing3,
 
-          ".MuiAccordion-root:first-of-type &": {
-            borderTopLeftRadius: odysseyTokens.BorderRadiusMain,
-            borderTopRightRadius: odysseyTokens.BorderRadiusMain,
-          },
-
-          ".MuiAccordion-root:last-of-type &": {
-            borderBottomLeftRadius: odysseyTokens.BorderRadiusMain,
-            borderBottomRightRadius: odysseyTokens.BorderRadiusMain,
-          },
-
-          ".MuiAccordion-root.Mui-expanded &": {
-            borderBottomLeftRadius: 0,
-            borderBottomRightRadius: 0,
-          },
-
-          "&:hover, &:focus": {
+          "&:hover": {
             backgroundColor: odysseyTokens.HueNeutral50,
           },
 
           "&:focus-visible": {
+            backgroundColor: odysseyTokens.HueNeutral50,
             outlineColor: odysseyTokens.PalettePrimaryMain,
             outlineWidth: 2,
             outlineStyle: "solid",
@@ -540,6 +521,7 @@ export const components = ({
           paddingBlock: odysseyTokens.Spacing3,
           paddingInline: odysseyTokens.Spacing4,
           display: "inline-flex",
+          height: odysseyTokens.Spacing7,
           position: "relative",
           marginBlock: "0",
           marginInline: "0",
@@ -552,7 +534,7 @@ export const components = ({
           borderRadius: odysseyTokens.BorderRadiusMain,
           borderColor: "transparent",
           fontSize: odysseyTokens.TypographySizeBody,
-          fontWeight: odysseyTokens.TypographyWeightBodyBold,
+          fontWeight: odysseyTokens.TypographyWeightHeading,
           fontFamily: odysseyTokens.TypographyFamilyButton,
           lineHeight: odysseyTokens.TypographyLineHeightUi,
           whiteSpace: "nowrap",
@@ -617,26 +599,6 @@ export const components = ({
             },
           }),
 
-          ...(ownerState.variant === "tertiary" && {
-            backgroundColor: odysseyTokens.HueNeutral100,
-            color: odysseyTokens.HueNeutral700,
-
-            "&:hover": {
-              backgroundColor: odysseyTokens.HueNeutral200,
-              color: odysseyTokens.HueNeutral800,
-            },
-
-            "&:active": {
-              backgroundColor: odysseyTokens.HueNeutral300,
-              color: odysseyTokens.HueNeutral800,
-            },
-
-            "&:disabled": {
-              backgroundColor: odysseyTokens.HueNeutral100,
-              color: odysseyTokens.TypographyColorDisabled,
-            },
-          }),
-
           ...(ownerState.variant === "danger" && {
             backgroundColor: odysseyTokens.PaletteDangerMain,
             color: odysseyTokens.HueNeutralWhite,
@@ -676,16 +638,17 @@ export const components = ({
             },
           }),
           ...(ownerState.size === "small" && {
+            height: odysseyTokens.Spacing6,
             paddingBlock: odysseyTokens.Spacing2,
             paddingInline: odysseyTokens.Spacing3,
             fontSize: odysseyTokens.TypographySizeBody,
           }),
           ...(ownerState.size === "large" && {
+            height: odysseyTokens.Spacing8,
             paddingBlock: odysseyTokens.Spacing4,
             paddingInline: odysseyTokens.Spacing4,
           }),
           ...(ownerState.fullWidth === true && {
-            display: "block",
             width: "100%",
             marginBlock: "0",
             marginInline: "0",
@@ -705,6 +668,10 @@ export const components = ({
             ...(ownerState.size === "small" && {
               padding: odysseyTokens.Spacing2,
             }),
+
+            ...(ownerState.size === "large" && {
+              padding: odysseyTokens.Spacing4,
+            }),
           }),
         }),