@moderneinc/design-system-components 7.1.0 → 7.1.1-next.9c4186

package/dist/index.esm.js

@@ -1542,37 +1542,35 @@ const panelSurfaceStyles = {
 const getDataGridHeaderStyles = () => {
     return {
         // Header cell container styles - from Figma: bg, border, padding
-        // Height is set via density-specific selectors in DataGrid.tsx
         [`& .${gridClasses.columnHeader}`]: {
             backgroundColor: semanticColors.color.surface.dataGrid.header,
             borderBottom: `1px solid ${semanticColors.color.border.primary}`,
-            paddingTop: `${spacing.spacing_1_1_2}px`,
-            paddingBottom: `${spacing.spacing_1_1_2}px`,
-            paddingLeft: `${spacing.spacing_2}px`,
-            paddingRight: `${spacing.spacing_2}px`,
+            paddingTop: `${spacing.spacing_1_1_4}px`,
+            paddingBottom: `${spacing.spacing_1_1_4}px`,
+            // WARNING: must track cell padding in DataGrid.tsx or labels sit left of
+            // the column they label.
+            paddingLeft: `${spacing.spacing_3}px`,
+            paddingRight: `${spacing.spacing_3}px`,
             boxSizing: 'border-box',
         },
         // Style the filler element to match header
-        // Height is set via density-specific selectors in DataGrid.tsx
         [`& .${gridClasses.filler}`]: {
             backgroundColor: semanticColors.color.surface.dataGrid.header,
             borderBottom: `1px solid ${semanticColors.color.border.primary}`,
             boxSizing: 'border-box',
         },
-        // Style the default MUI column header title text to match Figma
+        // NOTE: tertiary, not secondary — the mock's `--muted` is text.tertiary in
+        // both modes.
         [`& .${gridClasses.columnHeaderTitle}`]: {
-            fontFamily: `${typography.fontFamily.body}, sans-serif`,
-            fontSize: typography.fontSize.sm,
-            fontWeight: typography.fontWeight.semiBold, // Always semiBold (600)
-            lineHeight: 'normal',
-            color: semanticColors.color.text.secondary, // #6b7280 by default
+            ...semanticTypography.table.header,
+            color: semanticColors.color.text.tertiary,
             transition: 'color 0.2s',
             textOverflow: 'clip',
             whiteSpace: 'break-spaces',
         },
         // Change text color on hover
         [`& .${gridClasses.columnHeader}:hover .${gridClasses.columnHeaderTitle}`]: {
-            color: semanticColors.color.icon.default, // #1f2937 on hover
+            color: semanticColors.color.text.primary,
         },
         // Style the checkbox in header to match design system (16x16 from Figma)
         [`& .${gridClasses.columnHeaderCheckbox}`]: {
@@ -1587,9 +1585,8 @@ const getDataGridHeaderStyles = () => {
                 },
             },
         },
-        // MUI clips this container when header vertical padding leaves < 16px of content
-        // area (e.g. compact: 38px − 24px padding = 14px). overflow:visible lets the
-        // sort icon render at full 16px without being cropped.
+        // MUI clips this container when padding leaves < 16px of content area, which
+        // crops the sort icon. Guards callers that shrink height.table.header.
         [`& .${gridClasses.columnHeaderTitleContainer}`]: {
             overflow: 'visible',
             lineHeight: 1,
@@ -2388,7 +2385,9 @@ function ModDataGrid({ autoHeight = false, toolbar, slots, sx, density = 'standa
         },
         ...sx,
     };
-    return (jsx(DataGridPro, { apiRef: apiRef, columnHeaderHeight: density === 'compact' ? 38 : density === 'comfortable' ? 52 : 44, autoHeight: autoHeight, autoPageSize: !autoHeight, getRowHeight: () => 'auto', showToolbar: !!toolbar, slots: mergedSlots, slotProps: {
+    return (jsx(DataGridPro, { apiRef: apiRef, 
+        // One height at every density; `cell.minHeight` below still scales.
+        columnHeaderHeight: height.table.header, autoHeight: autoHeight, autoPageSize: !autoHeight, getRowHeight: () => 'auto', showToolbar: !!toolbar, slots: mergedSlots, slotProps: {
             basePopper: {
                 placement: 'bottom-start',
             },
@@ -2561,13 +2560,7 @@ ModDataGridFiltersButton.displayName = 'ModDataGridFiltersButton';
 /**
  * ModDataGridHeaderLabel - Header label for DataGrid columns with help icon support
  *
- * Design tokens used:
- * - Font: typography.fontWeight.semiBold (600)
- * - Font size: 14px (from Figma Table/header)
- * - Colors:
- *   - Default: semanticColors.typography.bodySecondary (#6b7280)
- *   - Hover: semanticColors.icons.default (#1f2937)
- *   - Icons: semanticColors.icons.utility (#9ca3af)
+ * Shares semanticTypography.table.header with the plain DataGrid column title.
  *
  * @example
  * // Basic usage (for custom header content)
@@ -2585,18 +2578,15 @@ const LabelContainer$2 = styled('div')({
     gap: `${spacing.spacing_1_2}px`,
 });
 const LabelText$1 = styled('p')({
-    fontFamily: `${typography.fontFamily.body}, sans-serif`,
-    fontSize: typography.fontSize.sm,
-    fontWeight: typography.fontWeight.semiBold, // Always semiBold (600)
-    lineHeight: 'normal',
-    color: semanticColors.color.text.secondary, // #6b7280 by default
+    ...semanticTypography.table.header,
+    color: semanticColors.color.text.tertiary,
     whiteSpace: 'nowrap',
     margin: 0,
     padding: 0,
     transition: 'color 0.2s',
     // Hover state - changes to primary text color
     '&:hover': {
-        color: semanticColors.color.icon.default, // #1f2937 on hover
+        color: semanticColors.color.text.primary,
     },
 });
 const IconWrapper = styled('div')({
@@ -5686,9 +5676,13 @@ const ModSkeleton = ({ colorTheme = 'light', position = 'start', ...props }) =>
 };
 ModSkeleton.displayName = 'ModSkeleton';
 
-const sizePadding = {
-    small: `${spacing.spacing_3_4}px ${spacing.spacing_1_1_2}px`,
-    medium: `${spacing.spacing_1_1_2}px ${spacing.spacing_2}px`,
+const sizePaddingY = {
+    small: spacing.spacing_3_4,
+    medium: spacing.spacing_1_1_2,
+};
+const sizePaddingX = {
+    small: spacing.spacing_1_1_2,
+    medium: spacing.spacing_2,
 };
 const sizeFontSize = {
     small: typography.fontSize.xs,
@@ -5699,14 +5693,12 @@ const StyledTable = styled(MuiTable, {
 })(({ theme, tableSize = 'small' }) => ({
     borderCollapse: 'separate',
     borderSpacing: 0,
+    // `size` scales the rows only; headers hold one treatment and one height.
     [`& .${tableCellClasses.head}`]: {
+        ...semanticTypography.table.header,
         backgroundColor: semanticColors.color.surface.dataGrid.header,
-        color: semanticColors.color.text.primary,
-        fontFamily: typography.fontFamily.body,
-        fontSize: theme.typography.pxToRem(sizeFontSize[tableSize]),
-        fontWeight: typography.fontWeight.semiBold,
-        lineHeight: 1.4,
-        padding: sizePadding[tableSize],
+        color: semanticColors.color.text.tertiary,
+        padding: `${spacing.spacing_1_1_4}px ${sizePaddingX[tableSize]}px`,
         borderBottom: `1px solid ${semanticColors.color.border.primary}`,
     },
     [`& .${tableCellClasses.body}`]: {
@@ -5715,7 +5707,7 @@ const StyledTable = styled(MuiTable, {
         fontSize: theme.typography.pxToRem(sizeFontSize[tableSize]),
         fontWeight: typography.fontWeight.regular,
         lineHeight: 1.4,
-        padding: sizePadding[tableSize],
+        padding: `${sizePaddingY[tableSize]}px ${sizePaddingX[tableSize]}px`,
         borderBottom: `1px solid ${semanticColors.color.border.primary}`,
     },
     // Drop the trailing row's bottom border so the container border (if