@primer/react 38.30.0 → 38.30.1-rc.0d136c1b7

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # @primer/react
 
+## 38.30.1
+
+### Patch Changes
+
+- [#8054](https://github.com/primer/react/pull/8054) [`1775f2e`](https://github.com/primer/react/commit/1775f2e8b968d59a1189456b14cff15e527937cb) Thanks [@joshblack](https://github.com/joshblack)! - AvatarStack: Improve rendering performance with React Compiler support
+
+- [#8053](https://github.com/primer/react/pull/8053) [`b2151ae`](https://github.com/primer/react/commit/b2151ae2d5719f09efb0b89731e52e4ca50e8bf1) Thanks [@joshblack](https://github.com/joshblack)! - Checkbox, CheckboxGroup, and RadioGroup: Improve rendering performance with React Compiler support
+
+- [#8052](https://github.com/primer/react/pull/8052) [`a1b24cf`](https://github.com/primer/react/commit/a1b24cf3411566641aeb846e62069dc26505da6b) Thanks [@joshblack](https://github.com/joshblack)! - Heading, Link, Pagehead, Portal, and SideNav: Improve rendering performance with React Compiler support
+
 ## 38.30.0
 
 ### Minor Changes

package/dist/AvatarStack/AvatarStack.js

@@ -1,10 +1,11 @@
+import { c } from 'react-compiler-runtime';
 import { clsx } from 'clsx';
 import React, { useState, useRef, useEffect } from 'react';
 import { DEFAULT_AVATAR_SIZE } from '../Avatar/Avatar.js';
 import { isResponsiveValue } from '../hooks/useResponsiveValue.js';
 import classes from './AvatarStack.module.css.js';
 import { hasInteractiveNodes } from '../internal/utils/hasInteractiveNodes.js';
-import { jsx, jsxs } from 'react/jsx-runtime';
+import { jsxs, jsx } from 'react/jsx-runtime';
 
 const transformChildren = (children, shape) => {
   return React.Children.map(children, child => {
@@ -16,136 +17,249 @@ const transformChildren = (children, shape) => {
     });
   });
 };
-const AvatarStackBody = ({
-  disableExpand,
-  hasInteractiveChildren,
-  stackContainer,
-  children
-}) => {
-  return /*#__PURE__*/jsx("div", {
-    "data-component": "AvatarStack.Body",
-    "data-disable-expand": disableExpand ? '' : undefined,
-    className: clsx({
-      'pc-AvatarStack--disableExpand': disableExpand
-    }, 'pc-AvatarStackBody', classes.AvatarStackBody),
-    tabIndex: !hasInteractiveChildren && !disableExpand ? 0 : undefined,
-    ref: stackContainer,
-    children: children
-  });
+const AvatarStackBody = t0 => {
+  const $ = c(8);
+  const {
+    disableExpand,
+    hasInteractiveChildren,
+    stackContainer,
+    children
+  } = t0;
+  const t1 = disableExpand ? "" : undefined;
+  let t2;
+  if ($[0] !== disableExpand) {
+    t2 = clsx({
+      "pc-AvatarStack--disableExpand": disableExpand
+    }, "pc-AvatarStackBody", classes.AvatarStackBody);
+    $[0] = disableExpand;
+    $[1] = t2;
+  } else {
+    t2 = $[1];
+  }
+  const t3 = !hasInteractiveChildren && !disableExpand ? 0 : undefined;
+  let t4;
+  if ($[2] !== children || $[3] !== stackContainer || $[4] !== t1 || $[5] !== t2 || $[6] !== t3) {
+    t4 = /*#__PURE__*/jsx("div", {
+      "data-component": "AvatarStack.Body",
+      "data-disable-expand": t1,
+      className: t2,
+      tabIndex: t3,
+      ref: stackContainer,
+      children: children
+    });
+    $[2] = children;
+    $[3] = stackContainer;
+    $[4] = t1;
+    $[5] = t2;
+    $[6] = t3;
+    $[7] = t4;
+  } else {
+    t4 = $[7];
+  }
+  return t4;
 };
-AvatarStackBody.displayName = "AvatarStackBody";
-const AvatarStack = ({
-  children,
-  variant = 'cascade',
-  shape = 'circle',
-  alignRight,
-  disableExpand,
-  size,
-  className,
-  style
-}) => {
+const AvatarStack = t0 => {
+  const $ = c(36);
+  const {
+    children,
+    variant: t1,
+    shape: t2,
+    alignRight,
+    disableExpand,
+    size,
+    className,
+    style
+  } = t0;
+  const variant = t1 === undefined ? "cascade" : t1;
+  const shape = t2 === undefined ? "circle" : t2;
   const [hasInteractiveChildren, setHasInteractiveChildren] = useState(false);
   const stackContainer = useRef(null);
   const count = React.Children.count(children);
-  const getAvatarChildSizes = () => {
-    const avatarSizeMap = {
-      narrow: [],
-      regular: [],
-      wide: []
-    };
-    return React.Children.toArray(children).reduce((acc, child) => {
-      // if child is not an Avatar, return the default avatar sizes from the accumulator
-      if (! /*#__PURE__*/React.isValidElement(child)) return acc;
-      for (const responsiveKey of Object.keys(avatarSizeMap)) {
-        // if the child has responsive `size` prop values, push the value to the appropriate viewport property in the avatarSizeMap
-        if (isResponsiveValue(child.props.size)) {
-          avatarSizeMap[responsiveKey].push(child.props.size[responsiveKey] || DEFAULT_AVATAR_SIZE);
+  let t3;
+  if ($[0] !== children) {
+    const getAvatarChildSizes = () => {
+      const avatarSizeMap = {
+        narrow: [],
+        regular: [],
+        wide: []
+      };
+      return React.Children.toArray(children).reduce((acc, child) => {
+        if (! /*#__PURE__*/React.isValidElement(child)) {
+          return acc;
         }
-        // otherwise, the size is a number (or undefined), so push the value to all viewport properties in the avatarSizeMap
-        else {
-          avatarSizeMap[responsiveKey].push(child.props.size || DEFAULT_AVATAR_SIZE);
+        for (const responsiveKey of Object.keys(avatarSizeMap)) {
+          if (isResponsiveValue(child.props.size)) {
+            avatarSizeMap[responsiveKey].push(child.props.size[responsiveKey] || DEFAULT_AVATAR_SIZE);
+          } else {
+            avatarSizeMap[responsiveKey].push(child.props.size || DEFAULT_AVATAR_SIZE);
+          }
+          acc[responsiveKey] = Math.min(...avatarSizeMap[responsiveKey]);
         }
-
-        // set the smallest size in each viewport property as the value for that viewport property in the accumulator
-        acc[responsiveKey] = Math.min(...avatarSizeMap[responsiveKey]);
-      }
-      return acc;
-    }, {
-      narrow: DEFAULT_AVATAR_SIZE,
-      regular: DEFAULT_AVATAR_SIZE,
-      wide: DEFAULT_AVATAR_SIZE
-    });
-  };
-  const childSizes = getAvatarChildSizes();
-  useEffect(() => {
-    if (stackContainer.current) {
-      const interactiveChildren = () => {
-        setHasInteractiveChildren(hasInteractiveNodes(stackContainer.current));
-      };
-      const observer = new MutationObserver(interactiveChildren);
-      observer.observe(stackContainer.current, {
-        childList: true
+        return acc;
+      }, {
+        narrow: DEFAULT_AVATAR_SIZE,
+        regular: DEFAULT_AVATAR_SIZE,
+        wide: DEFAULT_AVATAR_SIZE
       });
-
-      // Call on initial render, then call it again only if there's a mutation
-      // eslint-disable-next-line react-you-might-not-need-an-effect/no-initialize-state
-      interactiveChildren();
-      return () => {
-        observer.disconnect();
-      };
-    }
-  }, []);
-  const getResponsiveAvatarSizeStyles = () => {
-    // if there is no size set on the AvatarStack, use the `size` props of the Avatar children to set the `--avatar-stack-size` CSS variable
-    if (!size) {
-      return {
-        '--stackSize-narrow': `${childSizes.narrow}px`,
-        '--stackSize-regular': `${childSizes.regular}px`,
-        '--stackSize-wide': `${childSizes.wide}px`
-      };
-    }
-
-    // if the `size` prop is set and responsive, set the `--avatar-stack-size` CSS variable for each viewport
-    if (isResponsiveValue(size)) {
+    };
+    t3 = getAvatarChildSizes();
+    $[0] = children;
+    $[1] = t3;
+  } else {
+    t3 = $[1];
+  }
+  const childSizes = t3;
+  let t4;
+  let t5;
+  if ($[2] === Symbol.for("react.memo_cache_sentinel")) {
+    t4 = () => {
+      if (stackContainer.current) {
+        const interactiveChildren = () => {
+          setHasInteractiveChildren(hasInteractiveNodes(stackContainer.current));
+        };
+        const observer = new MutationObserver(interactiveChildren);
+        observer.observe(stackContainer.current, {
+          childList: true
+        });
+        interactiveChildren();
+        return () => {
+          observer.disconnect();
+        };
+      }
+    };
+    t5 = [];
+    $[2] = t4;
+    $[3] = t5;
+  } else {
+    t4 = $[2];
+    t5 = $[3];
+  }
+  useEffect(t4, t5);
+  let t6;
+  if ($[4] !== childSizes || $[5] !== size) {
+    t6 = () => {
+      if (!size) {
+        return {
+          "--stackSize-narrow": `${childSizes.narrow}px`,
+          "--stackSize-regular": `${childSizes.regular}px`,
+          "--stackSize-wide": `${childSizes.wide}px`
+        };
+      }
+      if (isResponsiveValue(size)) {
+        return {
+          "--stackSize-narrow": `${size.narrow || DEFAULT_AVATAR_SIZE}px`,
+          "--stackSize-regular": `${size.regular || DEFAULT_AVATAR_SIZE}px`,
+          "--stackSize-wide": `${size.wide || DEFAULT_AVATAR_SIZE}px`
+        };
+      }
       return {
-        '--stackSize-narrow': `${size.narrow || DEFAULT_AVATAR_SIZE}px`,
-        '--stackSize-regular': `${size.regular || DEFAULT_AVATAR_SIZE}px`,
-        '--stackSize-wide': `${size.wide || DEFAULT_AVATAR_SIZE}px`
+        "--avatar-stack-size": `${size}px`
       };
-    }
-
-    // if the `size` prop is set and not responsive, it is a number, so we can just set the `--avatar-stack-size` CSS variable to that number
-    return {
-      '--avatar-stack-size': `${size}px`
     };
-  };
-  return /*#__PURE__*/jsx("span", {
-    "data-component": "AvatarStack",
-    "data-variant": variant,
-    "data-shape": shape,
-    "data-avatar-count": count > 3 ? '3+' : count,
-    "data-align-right": alignRight ? '' : undefined,
-    "data-responsive": !size || isResponsiveValue(size) ? '' : undefined,
-    className: clsx({
-      'pc-AvatarStack--variant': variant,
-      'pc-AvatarStack--shape': shape,
-      'pc-AvatarStack--two': count === 2,
-      'pc-AvatarStack--three': count === 3,
-      'pc-AvatarStack--three-plus': count > 3,
-      'pc-AvatarStack--right': alignRight
-    }, className, classes.AvatarStack),
-    style: {
-      ...getResponsiveAvatarSizeStyles(),
+    $[4] = childSizes;
+    $[5] = size;
+    $[6] = t6;
+  } else {
+    t6 = $[6];
+  }
+  const getResponsiveAvatarSizeStyles = t6;
+  const t7 = count > 3 ? "3+" : count;
+  const t8 = alignRight ? "" : undefined;
+  const t9 = !size || isResponsiveValue(size) ? "" : undefined;
+  const t10 = count === 2;
+  const t11 = count === 3;
+  const t12 = count > 3;
+  let t13;
+  if ($[7] !== alignRight || $[8] !== className || $[9] !== shape || $[10] !== t10 || $[11] !== t11 || $[12] !== t12 || $[13] !== variant) {
+    t13 = clsx({
+      "pc-AvatarStack--variant": variant,
+      "pc-AvatarStack--shape": shape,
+      "pc-AvatarStack--two": t10,
+      "pc-AvatarStack--three": t11,
+      "pc-AvatarStack--three-plus": t12,
+      "pc-AvatarStack--right": alignRight
+    }, className, classes.AvatarStack);
+    $[7] = alignRight;
+    $[8] = className;
+    $[9] = shape;
+    $[10] = t10;
+    $[11] = t11;
+    $[12] = t12;
+    $[13] = variant;
+    $[14] = t13;
+  } else {
+    t13 = $[14];
+  }
+  let t14;
+  if ($[15] !== getResponsiveAvatarSizeStyles) {
+    t14 = getResponsiveAvatarSizeStyles();
+    $[15] = getResponsiveAvatarSizeStyles;
+    $[16] = t14;
+  } else {
+    t14 = $[16];
+  }
+  let t15;
+  if ($[17] !== style || $[18] !== t14) {
+    t15 = {
+      ...t14,
       ...style
-    },
-    children: /*#__PURE__*/jsxs(AvatarStackBody, {
+    };
+    $[17] = style;
+    $[18] = t14;
+    $[19] = t15;
+  } else {
+    t15 = $[19];
+  }
+  let t16;
+  if ($[20] !== children || $[21] !== shape) {
+    t16 = transformChildren(children, shape);
+    $[20] = children;
+    $[21] = shape;
+    $[22] = t16;
+  } else {
+    t16 = $[22];
+  }
+  let t17;
+  if ($[23] !== disableExpand || $[24] !== hasInteractiveChildren || $[25] !== t16) {
+    t17 = /*#__PURE__*/jsxs(AvatarStackBody, {
       disableExpand: disableExpand,
       hasInteractiveChildren: hasInteractiveChildren,
       stackContainer: stackContainer,
-      children: [' ', transformChildren(children, shape)]
-    })
-  });
+      children: [" ", t16]
+    });
+    $[23] = disableExpand;
+    $[24] = hasInteractiveChildren;
+    $[25] = t16;
+    $[26] = t17;
+  } else {
+    t17 = $[26];
+  }
+  let t18;
+  if ($[27] !== shape || $[28] !== t13 || $[29] !== t15 || $[30] !== t17 || $[31] !== t7 || $[32] !== t8 || $[33] !== t9 || $[34] !== variant) {
+    t18 = /*#__PURE__*/jsx("span", {
+      "data-component": "AvatarStack",
+      "data-variant": variant,
+      "data-shape": shape,
+      "data-avatar-count": t7,
+      "data-align-right": t8,
+      "data-responsive": t9,
+      className: t13,
+      style: t15,
+      children: t17
+    });
+    $[27] = shape;
+    $[28] = t13;
+    $[29] = t15;
+    $[30] = t17;
+    $[31] = t7;
+    $[32] = t8;
+    $[33] = t9;
+    $[34] = variant;
+    $[35] = t18;
+  } else {
+    t18 = $[35];
+  }
+  return t18;
 };
-AvatarStack.displayName = "AvatarStack";
 
 export { AvatarStack as default };

package/dist/Heading/Heading.js

@@ -1,3 +1,4 @@
+import { c } from 'react-compiler-runtime';
 import { clsx } from 'clsx';
 import React, { forwardRef } from 'react';
 import { useDevOnlyEffect } from '../internal/hooks/useDevOnlyEffect.js';
@@ -5,27 +6,76 @@ import classes from './Heading.module.css.js';
 import { jsx } from 'react/jsx-runtime';
 import { useMergedRefs } from '../hooks/useMergedRefs.js';
 
-const Heading = /*#__PURE__*/forwardRef(({
-  as: Component = 'h2',
-  className,
-  variant,
-  ...props
-}, forwardedRef) => {
+const Heading = /*#__PURE__*/forwardRef((t0, forwardedRef) => {
+  const $ = c(15);
+  let className;
+  let props;
+  let t1;
+  let variant;
+  if ($[0] !== t0) {
+    ({
+      as: t1,
+      className,
+      variant,
+      ...props
+    } = t0);
+    $[0] = t0;
+    $[1] = className;
+    $[2] = props;
+    $[3] = t1;
+    $[4] = variant;
+  } else {
+    className = $[1];
+    props = $[2];
+    t1 = $[3];
+    variant = $[4];
+  }
+  const Component = t1 === undefined ? "h2" : t1;
   const innerRef = React.useRef(null);
   const mergedRef = useMergedRefs(forwardedRef, innerRef);
-  useDevOnlyEffect(() => {
-    if (innerRef.current && !(innerRef.current instanceof HTMLHeadingElement)) {
-      // eslint-disable-next-line no-console
-      console.warn('This Heading component should be an instanceof of h1-h6');
-    }
-  }, [innerRef]);
-  return /*#__PURE__*/jsx(Component, {
-    className: clsx(className, classes.Heading),
-    "data-variant": variant,
-    "data-component": "Heading",
-    ...props,
-    ref: mergedRef
-  });
+  let t2;
+  let t3;
+  if ($[5] === Symbol.for("react.memo_cache_sentinel")) {
+    t2 = () => {
+      if (innerRef.current && !(innerRef.current instanceof HTMLHeadingElement)) {
+        console.warn("This Heading component should be an instanceof of h1-h6");
+      }
+    };
+    t3 = [innerRef];
+    $[5] = t2;
+    $[6] = t3;
+  } else {
+    t2 = $[5];
+    t3 = $[6];
+  }
+  useDevOnlyEffect(t2, t3);
+  let t4;
+  if ($[7] !== className) {
+    t4 = clsx(className, classes.Heading);
+    $[7] = className;
+    $[8] = t4;
+  } else {
+    t4 = $[8];
+  }
+  let t5;
+  if ($[9] !== Component || $[10] !== mergedRef || $[11] !== props || $[12] !== t4 || $[13] !== variant) {
+    t5 = /*#__PURE__*/jsx(Component, {
+      className: t4,
+      "data-variant": variant,
+      "data-component": "Heading",
+      ...props,
+      ref: mergedRef
+    });
+    $[9] = Component;
+    $[10] = mergedRef;
+    $[11] = props;
+    $[12] = t4;
+    $[13] = variant;
+    $[14] = t5;
+  } else {
+    t5 = $[14];
+  }
+  return t5;
 });
 Heading.displayName = 'Heading';
 

package/dist/Link/Link.js

@@ -1,3 +1,4 @@
+import { c } from 'react-compiler-runtime';
 import { clsx } from 'clsx';
 import React from 'react';
 import { useDevOnlyEffect } from '../internal/hooks/useDevOnlyEffect.js';
@@ -7,34 +8,89 @@ import { jsx } from 'react/jsx-runtime';
 import { useMergedRefs } from '../hooks/useMergedRefs.js';
 
 const UnwrappedLink = (props, ref) => {
-  const {
-    as: Component = 'a',
-    className,
-    inline,
-    muted,
-    hoverColor,
-    ...restProps
-  } = props;
+  const $ = c(19);
+  let className;
+  let hoverColor;
+  let inline;
+  let muted;
+  let restProps;
+  let t0;
+  if ($[0] !== props) {
+    ({
+      as: t0,
+      className,
+      inline,
+      muted,
+      hoverColor,
+      ...restProps
+    } = props);
+    $[0] = props;
+    $[1] = className;
+    $[2] = hoverColor;
+    $[3] = inline;
+    $[4] = muted;
+    $[5] = restProps;
+    $[6] = t0;
+  } else {
+    className = $[1];
+    hoverColor = $[2];
+    inline = $[3];
+    muted = $[4];
+    restProps = $[5];
+    t0 = $[6];
+  }
+  const Component = t0 === undefined ? "a" : t0;
   const innerRef = React.useRef(null);
   const mergedRef = useMergedRefs(ref, innerRef);
-  useDevOnlyEffect(() => {
-    if (innerRef.current && !(innerRef.current instanceof HTMLButtonElement) && !(innerRef.current instanceof HTMLAnchorElement)) {
-      // eslint-disable-next-line no-console
-      console.error('Error: Found `Link` component that renders an inaccessible element', innerRef.current, 'Please ensure `Link` always renders as <a> or <button>');
-    }
-  }, [innerRef]);
-  return /*#__PURE__*/jsx(Component, {
-    className: clsx(className, classes.Link),
-    "data-component": "Link",
-    "data-muted": muted,
-    "data-inline": inline,
-    "data-hover-color": hoverColor,
-    ...restProps,
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    ref: mergedRef
-  });
+  let t1;
+  let t2;
+  if ($[7] === Symbol.for("react.memo_cache_sentinel")) {
+    t1 = () => {
+      if (innerRef.current && !(innerRef.current instanceof HTMLButtonElement) && !(innerRef.current instanceof HTMLAnchorElement)) {
+        console.error("Error: Found `Link` component that renders an inaccessible element", innerRef.current, "Please ensure `Link` always renders as <a> or <button>");
+      }
+    };
+    t2 = [innerRef];
+    $[7] = t1;
+    $[8] = t2;
+  } else {
+    t1 = $[7];
+    t2 = $[8];
+  }
+  useDevOnlyEffect(t1, t2);
+  let t3;
+  if ($[9] !== className) {
+    t3 = clsx(className, classes.Link);
+    $[9] = className;
+    $[10] = t3;
+  } else {
+    t3 = $[10];
+  }
+  const t4 = mergedRef;
+  let t5;
+  if ($[11] !== Component || $[12] !== hoverColor || $[13] !== inline || $[14] !== muted || $[15] !== restProps || $[16] !== t3 || $[17] !== t4) {
+    t5 = /*#__PURE__*/jsx(Component, {
+      className: t3,
+      "data-component": "Link",
+      "data-muted": muted,
+      "data-inline": inline,
+      "data-hover-color": hoverColor,
+      ...restProps,
+      ref: t4
+    });
+    $[11] = Component;
+    $[12] = hoverColor;
+    $[13] = inline;
+    $[14] = muted;
+    $[15] = restProps;
+    $[16] = t3;
+    $[17] = t4;
+    $[18] = t5;
+  } else {
+    t5 = $[18];
+  }
+  return t5;
 };
-UnwrappedLink.displayName = "UnwrappedLink";
 const LinkComponent = fixedForwardRef(UnwrappedLink);
 const Link = Object.assign(LinkComponent, {
   displayName: 'Link'

package/dist/Pagehead/Pagehead.js

@@ -1,18 +1,52 @@
+import { c } from 'react-compiler-runtime';
 import { clsx } from 'clsx';
 import classes from './Pagehead.module.css.js';
 import { jsx } from 'react/jsx-runtime';
 
-const Pagehead = ({
-  as: BaseComponent = 'div',
-  className,
-  ...rest
-}) => {
-  return /*#__PURE__*/jsx(BaseComponent, {
-    className: clsx(classes.Pagehead, className),
-    "data-component": "Pagehead",
-    ...rest
-  });
+const Pagehead = t0 => {
+  const $ = c(10);
+  let className;
+  let rest;
+  let t1;
+  if ($[0] !== t0) {
+    ({
+      as: t1,
+      className,
+      ...rest
+    } = t0);
+    $[0] = t0;
+    $[1] = className;
+    $[2] = rest;
+    $[3] = t1;
+  } else {
+    className = $[1];
+    rest = $[2];
+    t1 = $[3];
+  }
+  const BaseComponent = t1 === undefined ? "div" : t1;
+  let t2;
+  if ($[4] !== className) {
+    t2 = clsx(classes.Pagehead, className);
+    $[4] = className;
+    $[5] = t2;
+  } else {
+    t2 = $[5];
+  }
+  let t3;
+  if ($[6] !== BaseComponent || $[7] !== rest || $[8] !== t2) {
+    t3 = /*#__PURE__*/jsx(BaseComponent, {
+      className: t2,
+      "data-component": "Pagehead",
+      ...rest
+    });
+    $[6] = BaseComponent;
+    $[7] = rest;
+    $[8] = t2;
+    $[9] = t3;
+  } else {
+    t3 = $[9];
+  }
+  return t3;
 };
-Pagehead.displayName = "Pagehead";
 
 export { Pagehead as default };