@sproutsocial/seeds-react-menu 1.5.2 → 1.6.1

package/.turbo/turbo-build.log

@@ -8,90 +8,14 @@ CLI Target: es2022
 CLI Cleaning output folder
 CJS Build start
 ESM Build start
-[warn] ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
-
-    ../../seeds-design-tokens/seeds-motion/package.json:12:6:
-      12 │       "types": "./dist/seeds-motion.d.ts"
-         ╵       ~~~~~~~
-
-  The "import" condition comes earlier and will be used for all "import" statements:
-
-    ../../seeds-design-tokens/seeds-motion/package.json:10:6:
-      10 │       "import": "./dist/seeds-motion.esm.js",
-         ╵       ~~~~~~~~
-
-  The "require" condition comes earlier and will be used for all "require" calls:
-
-    ../../seeds-design-tokens/seeds-motion/package.json:11:6:
-      11 │       "require": "./dist/seeds-motion.js",
-         ╵       ~~~~~~~~~
-
-
-[warn] ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
-
-    ../../seeds-design-tokens/seeds-motion/package.json:17:6:
-      17 │       "types": "./dist/seeds-motion-unitless.d.ts"
-         ╵       ~~~~~~~
-
-  The "import" condition comes earlier and will be used for all "import" statements:
-
-    ../../seeds-design-tokens/seeds-motion/package.json:15:6:
-      15 │       "import": "./dist/seeds-motion-unitless.esm.js",
-         ╵       ~~~~~~~~
-
-  The "require" condition comes earlier and will be used for all "require" calls:
-
-    ../../seeds-design-tokens/seeds-motion/package.json:16:6:
-      16 │       "require": "./dist/seeds-motion-unitless.js",
-         ╵       ~~~~~~~~~
-
-
-[warn] ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
-
-    ../../seeds-design-tokens/seeds-motion/package.json:12:6:
-      12 │       "types": "./dist/seeds-motion.d.ts"
-         ╵       ~~~~~~~
-
-  The "import" condition comes earlier and will be used for all "import" statements:
-
-    ../../seeds-design-tokens/seeds-motion/package.json:10:6:
-      10 │       "import": "./dist/seeds-motion.esm.js",
-         ╵       ~~~~~~~~
-
-  The "require" condition comes earlier and will be used for all "require" calls:
-
-    ../../seeds-design-tokens/seeds-motion/package.json:11:6:
-      11 │       "require": "./dist/seeds-motion.js",
-         ╵       ~~~~~~~~~
-
-
-[warn] ▲ [WARNING] The condition "types" here will never be used as it comes after both "import" and "require" [package.json]
-
-    ../../seeds-design-tokens/seeds-motion/package.json:17:6:
-      17 │       "types": "./dist/seeds-motion-unitless.d.ts"
-         ╵       ~~~~~~~
-
-  The "import" condition comes earlier and will be used for all "import" statements:
-
-    ../../seeds-design-tokens/seeds-motion/package.json:15:6:
-      15 │       "import": "./dist/seeds-motion-unitless.esm.js",
-         ╵       ~~~~~~~~
-
-  The "require" condition comes earlier and will be used for all "require" calls:
-
-    ../../seeds-design-tokens/seeds-motion/package.json:16:6:
-      16 │       "require": "./dist/seeds-motion-unitless.js",
-         ╵       ~~~~~~~~~
-
-
-CJS dist/index.js     143.99 KB
-CJS dist/index.js.map 371.54 KB
-CJS ⚡️ Build success in 717ms
-ESM dist/esm/index.js     134.23 KB
-ESM dist/esm/index.js.map 370.30 KB
-ESM ⚡️ Build success in 716ms
+CJS dist/index.js     144.30 KB
+CJS dist/index.js.map 371.92 KB
+CJS ⚡️ Build success in 706ms
+ESM dist/esm/index.js     134.40 KB
+ESM dist/esm/index.js.map 370.64 KB
+ESM ⚡️ Build success in 709ms
 DTS Build start
-DTS ⚡️ Build success in 12045ms
-DTS dist/index.d.ts  38.55 KB
-DTS dist/index.d.mts 38.55 KB
-Done in 15.35s.
+DTS ⚡️ Build success in 15887ms
+DTS dist/index.d.ts  39.00 KB
+DTS dist/index.d.mts 39.00 KB
+Done in 19.66s.

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # @sproutsocial/seeds-react-menu
 
+## 1.6.1
+
+### Patch Changes
+
+- 70e4ee7: [DS-3753] Fix handling of conditional items in menu groups
+
+## 1.6.0
+
+### Minor Changes
+
+- 74bb102: Export the DefaultMenuEmptyState for external use
+
 ## 1.5.2
 
 ### Patch Changes

package/dist/esm/index.js

@@ -2139,10 +2139,11 @@ var walkAllChildren = (children, callback) => {
     if (element === null || element === void 0) return;
     callback(element, parents2);
     const newParents = [...parents2, element];
-    const children2 = element.props?.children;
-    React4.Children.toArray(children2).forEach((child) => {
-      walk(child, newParents);
-    });
+    if (React4.isValidElement(element) && element.props?.children) {
+      React4.Children.toArray(element.props.children).forEach((child) => {
+        walk(child, newParents);
+      });
+    }
   };
   const parents = [];
   React4.Children.toArray(children).forEach((child) => {
@@ -2186,21 +2187,20 @@ var addItemsFromGroup = ({
       });
     });
   } else if (menuGroup.props.children) {
-    let children = menuGroup.props.children;
-    if (!Array.isArray(children)) {
-      children = [children];
-    }
+    const children = React4.Children.toArray(menuGroup.props.children);
     children.forEach((menuItem) => {
-      addMenuItem({
-        itemProps: {
-          id: menuItem.props.id,
-          disabled: menuItem.props.disabled,
-          menuItemProps: menuItem.props,
-          menuGroupProps: menuGroup.props
-        },
-        allMenuItems,
-        menuItemsMap
-      });
+      if (React4.isValidElement(menuItem)) {
+        addMenuItem({
+          itemProps: {
+            id: menuItem.props.id,
+            disabled: menuItem.props.disabled,
+            menuItemProps: menuItem.props,
+            menuGroupProps: menuGroup.props
+          },
+          allMenuItems,
+          menuItemsMap
+        });
+      }
     });
   }
 };
@@ -2307,16 +2307,18 @@ import { useCallback } from "react";
 
 // src/MenuContent/DefaultMenuEmptyState.tsx
 import styled4 from "styled-components";
+import { Box as Box2 } from "@sproutsocial/seeds-react-box";
 import { jsx as jsx6 } from "react/jsx-runtime";
-var StyledEmptyState = styled4.div`
+var StyledEmptyState = styled4(Box2)`
   padding: ${({ theme: theme2 }) => theme2.space[350]};
   text-align: center;
   font-size: ${({ theme: theme2 }) => theme2.typography[200].fontSize};
 `;
 var DefaultMenuEmptyState = ({
-  children
+  children = "No items found",
+  ...props
 }) => {
-  return /* @__PURE__ */ jsx6(StyledEmptyState, { children: children || "No results found." });
+  return /* @__PURE__ */ jsx6(StyledEmptyState, { ...props, children });
 };
 
 // src/MenuContent/MenuContent.tsx
@@ -2381,8 +2383,8 @@ import "react";
 
 // src/MenuFooter/styles.tsx
 import styled5 from "styled-components";
-import { Box as Box2 } from "@sproutsocial/seeds-react-box";
-var StyledMenuFooterBox = styled5(Box2)`
+import { Box as Box3 } from "@sproutsocial/seeds-react-box";
+var StyledMenuFooterBox = styled5(Box3)`
   padding: ${({ theme: theme2 }) => theme2.space[350]} ${({ theme: theme2 }) => theme2.space[400]};
   border-top: 1px solid ${(p) => p.theme.colors.container.border.base};
 `;
@@ -3052,7 +3054,7 @@ var MenuSearchTokenInput = ({
 // src/SubmenuItem/SubmenuItem.tsx
 import React13 from "react";
 import { Icon as Icon2 } from "@sproutsocial/seeds-react-icon";
-import { Box as Box3 } from "@sproutsocial/seeds-react-box";
+import { Box as Box4 } from "@sproutsocial/seeds-react-box";
 import { mergeRefs as mergeRefs2 } from "@sproutsocial/seeds-react-utilities";
 import { jsx as jsx17, jsxs as jsxs4 } from "react/jsx-runtime";
 var SubmenuItemTrigger = ({
@@ -3080,7 +3082,7 @@ var SubmenuItemTrigger = ({
         { suppressRefError: true }
       ),
       id,
-      children: /* @__PURE__ */ jsxs4(Box3, { display: "flex", justifyContent: "space-between", children: [
+      children: /* @__PURE__ */ jsxs4(Box4, { display: "flex", justifyContent: "space-between", children: [
         children,
         /* @__PURE__ */ jsx17(MenuItemActionRight, { children: /* @__PURE__ */ jsx17(Icon2, { name: "chevron-right-outline" }) })
       ] })
@@ -4072,7 +4074,7 @@ import { mergeRefs as mergeRefs3 } from "@sproutsocial/seeds-react-utilities";
 
 // src/NestedMenu/NestedMenuItem.tsx
 import { Icon as Icon5 } from "@sproutsocial/seeds-react-icon";
-import { Box as Box4 } from "@sproutsocial/seeds-react-box";
+import { Box as Box5 } from "@sproutsocial/seeds-react-box";
 import { jsx as jsx29, jsxs as jsxs6 } from "react/jsx-runtime";
 var NestedMenuItem = ({
   children,
@@ -4089,7 +4091,7 @@ var NestedMenuItem = ({
       }
     }
   } : {};
-  return /* @__PURE__ */ jsx29(MenuItem, { ...onClickProps, ...rest, children: /* @__PURE__ */ jsxs6(Box4, { display: "flex", justifyContent: "space-between", children: [
+  return /* @__PURE__ */ jsx29(MenuItem, { ...onClickProps, ...rest, children: /* @__PURE__ */ jsxs6(Box5, { display: "flex", justifyContent: "space-between", children: [
     children,
     childMenuId && /* @__PURE__ */ jsx29(MenuItemActionRight, { children: /* @__PURE__ */ jsx29(Icon5, { name: "arrow-right-outline" }) })
   ] }) });
@@ -4484,6 +4486,7 @@ export {
   AutocompleteSelect,
   AutocompleteTokenInput,
   CustomPopoutContent,
+  DefaultMenuEmptyState,
   INPUT_TYPES,
   ListboxToggleButton,
   MenuContent,