@carbon/react 1.89.0 → 1.90.0

package/es/components/FluidTimePicker/FluidTimePicker.js

@@ -15,7 +15,8 @@ import '../FluidTextInput/FluidTextInput.Skeleton.js';
 import { usePrefix } from '../../internal/usePrefix.js';
 import { WarningFilled, WarningAltFilled } from '@carbon/icons-react';
 
-const FluidTimePicker = /*#__PURE__*/React.forwardRef(function FluidTimePicker({
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
+const FluidTimePicker = /*#__PURE__*/React.forwardRef(({
   className,
   children,
   disabled,
@@ -25,7 +26,7 @@ const FluidTimePicker = /*#__PURE__*/React.forwardRef(function FluidTimePicker({
   warnText,
   readOnly,
   ...other
-}, ref) {
+}, ref) => {
   const prefix = usePrefix();
   const classNames = cx(className, {
     [`${prefix}--time-picker--fluid`]: true,
@@ -45,12 +46,18 @@ const FluidTimePicker = /*#__PURE__*/React.forwardRef(function FluidTimePicker({
   const error = invalid || warn;
   const childrenWithProps = () => {
     if (disabled) {
-      return React.Children.toArray(children).map(child => /*#__PURE__*/React.cloneElement(child, {
+      return React.Children.toArray(children).map(child =>
+      /*#__PURE__*/
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any -- https://github.com/carbon-design-system/carbon/issues/20071
+      React.cloneElement(child, {
         disabled: true
       }));
     }
     if (readOnly) {
-      return React.Children.toArray(children).map(child => /*#__PURE__*/React.cloneElement(child, {
+      return React.Children.toArray(children).map(child =>
+      /*#__PURE__*/
+      // eslint-disable-next-line @typescript-eslint/no-explicit-any -- https://github.com/carbon-design-system/carbon/issues/20071
+      React.cloneElement(child, {
         readOnly: true
       }));
     }

package/es/components/FluidTimePickerSelect/FluidTimePickerSelect.js

@@ -11,11 +11,12 @@ import PropTypes from 'prop-types';
 import FluidSelect from '../FluidSelect/FluidSelect.js';
 import '../FluidSelect/FluidSelect.Skeleton.js';
 
-const FluidTimePickerSelect = /*#__PURE__*/React.forwardRef(function FluidTimePickerSelect({
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
+const FluidTimePickerSelect = /*#__PURE__*/React.forwardRef(({
   children,
   className,
   ...other
-}, ref) {
+}, ref) => {
   return /*#__PURE__*/React.createElement(FluidSelect, _extends({
     className: className,
     ref: ref

package/es/components/Grid/CSSGrid.js

@@ -12,6 +12,7 @@ import React from 'react';
 import { usePrefix } from '../../internal/usePrefix.js';
 import { useGridSettings, GridSettings } from './GridContext.js';
 
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
 const CSSGrid = /*#__PURE__*/React.forwardRef(({
   align,
   as,
@@ -94,6 +95,10 @@ CSSGrid.propTypes = {
    */
   narrow: PropTypes.bool
 };
+
+// eslint-disable-next-line @typescript-eslint/no-unused-vars -- https://github.com/carbon-design-system/carbon/issues/20071
+
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
 const Subgrid = /*#__PURE__*/React.forwardRef(({
   as,
   className: customClassName,

package/es/components/Grid/Column.js

@@ -13,6 +13,7 @@ import React from 'react';
 import { usePrefix } from '../../internal/usePrefix.js';
 import { useGridSettings } from './GridContext.js';
 
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
 const Column = /*#__PURE__*/React.forwardRef(({
   as,
   children,
@@ -114,6 +115,8 @@ Column.propTypes = {
    */
   xlg: spanPropType
 };
+
+// eslint-disable-next-line @typescript-eslint/no-explicit-any , react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
 const CSSGridColumn = /*#__PURE__*/React.forwardRef(({
   as: BaseComponent = 'div',
   children,

package/es/components/Grid/ColumnHang.js

@@ -24,6 +24,7 @@ function ColumnHang({
   const prefix = usePrefix();
   const className = cx(customClassName, `${prefix}--grid-column-hang`);
   // cast as any to let TypeScript allow passing in attributes to base component
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- https://github.com/carbon-design-system/carbon/issues/20071
   const BaseComponentAsAny = BaseComponent;
   return /*#__PURE__*/React.createElement(BaseComponentAsAny, _extends({}, rest, {
     className: className

package/es/components/Grid/FlexGrid.js

@@ -12,6 +12,7 @@ import React from 'react';
 import { usePrefix } from '../../internal/usePrefix.js';
 import { GridSettings } from './GridContext.js';
 
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
 const FlexGrid = /*#__PURE__*/React.forwardRef(({
   as,
   condensed = false,

package/es/components/Grid/Row.js

@@ -26,6 +26,7 @@ function Row({
     [`${prefix}--row--narrow`]: narrow
   });
   // TypeScript type validation reports conflicts on different instances of keyof JSX.IntrinsicElements
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- https://github.com/carbon-design-system/carbon/issues/20071
   const BaseComponentAsAny = BaseComponent;
   return /*#__PURE__*/React.createElement(BaseComponentAsAny, _extends({
     className: className

package/es/components/Heading/index.js

@@ -17,6 +17,7 @@ const Section = /*#__PURE__*/React.forwardRef(function Section({
 }, ref) {
   const parentLevel = React.useContext(HeadingContext);
   const level = levelOverride ?? parentLevel + 1;
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- https://github.com/carbon-design-system/carbon/issues/20071
   const BaseComponentAsAny = BaseComponent;
   return /*#__PURE__*/React.createElement(HeadingContext.Provider, {
     value: Math.min(level, 6)
@@ -43,7 +44,8 @@ Section.propTypes = {
    */
   level: PropTypes.number
 };
-const Heading = /*#__PURE__*/React.forwardRef(function Heading(props, ref) {
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
+const Heading = /*#__PURE__*/React.forwardRef((props, ref) => {
   const HeadingIntrinsic = `h${React.useContext(HeadingContext)}`;
   return /*#__PURE__*/React.createElement(HeadingIntrinsic, _extends({
     ref: ref

package/es/components/IconButton/index.js

@@ -19,7 +19,8 @@ import { BadgeIndicator } from '../BadgeIndicator/index.js';
 import { mapPopoverAlign } from '../../tools/mapPopoverAlign.js';
 
 const IconButtonKinds = ['primary', 'secondary', 'ghost', 'tertiary'];
-const IconButton = /*#__PURE__*/React.forwardRef(function IconButton({
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
+const IconButton = /*#__PURE__*/React.forwardRef(({
   align,
   autoAlign = false,
   badgeCount,
@@ -38,12 +39,13 @@ const IconButton = /*#__PURE__*/React.forwardRef(function IconButton({
   size,
   isSelected,
   ...rest
-}, ref) {
+}, ref) => {
   const prefix = usePrefix();
   const tooltipClasses = cx(wrapperClasses, `${prefix}--icon-tooltip`, {
     [`${prefix}--icon-tooltip--disabled`]: disabled
   });
   if (badgeCount && (kind !== 'ghost' || size !== 'lg')) {
+    // eslint-disable-next-line no-console -- https://github.com/carbon-design-system/carbon/issues/20071
     console.warn("The prop BadgeCount must be used with hasIconOnly=true, kind='ghost' and size='lg'");
   }
   const badgeId = useId('badge-indicator');

package/es/components/IconIndicator/index.js

@@ -26,13 +26,15 @@ const iconTypes = {
   unknown: UnknownFilled,
   informative: WarningSquareFilled
 };
-const IconIndicator = /*#__PURE__*/React.forwardRef(function IconIndicatorContent({
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
+const IconIndicator = /*#__PURE__*/React.forwardRef(({
   className: customClassName,
   kind,
   label,
   size = 16,
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars -- https://github.com/carbon-design-system/carbon/issues/20071
   ...rest
-}, ref) {
+}, ref) => {
   const prefix = usePrefix();
   const classNames = cx(`${prefix}--icon-indicator`, customClassName, {
     [`${prefix}--icon-indicator--20`]: size == 20

package/es/components/InlineCheckbox/InlineCheckbox.js

@@ -11,7 +11,8 @@ import { deprecate } from '../../prop-types/deprecate.js';
 import { usePrefix } from '../../internal/usePrefix.js';
 import { useMergedRefs } from '../../internal/useMergedRefs.js';
 
-const InlineCheckbox = /*#__PURE__*/React.forwardRef(function InlineCheckbox(props, forwardRef) {
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
+const InlineCheckbox = /*#__PURE__*/React.forwardRef((props, forwardRef) => {
   const {
     ['aria-label']: ariaLabel,
     ariaLabel: deprecatedAriaLabel,
@@ -60,7 +61,7 @@ const InlineCheckbox = /*#__PURE__*/React.forwardRef(function InlineCheckbox(pro
     className: `${prefix}--checkbox--inline`
   }, /*#__PURE__*/React.createElement("input", inputProps),
   /*#__PURE__*/
-  /* eslint-disable jsx-a11y/label-has-for,jsx-a11y/label-has-associated-control,jsx-a11y/click-events-have-key-events,jsx-a11y/no-noninteractive-element-interactions */
+  /* eslint-disable jsx-a11y/click-events-have-key-events,jsx-a11y/no-noninteractive-element-interactions */
   React.createElement("label", {
     htmlFor: id,
     className: `${prefix}--checkbox-label`,

package/es/components/Layout/index.js

@@ -22,14 +22,15 @@ const densities = ['condensed', 'normal'];
  * Size of components within this layout
  */
 
-const Layout = /*#__PURE__*/React.forwardRef(function Layout({
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
+const Layout = /*#__PURE__*/React.forwardRef(({
   as: BaseComponent = 'div',
   children,
   className,
   density,
   size,
   ...rest
-}, forwardRef) {
+}, forwardRef) => {
   const prefix = usePrefix();
   const classes = cx(className, `${prefix}--layout`, {
     [`${prefix}--layout--size-${size}`]: size && sizes.includes(size),
@@ -64,14 +65,15 @@ Layout.propTypes = {
    */
   size: PropTypes.oneOf(sizes)
 };
-const LayoutConstraint = /*#__PURE__*/React.forwardRef(function Layout({
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
+const LayoutConstraint = /*#__PURE__*/React.forwardRef(({
   as: BaseComponent = 'div',
   children,
   className,
   density,
   size,
   ...rest
-}, forwardRef) {
+}, forwardRef) => {
   const prefix = usePrefix();
   const classes = cx(className, Object.entries({
     size,

package/es/components/LayoutDirection/LayoutDirection.js

@@ -10,6 +10,8 @@ import PropTypes from 'prop-types';
 import React from 'react';
 import { LayoutDirectionContext } from './LayoutDirectionContext.js';
 
+// eslint-disable-next-line @typescript-eslint/no-unused-vars -- https://github.com/carbon-design-system/carbon/issues/20071
+
 function LayoutDirection({
   as: BaseComponent = 'div',
   children,

package/es/components/Link/Link.js

@@ -12,6 +12,7 @@ import React from 'react';
 import { usePrefix } from '../../internal/usePrefix.js';
 
 // First create the component with basic types
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
 const LinkBase = /*#__PURE__*/React.forwardRef(({
   as: BaseComponent,
   children,
@@ -47,6 +48,8 @@ const LinkBase = /*#__PURE__*/React.forwardRef(({
     linkProps.role = 'link';
     linkProps['aria-disabled'] = true;
   }
+
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- https://github.com/carbon-design-system/carbon/issues/20071
   const BaseComponentAsAny = BaseComponent ?? 'a';
   const handleOnClick = event => {
     if (disabled) {

package/es/components/ListBox/ListBoxMenuItem.js

@@ -22,7 +22,9 @@ import { useMergedRefs } from '../../internal/useMergedRefs.js';
  * @param deps - Dependencies to re-run the truncation check.
  * @returns An object containing the truncation state and the merged ref.
  */
-const useIsTruncated = (forwardedRef, deps = []) => {
+const useIsTruncated = (forwardedRef,
+// eslint-disable-next-line @typescript-eslint/no-explicit-any -- https://github.com/carbon-design-system/carbon/issues/20071
+deps = []) => {
   const localRef = useRef(null);
   const mergedRef = useMergedRefs([...(forwardedRef ? [forwardedRef] : []), localRef]);
   const [isTruncated, setIsTruncated] = useState(false);
@@ -35,6 +37,7 @@ const useIsTruncated = (forwardedRef, deps = []) => {
       } = element;
       setIsTruncated(offsetWidth < scrollWidth);
     }
+    // eslint-disable-next-line  react-hooks/exhaustive-deps -- https://github.com/carbon-design-system/carbon/issues/20071
   }, [localRef, ...deps]);
   return {
     isTruncated,

package/es/components/ListBox/next/ListBoxTrigger.js

@@ -27,11 +27,12 @@ const defaultTranslateWithId = id => defaultTranslations[id];
  * state of the menu for a given `ListBox`
  */
 
-const ListBoxTrigger = /*#__PURE__*/React.forwardRef(function ListBoxTrigger({
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
+const ListBoxTrigger = /*#__PURE__*/React.forwardRef(({
   isOpen,
   translateWithId: t = defaultTranslateWithId,
   ...rest
-}, ref) {
+}, ref) => {
   const prefix = usePrefix();
   const className = cx({
     [`${prefix}--list-box__menu-icon`]: true,

package/es/components/Menu/Menu.js

@@ -28,7 +28,6 @@ const Menu = /*#__PURE__*/forwardRef(function Menu({
   containerRef,
   label,
   menuAlignment,
-  mode,
   onClose,
   onOpen,
   open,
@@ -118,6 +117,8 @@ const Menu = /*#__PURE__*/forwardRef(function Menu({
     }
   }
   function focusItem(e) {
+    const validItems = focusableItems?.filter(item => item?.ref?.current);
+    if (!validItems?.length) return;
     const currentItem = focusableItems.findIndex(item => item.ref?.current?.contains(document.activeElement));
     let indexToFocus = currentItem;
 
@@ -134,14 +135,14 @@ const Menu = /*#__PURE__*/forwardRef(function Menu({
       }
     }
     if (indexToFocus < 0) {
-      indexToFocus = focusableItems.length - 1;
+      indexToFocus = validItems.length - 1;
     }
-    if (indexToFocus >= focusableItems.length) {
+    if (indexToFocus >= validItems.length) {
       indexToFocus = 0;
     }
     if (indexToFocus !== currentItem) {
-      const nodeToFocus = focusableItems[indexToFocus];
-      nodeToFocus.ref?.current?.focus();
+      const nodeToFocus = validItems[indexToFocus];
+      nodeToFocus?.ref?.current?.focus();
       e?.preventDefault();
     }
   }
@@ -304,7 +305,6 @@ Menu.propTypes = {
   /**
    * A label describing the Menu.
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   label: PropTypes.string,
   /**
    * Specify how the menu should align with the button element
@@ -340,17 +340,14 @@ Menu.propTypes = {
   /**
    * Specify a DOM node where the Menu should be rendered in. Defaults to document.body.
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   target: PropTypes.object,
   /**
    * Specify the x position of the Menu. Either pass a single number or an array with two numbers describing your activator's boundaries ([x1, x2])
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   x: PropTypes.oneOfType([PropTypes.number, PropTypes.arrayOf(PropTypes.number)]),
   /**
    * Specify the y position of the Menu. Either pass a single number or an array with two numbers describing your activator's boundaries ([y1, y2])
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   y: PropTypes.oneOfType([PropTypes.number, PropTypes.arrayOf(PropTypes.number)])
 };
 

package/es/components/Menu/MenuContext.js

@@ -44,6 +44,7 @@ function menuReducer(state, action) {
 const MenuContext = /*#__PURE__*/createContext({
   state: menuDefaultState,
   // 'dispatch' is populated by the root menu
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars -- https://github.com/carbon-design-system/carbon/issues/20071
   dispatch: _ => {}
 });
 

package/es/components/Menu/MenuItem.js

@@ -150,7 +150,7 @@ const MenuItem = /*#__PURE__*/forwardRef(function MenuItem({
   }, [direction]);
   useEffect(() => {
     if (IconElement && !context.state.hasIcons) {
-      // @ts-ignore - TODO: Should we be passing payload?
+      // @ts-expect-error - TODO: Should we be passing payload?
       context.dispatch({
         type: 'enableIcons'
       });
@@ -225,17 +225,14 @@ MenuItem.propTypes = {
   /**
    * Provide an optional function to be called when the MenuItem is clicked.
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   onClick: PropTypes.func,
   /**
    * A component used to render an icon.
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   renderIcon: PropTypes.oneOfType([PropTypes.func, PropTypes.object]),
   /**
    * Provide a shortcut for the action of this MenuItem. Note that the component will only render it as a hint but not actually register the shortcut.
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   shortcut: PropTypes.string
 };
 const MenuItemSelectable = /*#__PURE__*/forwardRef(function MenuItemSelectable({
@@ -253,12 +250,12 @@ const MenuItemSelectable = /*#__PURE__*/forwardRef(function MenuItemSelectable({
     onChange,
     defaultValue: defaultSelected ?? false
   });
-  function handleClick(e) {
+  function handleClick() {
     setChecked(!checked);
   }
   useEffect(() => {
     if (!context.state.hasSelectableItems) {
-      // @ts-ignore - TODO: Should we be passing payload?
+      // @ts-expect-error - TODO: Should we be passing payload?
       context.dispatch({
         type: 'enableSelectableItems'
       });
@@ -282,7 +279,6 @@ MenuItemSelectable.propTypes = {
   /**
    * Specify whether the option should be selected by default.
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   defaultSelected: PropTypes.bool,
   /**
    * A required label titling this option.
@@ -291,12 +287,10 @@ MenuItemSelectable.propTypes = {
   /**
    * Provide an optional function to be called when the selection state changes.
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   onChange: PropTypes.func,
   /**
    * Pass a bool to props.selected to control the state of this option.
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   selected: PropTypes.bool
 };
 const MenuItemGroup = /*#__PURE__*/forwardRef(function MenuItemGroup({
@@ -348,12 +342,13 @@ const MenuItemRadioGroup = /*#__PURE__*/forwardRef(function MenuItemRadioGroup({
     onChange,
     defaultValue: defaultSelectedItem ?? {}
   });
+  //eslint-disable-next-line  @typescript-eslint/no-unused-vars -- https://github.com/carbon-design-system/carbon/issues/20071
   function handleClick(item, e) {
     setSelection(item);
   }
   useEffect(() => {
     if (!context.state.hasSelectableItems) {
-      // @ts-ignore - TODO: Should we be passing payload?
+      // @ts-expect-error - TODO: Should we be passing payload?
       context.dispatch({
         type: 'enableSelectableItems'
       });
@@ -389,12 +384,10 @@ MenuItemRadioGroup.propTypes = {
   /**
    * Provide a function to convert an item to the string that will be rendered. Defaults to item.toString().
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   itemToString: PropTypes.func,
   /**
    * Provide the options for this radio group. Can be of any type, as long as you provide an appropriate props.itemToString function.
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   items: PropTypes.array,
   /**
    * A required label titling this radio group.
@@ -403,7 +396,6 @@ MenuItemRadioGroup.propTypes = {
   /**
    * Provide an optional function to be called when the selection changes.
    */
-  // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   onChange: PropTypes.func,
   /**
    * Provide props.selectedItem to control the state of this radio group. Must match the type of props.items.

package/es/components/MenuButton/index.js

@@ -23,7 +23,8 @@ import mergeRefs from '../../tools/mergeRefs.js';
 
 const validButtonKinds = ['primary', 'tertiary', 'ghost'];
 const defaultButtonKind = 'primary';
-const MenuButton = /*#__PURE__*/forwardRef(function MenuButton({
+// eslint-disable-next-line react/display-name -- https://github.com/carbon-design-system/carbon/issues/20071
+const MenuButton = /*#__PURE__*/forwardRef(({
   children,
   className,
   disabled,
@@ -34,13 +35,14 @@ const MenuButton = /*#__PURE__*/forwardRef(function MenuButton({
   tabIndex = 0,
   menuTarget,
   ...rest
-}, forwardRef) {
+}, forwardRef) => {
   // feature flag utilized to separate out only the dynamic styles from @floating-ui
   // flag is turned on when collision detection (ie. flip, hide) logic is not desired
   const enableOnlyFloatingStyles = useFeatureFlag('enable-v12-dynamic-floating-styles');
   const id = useId('MenuButton');
   const prefix = usePrefix();
   const triggerRef = useRef(null);
+  // eslint-disable-next-line @typescript-eslint/no-explicit-any -- https://github.com/carbon-design-system/carbon/issues/20071
   let middlewares = [];
   if (!enableOnlyFloatingStyles) {
     middlewares = [flip({
@@ -159,6 +161,7 @@ MenuButton.propTypes = {
   /**
    * Specify the type of button to be used as the base for the trigger button.
    */
+  // eslint-disable-next-line @typescript-eslint/ban-ts-comment -- https://github.com/carbon-design-system/carbon/issues/20071
   // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   kind: PropTypes.oneOf(validButtonKinds),
   /**
@@ -172,11 +175,13 @@ MenuButton.propTypes = {
   /**
    * Specify the size of the button and menu.
    */
+  // eslint-disable-next-line @typescript-eslint/ban-ts-comment -- https://github.com/carbon-design-system/carbon/issues/20071
   // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   size: PropTypes.oneOf(['sm', 'md', 'lg']),
   /**
    * Specify the tabIndex of the button.
    */
+  // eslint-disable-next-line @typescript-eslint/ban-ts-comment -- https://github.com/carbon-design-system/carbon/issues/20071
   // @ts-ignore-next-line -- avoid spurious (?) TS2322 error
   tabIndex: PropTypes.number,
   /**

package/es/components/Modal/Modal.js

@@ -38,7 +38,7 @@ import { debounce } from '../../node_modules/es-toolkit/dist/compat/function/deb
 import { Text } from '../Text/Text.js';
 
 const ModalSizes = ['xs', 'sm', 'md', 'lg'];
-const invalidOutsideClickMessage = '`Modal`: `preventCloseOnClickOutside` should not be `false` when `passiveModal` is `false`. Non-passive `Modal`s should not be dismissible by clicking outside.';
+const invalidOutsideClickMessage = '`<Modal>` prop `preventCloseOnClickOutside` should not be `false` when ' + '`passiveModal` is `false`. Transactional, non-passive Modals should ' + 'not be dissmissable by clicking outside. ' + 'See: https://carbondesignsystem.com/components/modal/usage/#transactional-modal';
 const Modal = /*#__PURE__*/React.forwardRef(function Modal({
   'aria-label': ariaLabelProp,
   children,
@@ -64,7 +64,7 @@ const Modal = /*#__PURE__*/React.forwardRef(function Modal({
   size,
   hasScrollingContent = false,
   closeButtonLabel = 'Close',
-  preventCloseOnClickOutside = !passiveModal,
+  preventCloseOnClickOutside,
   isFullWidth,
   launcherButtonRef,
   loadingStatus = 'inactive',
@@ -96,9 +96,7 @@ const Modal = /*#__PURE__*/React.forwardRef(function Modal({
   const focusTrapWithoutSentinels = useFeatureFlag('enable-experimental-focus-wrap-without-sentinels');
   const enableDialogElement = useFeatureFlag('enable-dialog-element');
   process.env.NODE_ENV !== "production" ? warning(!(focusTrapWithoutSentinels && enableDialogElement), '`<Modal>` detected both `focusTrapWithoutSentinels` and ' + '`enableDialogElement` feature flags are enabled. The native dialog ' + 'element handles focus, so `enableDialogElement` must be off for ' + '`focusTrapWithoutSentinels` to have any effect.') : void 0;
-  if (!passiveModal && preventCloseOnClickOutside === false) {
-    console.error(invalidOutsideClickMessage);
-  }
+  process.env.NODE_ENV !== "production" ? warning(!(!passiveModal && preventCloseOnClickOutside === false), invalidOutsideClickMessage) : void 0;
   function isCloseButton(element) {
     return !onSecondarySubmit && element === secondaryButton.current || element.classList.contains(modalCloseButtonClass);
   }
@@ -125,7 +123,15 @@ const Modal = /*#__PURE__*/React.forwardRef(function Modal({
       target
     } = evt;
     evt.stopPropagation();
-    if (!preventCloseOnClickOutside && target instanceof Node && !elementOrParentIsFloatingMenu(target, selectorsFloatingMenus) && innerModal.current && !innerModal.current.contains(target)) {
+    const shouldCloseOnOutsideClick =
+    // Passive modals can close on clicks outside the modal when
+    // preventCloseOnClickOutside is undefined or explicitly set to false.
+    passiveModal && !preventCloseOnClickOutside ||
+    // Non-passive modals have to explicitly opt-in for close on outside
+    // behavior by explicitly setting preventCloseOnClickOutside to false,
+    // rather than just leaving it undefined.
+    !passiveModal && preventCloseOnClickOutside === false;
+    if (shouldCloseOnOutsideClick && target instanceof Node && !elementOrParentIsFloatingMenu(target, selectorsFloatingMenus) && innerModal.current && !innerModal.current.contains(target)) {
       onRequestClose(evt);
     }
   }
@@ -167,6 +173,8 @@ const Modal = /*#__PURE__*/React.forwardRef(function Modal({
     }
     const lastContent = modalContent.children[modalContent.children.length - 1];
     const gradientSpacing = modalContent.scrollHeight - lastContent.offsetTop - lastContent.clientHeight;
+
+    // eslint-disable-next-line   prefer-const -- https://github.com/carbon-design-system/carbon/issues/20071
     for (let elem of modalContent.children) {
       if (elem.contains(currentActiveNode)) {
         const spaceBelow = modalContent.clientHeight - elem.offsetTop + modalContent.scrollTop - elem.clientHeight;
@@ -229,6 +237,7 @@ const Modal = /*#__PURE__*/React.forwardRef(function Modal({
     return () => {
       document.removeEventListener('keydown', handleEscapeKey, true);
     };
+    // eslint-disable-next-line  react-hooks/exhaustive-deps -- https://github.com/carbon-design-system/carbon/issues/20071
   }, [open]);
   useEffect(() => {
     return () => {

package/es/components/ModalWrapper/ModalWrapper.js

@@ -17,6 +17,7 @@ import { Escape } from '../../internal/keyboard/keys.js';
 import { match } from '../../internal/keyboard/match.js';
 
 let didWarnAboutDeprecation = false;
+// eslint-disable-next-line @typescript-eslint/no-unused-vars -- https://github.com/carbon-design-system/carbon/issues/20071
 process.env.NODE_ENV !== 'production';
 class ModalWrapper extends React.Component {
   constructor(...args) {
@@ -53,6 +54,7 @@ class ModalWrapper extends React.Component {
       handleSubmit?.(evt);
     });
   }
+  // eslint-disable-next-line @typescript-eslint/no-unused-vars -- https://github.com/carbon-design-system/carbon/issues/20071
   if(isDev) {
     process.env.NODE_ENV !== "production" ? warning(didWarnAboutDeprecation, '`<ModalWrapper>` has been deprecated in favor of `<ComposedModal/>` and will be removed in the next major version, `@carbon/react@v2.x`') : void 0;
     didWarnAboutDeprecation = true;