@jobber/components 9.5.3 → 9.6.0

package/dist/useLabel-cjs.js

@@ -1,6 +1,5 @@
 'use strict';
 
-var floatingUi_utils = require('./floating-ui.utils-cjs.js');
 var useScrollLock = require('./useScrollLock-cjs.js');
 var useCompositeListItem = require('./useCompositeListItem-cjs.js');
 
@@ -9,7 +8,7 @@ function useRegisteredLabelId(idProp, setLabelId) {
   useScrollLock.useIsoLayoutEffect(() => {
     setLabelId(id);
     return () => {
-      setLabelId(undefined);
+      setLabelId(currentId => currentId === id ? undefined : currentId);
     };
   }, [id, setLabelId]);
   return id;
@@ -42,7 +41,7 @@ function useLabel(params = {}) {
       return;
     }
     const controlElement = useScrollLock.ownerDocument(event.currentTarget).getElementById(resolvedControlId);
-    if (floatingUi_utils.isHTMLElement(controlElement)) {
+    if (useScrollLock.isHTMLElement(controlElement)) {
       focusElementWithVisible(controlElement);
     }
   }

package/dist/useLabel-es.js

@@ -1,5 +1,4 @@
-import { i as isHTMLElement } from './floating-ui.utils-es.js';
-import { u as useBaseUiId, a as useIsoLayoutEffect, b as useStableCallback, g as getTarget, o as ownerDocument } from './useScrollLock-es.js';
+import { u as useBaseUiId, c as useIsoLayoutEffect, d as useStableCallback, e as getTarget, o as ownerDocument, i as isHTMLElement } from './useScrollLock-es.js';
 import { u as useLabelableContext } from './useCompositeListItem-es.js';
 
 function useRegisteredLabelId(idProp, setLabelId) {
@@ -7,7 +6,7 @@ function useRegisteredLabelId(idProp, setLabelId) {
   useIsoLayoutEffect(() => {
     setLabelId(id);
     return () => {
-      setLabelId(undefined);
+      setLabelId(currentId => currentId === id ? undefined : currentId);
     };
   }, [id, setLabelId]);
   return id;

package/dist/useRenderElement-cjs.js

@@ -364,7 +364,9 @@ function update(forkRef, refs) {
                 if (typeof cleanupCallback === 'function') {
                   cleanupCallback();
                 } else {
-                  ref(null);
+                  // Legacy ref with no attach-time cleanup: detach by calling it with `null`.
+                  // It returns nothing; React 19 cleanups are handled in the branch above.
+                  void ref(null);
                 }
                 break;
               }
@@ -536,7 +538,8 @@ function useRenderElementProps(componentProps, params = {}) {
   /* eslint-disable react-hooks/rules-of-hooks */
   if (typeof document !== 'undefined') {
     if (!enabled) {
-      useMergedRefs(null, null);
+      // Called only to keep the hook order stable when disabled; the merged ref is unused.
+      void useMergedRefs(null, null);
     } else if (Array.isArray(ref)) {
       outProps.ref = useMergedRefsN([outProps.ref, getReactElementRef(renderProp), ...ref]);
     } else {
@@ -562,9 +565,9 @@ function resolveRenderFunctionProps(props) {
 }
 
 // The symbol React uses internally for lazy components
-// https://github.com/facebook/react/blob/a0566250b210499b4c5677f5ac2eedbd71d51a1b/packages/shared/ReactSymbols.js#L31
+// https://github.com/react/react/blob/a0566250b210499b4c5677f5ac2eedbd71d51a1b/packages/shared/ReactSymbols.js#L31
 //
-// TODO delete once https://github.com/facebook/react/issues/32392 is fixed
+// TODO delete once https://github.com/react/react/issues/32392 is fixed
 const REACT_LAZY_TYPE = Symbol.for('react.lazy');
 const COMPONENT_IDENTIFIER_PATTERN = /^[A-Z][A-Za-z0-9$]*$/;
 const LOWERCASE_CHARACTER_PATTERN = /[a-z]/;
@@ -579,10 +582,9 @@ function evaluateRenderProp(element, render, props, state) {
     const mergedProps = mergeProps(props, render.props);
     mergedProps.ref = props.ref;
     let newElement = render;
-
-    // Workaround for https://github.com/facebook/react/issues/32392
+    // Workaround for https://github.com/react/react/issues/32392
     // This works because the toArray() logic unwrap lazy element type in
-    // https://github.com/facebook/react/blob/a0566250b210499b4c5677f5ac2eedbd71d51a1b/packages/react/src/ReactChildren.js#L186
+    // https://github.com/react/react/blob/a0566250b210499b4c5677f5ac2eedbd71d51a1b/packages/react/src/ReactChildren.js#L186
     if (newElement?.$$typeof === REACT_LAZY_TYPE) {
       const children = React__namespace.Children.toArray(render);
       newElement = children[0];

package/dist/useRenderElement-es.js

@@ -344,7 +344,9 @@ function update(forkRef, refs) {
                 if (typeof cleanupCallback === 'function') {
                   cleanupCallback();
                 } else {
-                  ref(null);
+                  // Legacy ref with no attach-time cleanup: detach by calling it with `null`.
+                  // It returns nothing; React 19 cleanups are handled in the branch above.
+                  void ref(null);
                 }
                 break;
               }
@@ -516,7 +518,8 @@ function useRenderElementProps(componentProps, params = {}) {
   /* eslint-disable react-hooks/rules-of-hooks */
   if (typeof document !== 'undefined') {
     if (!enabled) {
-      useMergedRefs(null, null);
+      // Called only to keep the hook order stable when disabled; the merged ref is unused.
+      void useMergedRefs(null, null);
     } else if (Array.isArray(ref)) {
       outProps.ref = useMergedRefsN([outProps.ref, getReactElementRef(renderProp), ...ref]);
     } else {
@@ -542,9 +545,9 @@ function resolveRenderFunctionProps(props) {
 }
 
 // The symbol React uses internally for lazy components
-// https://github.com/facebook/react/blob/a0566250b210499b4c5677f5ac2eedbd71d51a1b/packages/shared/ReactSymbols.js#L31
+// https://github.com/react/react/blob/a0566250b210499b4c5677f5ac2eedbd71d51a1b/packages/shared/ReactSymbols.js#L31
 //
-// TODO delete once https://github.com/facebook/react/issues/32392 is fixed
+// TODO delete once https://github.com/react/react/issues/32392 is fixed
 const REACT_LAZY_TYPE = Symbol.for('react.lazy');
 const COMPONENT_IDENTIFIER_PATTERN = /^[A-Z][A-Za-z0-9$]*$/;
 const LOWERCASE_CHARACTER_PATTERN = /[a-z]/;
@@ -559,10 +562,9 @@ function evaluateRenderProp(element, render, props, state) {
     const mergedProps = mergeProps(props, render.props);
     mergedProps.ref = props.ref;
     let newElement = render;
-
-    // Workaround for https://github.com/facebook/react/issues/32392
+    // Workaround for https://github.com/react/react/issues/32392
     // This works because the toArray() logic unwrap lazy element type in
-    // https://github.com/facebook/react/blob/a0566250b210499b4c5677f5ac2eedbd71d51a1b/packages/react/src/ReactChildren.js#L186
+    // https://github.com/react/react/blob/a0566250b210499b4c5677f5ac2eedbd71d51a1b/packages/react/src/ReactChildren.js#L186
     if (newElement?.$$typeof === REACT_LAZY_TYPE) {
       const children = React.Children.toArray(render);
       newElement = children[0];