@sproutsocial/seeds-react-menu 1.5.1 → 1.5.2

package/.turbo/turbo-build.log

@@ -84,14 +84,14 @@ ESM Build start
          ╵       ~~~~~~~~~
 
 
-CJS dist/index.js     143.93 KB
-CJS dist/index.js.map 371.38 KB
-CJS ⚡️ Build success in 626ms
-ESM dist/esm/index.js     134.17 KB
-ESM dist/esm/index.js.map 370.11 KB
-ESM ⚡️ Build success in 628ms
+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
 DTS Build start
-DTS ⚡️ Build success in 15609ms
+DTS ⚡️ Build success in 12045ms
 DTS dist/index.d.ts  38.55 KB
 DTS dist/index.d.mts 38.55 KB
-Done in 18.46s.
+Done in 15.35s.

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # @sproutsocial/seeds-react-menu
 
+## 1.5.2
+
+### Patch Changes
+
+- a1ce7c5: [DS-3609] Fix the infinite rerender in the NestedMenu related to dynamic Popout positioning
+- Updated dependencies [a1ce7c5]
+  - @sproutsocial/seeds-react-popout@2.4.2
+
 ## 1.5.1
 
 ### Patch Changes

package/dist/esm/index.js

@@ -4000,7 +4000,7 @@ var AutocompleteSelect = (props) => {
 };
 
 // src/NestedMenu/NestedMenu.tsx
-import { useId, useEffect as useEffect4, useState } from "react";
+import { useId, useEffect as useEffect4, useState, useRef as useRef2 } from "react";
 import { AnimatePresence as AnimatePresence2, motion as motion3 } from "motion/react";
 import { useSelect as useSelect6 } from "downshift";
 
@@ -4302,7 +4302,7 @@ var NestedMenu = ({
   const [activeMenuContext, setActiveMenuContext] = useState(defaultMenuContext);
   const [currentMenuId, setCurrentMenuId] = useState(initialMenuId);
   const [isAdding, setIsAdding] = useState(false);
-  const [popoutUpdateRef, setPopoutUpdateRef] = useState();
+  const popoutUpdateRef = useRef2();
   const setSelectedMenuId = (id) => {
     setIsAdding(true);
     setSelectedMenuPath((currentPath) => [...currentPath, id]);
@@ -4311,7 +4311,6 @@ var NestedMenu = ({
     if (selectedMenuPath.length > 0) {
       const newMenuId = selectedMenuPath[selectedMenuPath.length - 1];
       setCurrentMenuId(newMenuId);
-      popoutUpdateRef?.();
     }
   }, [selectedMenuPath.join("-")]);
   const resetSelectedMenuPath = () => {
@@ -4433,6 +4432,9 @@ var NestedMenu = ({
     ),
     ...restMenuProps
   };
+  useEffect4(() => {
+    popoutUpdateRef.current?.();
+  }, [nestedMenuProps.children]);
   return /* @__PURE__ */ jsx34(
     NestedMenuProvider,
     {
@@ -4466,7 +4468,7 @@ var NestedMenu = ({
           menuToggleElement,
           width: width2,
           scheduleUpdateRef: (callback) => {
-            setPopoutUpdateRef(callback);
+            popoutUpdateRef.current = callback;
             externalPopoutUpdateRef?.(callback);
           },
           ...restPopoutProps