@primer/react 38.30.1-rc.b8979ee01 → 38.30.1-rc.bce2a87c7

package/CHANGELOG.md

@@ -4,6 +4,8 @@
 
 ### 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

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/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@primer/react",
   "type": "module",
-  "version": "38.30.1-rc.b8979ee01",
+  "version": "38.30.1-rc.bce2a87c7",
   "description": "An implementation of GitHub's Primer Design System using React",
   "main": "./dist/index.js",
   "module": "./dist/index.js",