@simplybusiness/mobius 5.29.4 → 5.29.5

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # Changelog
 
+## 5.29.5
+
+### Patch Changes
+
+- 4ce860b: fix(useOnClickOutside): Allow null in RefObject type parameter
+
+  Updated useOnClickOutside hook to accept RefObject<HTMLElement | null> instead of RefObject<HTMLElement>, improving type safety and compatibility with React refs that can be null.
+
+- 845388a: fix: improve strong typing for React cloneElement usage
+
+  Adds proper TypeScript typing for all React.cloneElement calls throughout the codebase to improve type safety and remove unsafe type assertions where possible.
+
+- 39b6975: Updated all JSX type references to use React.JSX namespace instead of the global JSX namespace for better TypeScript compatibility with React 19.
+
 ## 5.29.4
 
 ### Patch Changes

package/dist/cjs/index.js

@@ -1995,9 +1995,13 @@ var Breadcrumbs = (0, import_react29.forwardRef)((props, ref) => {
     otherProps.className
   );
   return /* @__PURE__ */ (0, import_jsx_runtime16.jsx)("nav", { ref, ...otherProps, ...navProps, className: classes, children: /* @__PURE__ */ (0, import_jsx_runtime16.jsx)("ul", { children: childArray.map(
-    (child, i) => (0, import_react29.cloneElement)(child, {
-      isCurrent: i === childArray.length - 1
-    })
+    (child, i) => (0, import_react29.cloneElement)(
+      child,
+      {
+        key: i,
+        isCurrent: i === childArray.length - 1
+      }
+    )
   ) }) });
 });
 Breadcrumbs.displayName = "Breadcrumbs";
@@ -2029,7 +2033,11 @@ var BreadcrumbItem = (0, import_react30.forwardRef)((props, ref) => {
   );
   const child = typeof children === "string" ? /* @__PURE__ */ (0, import_jsx_runtime17.jsx)("span", { ...itemProps, children }) : import_react30.Children.only(children);
   return /* @__PURE__ */ (0, import_jsx_runtime17.jsxs)("li", { className: classes, children: [
-    (0, import_react30.cloneElement)(child, { ...child.props, ...itemProps, ref }),
+    (0, import_react30.cloneElement)(child, {
+      ...child.props,
+      ...itemProps,
+      ref
+    }),
     !props.isCurrent && separator
   ] });
 });
@@ -2310,16 +2318,21 @@ var CheckboxGroup = (0, import_react33.forwardRef)((props, ref) => {
           if ((0, import_react33.isValidElement)(child)) {
             const isLastItem = child === lastCheckbox;
             const isChildDisabled = isDisabled || lastItemDisables && lastCheckboxIsChecked && !isLastItem;
-            return (0, import_react33.cloneElement)(child, {
-              isDisabled: isChildDisabled,
-              isRequired,
-              isReadOnly,
-              isInvalid,
-              isLastItem,
-              selected: selected.includes(child.props.value),
-              onChange: handleChange,
-              "aria-describedby": describedBy
-            });
+            return (0, import_react33.cloneElement)(
+              child,
+              {
+                isDisabled: isChildDisabled,
+                isRequired,
+                isReadOnly,
+                isInvalid,
+                isLastItem,
+                selected: selected.includes(
+                  child.props.value
+                ),
+                onChange: handleChange,
+                "aria-describedby": describedBy
+              }
+            );
           }
           return child;
         }) }),
@@ -2693,10 +2706,16 @@ var Item = (0, import_react42.forwardRef)(
     );
     return /* @__PURE__ */ (0, import_jsx_runtime30.jsx)(Element, { ref, onClick, ...otherProps, children: import_react42.Children.map(children, (child) => {
       if ((0, import_react42.isValidElement)(child)) {
-        const childClasses = (0, import_dedupe23.default)(child.props.className, classes);
-        return (0, import_react42.cloneElement)(child, {
-          className: childClasses
-        });
+        const childClasses = (0, import_dedupe23.default)(
+          child.props.className,
+          classes
+        );
+        return (0, import_react42.cloneElement)(
+          child,
+          {
+            className: childClasses
+          }
+        );
       }
       return /* @__PURE__ */ (0, import_jsx_runtime30.jsx)("span", { className: classes, children: child });
     }) });
@@ -3348,7 +3367,10 @@ var Popover = (props) => {
   };
   const triggerComponent = (0, import_react61.cloneElement)(trigger, {
     ref: refs.setReference,
-    className: (0, import_classnames3.default)(trigger.props.className, "mobius-popover__toggle"),
+    className: (0, import_classnames3.default)(
+      trigger.props.className,
+      "mobius-popover__toggle"
+    ),
     onClick: toggleVisibility,
     ...getReferenceProps()
   });
@@ -3667,16 +3689,19 @@ var RadioGroup = (0, import_react64.forwardRef)((props, ref) => {
         label && /* @__PURE__ */ (0, import_jsx_runtime51.jsx)(Label, { htmlFor: name, id: labelId, className: labelClasses, children: label }),
         /* @__PURE__ */ (0, import_jsx_runtime51.jsx)("div", { className: radioWrapperClasses, children: import_react64.Children.map(children, (child) => {
           if ((0, import_react64.isValidElement)(child)) {
-            return (0, import_react64.cloneElement)(child, {
-              orientation,
-              groupDisabled: isDisabled,
-              name: nameAttribute,
-              selected,
-              setSelected,
-              isRequired,
-              "aria-describedby": describedBy,
-              onChange
-            });
+            return (0, import_react64.cloneElement)(
+              child,
+              {
+                orientation,
+                groupDisabled: isDisabled,
+                name: nameAttribute,
+                selected,
+                setSelected,
+                isRequired,
+                "aria-describedby": describedBy,
+                onChange
+              }
+            );
           }
           return child;
         }) }),