feat(floating-ui): add feature flag for dynamic floating-ui styles

packages/feature-flags/feature-flags.yml

@@ -45,3 +45,8 @@ feature-flags:
     description: >
       Enable the new focus wrap behavior that doesn't use sentinel nodes
     enabled: false
+  - name: enable-v12-dynamic-floating-styles
+    description: >
+      Enable dynamic setting of floating styles for components like Popover,
+      Tooltip, etc.
+    enabled: false

packages/react/src/components/ComboBox/ComboBox.tsx

@@ -47,6 +47,7 @@ import { FormContext } from '../FluidForm';
 import { useFloating, flip, autoUpdate } from '@floating-ui/react';
 import { hide } from '@floating-ui/dom';
 import { TranslateWithId } from '../../types/common';
+import { useFeatureFlag } from '../FeatureFlags';
 
 const {
   InputBlur,
@@ -388,20 +389,24 @@ const ComboBox = forwardRef(
       slug,
       ...rest
     } = props;
+
+    const enableFloatingStyles =
+      useFeatureFlag('enable-v12-dynamic-floating-styles') || autoAlign;
+
     const { refs, floatingStyles, middlewareData } = useFloating(
-      autoAlign
+      enableFloatingStyles
         ? {
             placement: direction,
             strategy: 'fixed',
-            middleware: [flip(), hide()],
+            middleware: autoAlign ? [flip(), hide()] : undefined,
             whileElementsMounted: autoUpdate,
           }
         : {}
     );
     const parentWidth = (refs?.reference?.current as HTMLElement)?.clientWidth;
 
     useEffect(() => {
-      if (autoAlign) {
+      if (enableFloatingStyles) {
         const updatedFloatingStyles = {
           ...floatingStyles,
           visibility: middlewareData.hide?.referenceHidden
@@ -417,7 +422,7 @@ const ComboBox = forwardRef(
           refs.floating.current.style.width = parentWidth + 'px';
         }
       }
-    }, [autoAlign, floatingStyles, refs.floating, parentWidth]);
+    }, [enableFloatingStyles, floatingStyles, refs.floating, parentWidth]);
     const prefix = usePrefix();
     const { isFluid } = useContext(FormContext);
     const textInput = useRef<HTMLInputElement>(null);
@@ -607,7 +612,7 @@ const ComboBox = forwardRef(
       [`${prefix}--list-box--up`]: direction === 'top',
       [`${prefix}--combo-box--warning`]: showWarning,
       [`${prefix}--combo-box--readonly`]: readOnly,
-      [`${prefix}--autoalign`]: autoAlign,
+      [`${prefix}--autoalign`]: enableFloatingStyles,
     });
 
     const titleClasses = cx(`${prefix}--label`, {
@@ -785,10 +790,10 @@ const ComboBox = forwardRef(
       () =>
         getMenuProps({
           'aria-label': deprecatedAriaLabel || ariaLabel,
-          ref: autoAlign ? refs.setFloating : null,
+          ref: enableFloatingStyles ? refs.setFloating : null,
         }),
       [
-        autoAlign,
+        enableFloatingStyles,
         deprecatedAriaLabel,
         ariaLabel,
         getMenuProps,
@@ -815,7 +820,7 @@ const ComboBox = forwardRef(
           light={light}
           size={size}
           warn={warn}
-          ref={autoAlign ? refs.setReference : null}
+          ref={enableFloatingStyles ? refs.setReference : null}
           warnText={warnText}
           warnTextId={warnTextId}>
           <div className={`${prefix}--list-box__field`}>

packages/react/src/components/ComboButton/index.tsx

@@ -22,6 +22,7 @@ import {
   autoUpdate,
 } from '@floating-ui/react';
 import { hide } from '@floating-ui/dom';
+import { useFeatureFlag } from '../FeatureFlags';
 import mergeRefs from '../../tools/mergeRefs';
 import { MenuAlignment } from '../MenuButton';
 import { TranslateWithId } from '../../types/common';
@@ -97,10 +98,20 @@ const ComboButton = React.forwardRef<HTMLDivElement, ComboButtonProps>(
     },
     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('combobutton');
     const prefix = usePrefix();
     const containerRef = useRef<HTMLDivElement>(null);
-    const middlewares = [flip({ crossAxis: false }), hide()];
+    let middlewares: any[] = [];
+
+    if (!enableOnlyFloatingStyles) {
+      middlewares = [flip({ crossAxis: false }), hide()];
+    }
 
     if (menuAlignment === 'bottom' || menuAlignment === 'top') {
       middlewares.push(

packages/react/src/components/Dropdown/Dropdown.tsx

@@ -48,6 +48,7 @@ import {
   size as floatingSize,
 } from '@floating-ui/react';
 import { hide } from '@floating-ui/dom';
+import { useFeatureFlag } from '../FeatureFlags';
 
 const { ItemMouseMove, MenuMouseLeave } =
   useSelect.stateChangeTypes as UseSelectInterface['stateChangeTypes'] & {
@@ -274,8 +275,12 @@ const Dropdown = React.forwardRef(
     }: DropdownProps<ItemType>,
     ref: ForwardedRef<HTMLButtonElement>
   ) => {
+    const enableFloatingStyles = useFeatureFlag(
+      'enable-v12-dynamic-floating-styles'
+    );
+
     const { refs, floatingStyles, middlewareData } = useFloating(
-      autoAlign
+      enableFloatingStyles || autoAlign
         ? {
             placement: direction,
 
@@ -294,16 +299,18 @@ const Dropdown = React.forwardRef(
                   });
                 },
               }),
-              flip(),
-              hide(),
+              autoAlign && flip(),
+              autoAlign && hide(),
             ],
             whileElementsMounted: autoUpdate,
           }
-        : {} // When autoAlign is turned off, floating-ui will not be used
+        : {}
+      // When autoAlign is turned off & the `enable-v12-dynamic-floating-styles` feature flag is not
+      // enabled, floating-ui will not be used
     );
 
     useEffect(() => {
-      if (autoAlign) {
+      if (enableFloatingStyles || autoAlign) {
         const updatedFloatingStyles = {
           ...floatingStyles,
           visibility: middlewareData.hide?.referenceHidden
@@ -503,7 +510,7 @@ const Dropdown = React.forwardRef(
     const menuProps = useMemo(
       () =>
         getMenuProps({
-          ref: autoAlign ? refs.setFloating : null,
+          ref: enableFloatingStyles || autoAlign ? refs.setFloating : null,
         }),
       [autoAlign, getMenuProps, refs.setFloating]
     );
@@ -534,7 +541,7 @@ const Dropdown = React.forwardRef(
           warnText={warnText}
           light={light}
           isOpen={isOpen}
-          ref={autoAlign ? refs.setReference : null}
+          ref={enableFloatingStyles || autoAlign ? refs.setReference : null}
           id={id}>
           {invalid && (
             <WarningFilled className={`${prefix}--list-box__invalid-icon`} />

packages/react/src/components/FeatureFlags/overview.mdx

@@ -44,15 +44,16 @@ For more details on this approach, see the
 [feature flag documentation](https://github.com/carbon-design-system/carbon/blob/main/docs/experimental-code.md)
 in the Carbon monorepo.
 
-| Flag                                               | Description                                                              | Default | Javascript flag | Sass flag |
-| -------------------------------------------------- | ------------------------------------------------------------------------ | ------- | --------------- | --------- |
-| `enable-experimental-tile-contrast`                | Enable the improved styling for tiles that provides better contrast      | `false` |                 | ✅        |
-| `enable-experimental-focus-wrap-without-sentinels` | Enable the new focus wrap behavior that doesn't use sentinel nodes       | `false` | ✅              |           |
-| `enable-treeview-controllable`                     | Enable the new TreeView controllable API                                 | `false` | ✅              |           |
-| `enable-v12-tile-default-icons`                    | Enable default icons for Tile components                                 | `false` | ✅              |           |
-| `enable-v12-overflowmenu`                          | Enable the use of the v12 OverflowMenu leveraging the Menu subcomponents | `false` | ✅              |           |
-| `enable-v12-tile-radio-icons`                      | Enable rendering of default icons in the tile components                 | `false` | ✅              | ✅        |
-| `enable-v12-structured-list-visible-icons`         | Enable icon components within StructuredList to always be visible        | `false` |                 | ✅        |
+| Flag                                               | Description                                                                          | Default | Javascript flag | Sass flag |
+| -------------------------------------------------- | ------------------------------------------------------------------------------------ | ------- | --------------- | --------- |
+| `enable-experimental-tile-contrast`                | Enable the improved styling for tiles that provides better contrast                  | `false` |                 | ✅        |
+| `enable-experimental-focus-wrap-without-sentinels` | Enable the new focus wrap behavior that doesn't use sentinel nodes                   | `false` | ✅              |           |
+| `enable-treeview-controllable`                     | Enable the new TreeView controllable API                                             | `false` | ✅              |           |
+| `enable-v12-tile-default-icons`                    | Enable default icons for Tile components                                             | `false` | ✅              |           |
+| `enable-v12-overflowmenu`                          | Enable the use of the v12 OverflowMenu leveraging the Menu subcomponents             | `false` | ✅              |           |
+| `enable-v12-tile-radio-icons`                      | Enable rendering of default icons in the tile components                             | `false` | ✅              | ✅        |
+| `enable-v12-structured-list-visible-icons`         | Enable icon components within StructuredList to always be visible                    | `false` |                 | ✅        |
+| `enable-v12-dynamic-floating-styles`               | Enable dynamic setting of floating styles for components like Popover, Tooltip, etc. | `false` | ✅              | ✅        |
 
 ## Turning on feature flags in Javascript/react
 
@@ -117,11 +118,14 @@ Feature flags can also be enabled via the provided `enable()` mixin
 
 ## FeatureFlags Prop Update
 
-The `FeatureFlags` component has been updated to improve compatibility. The `flags` object prop is now deprecated and is replaced with individual boolean props for each feature flag.
-
-The `flags` prop will be removed in a future release. Instead, use individual boolean props for each feature flag. 
-A `featureflag-deprecate-flags-prop` codemod has been provided to help deprecate the `flags` object prop and switch to individual boolean props.
+The `FeatureFlags` component has been updated to improve compatibility. The
+`flags` object prop is now deprecated and is replaced with individual boolean
+props for each feature flag.
 
+The `flags` prop will be removed in a future release. Instead, use individual
+boolean props for each feature flag. A `featureflag-deprecate-flags-prop`
+codemod has been provided to help deprecate the `flags` object prop and switch
+to individual boolean props.
 
 ```bash
 npx @carbon/upgrade migrate featureflag-deprecate-flags-prop --write
@@ -130,9 +134,9 @@ npx @carbon/upgrade migrate featureflag-deprecate-flags-prop --write
 ```jsx
 //Before migration
 
- <FeatureFlags   
+ <FeatureFlags
   flags={{
-    'enable-v12-tile-default-icons': true, 
+    'enable-v12-tile-default-icons': true,
   }}>
     <App />
   </FeatureFlags>
@@ -143,4 +147,4 @@ npx @carbon/upgrade migrate featureflag-deprecate-flags-prop --write
   <FeatureFlags enableV12TileDefaultIcons>
     <App />
   </FeatureFlags>
-```
+```

packages/react/src/components/MenuButton/index.tsx

@@ -28,6 +28,7 @@ import {
   size as floatingSize,
   autoUpdate,
 } from '@floating-ui/react';
+import { useFeatureFlag } from '../FeatureFlags';
 import mergeRefs from '../../tools/mergeRefs';
 
 const validButtonKinds = ['primary', 'tertiary', 'ghost'];
@@ -97,10 +98,20 @@ const MenuButton = forwardRef<HTMLDivElement, MenuButtonProps>(
     },
     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<HTMLDivElement>(null);
-    const middlewares = [flip({ crossAxis: false })];
+    let middlewares: any[] = [];
+
+    if (!enableOnlyFloatingStyles) {
+      middlewares = [flip({ crossAxis: false })];
+    }
 
     if (menuAlignment === 'bottom' || menuAlignment === 'top') {
       middlewares.push(
@@ -113,6 +124,7 @@ const MenuButton = forwardRef<HTMLDivElement, MenuButtonProps>(
         })
       );
     }
+
     const { refs, floatingStyles, placement, middlewareData } = useFloating({
       placement: menuAlignment,
 

packages/react/src/components/MultiSelect/MultiSelect.tsx

@@ -53,6 +53,7 @@ import {
   autoUpdate,
 } from '@floating-ui/react';
 import { hide } from '@floating-ui/dom';
+import { useFeatureFlag } from '../FeatureFlags';
 
 const {
   ItemClick,
@@ -362,8 +363,11 @@ const MultiSelect = React.forwardRef(
     const [topItems, setTopItems] = useState([]);
     const [itemsCleared, setItemsCleared] = useState(false);
 
+    const enableFloatingStyles =
+      useFeatureFlag('enable-v12-dynamic-floating-styles') || autoAlign;
+
     const { refs, floatingStyles, middlewareData } = useFloating(
-      autoAlign
+      enableFloatingStyles
         ? {
             placement: direction,
 
@@ -375,23 +379,23 @@ const MultiSelect = React.forwardRef(
 
             // Middleware order matters, arrow should be last
             middleware: [
-              flip({ crossAxis: false }),
+              autoAlign && flip({ crossAxis: false }),
               floatingSize({
                 apply({ rects, elements }) {
                   Object.assign(elements.floating.style, {
                     width: `${rects.reference.width}px`,
                   });
                 },
               }),
-              hide(),
+              autoAlign && hide(),
             ],
             whileElementsMounted: autoUpdate,
           }
         : {}
     );
 
     useLayoutEffect(() => {
-      if (autoAlign) {
+      if (enableFloatingStyles) {
         const updatedFloatingStyles = {
           ...floatingStyles,
           visibility: middlewareData.hide?.referenceHidden
@@ -404,7 +408,13 @@ const MultiSelect = React.forwardRef(
           }
         });
       }
-    }, [autoAlign, floatingStyles, refs.floating, middlewareData, open]);
+    }, [
+      enableFloatingStyles,
+      floatingStyles,
+      refs.floating,
+      middlewareData,
+      open,
+    ]);
 
     const {
       selectedItems: controlledSelectedItems,
@@ -551,7 +561,7 @@ const MultiSelect = React.forwardRef(
         selectedItems && selectedItems.length > 0,
       [`${prefix}--list-box--up`]: direction === 'top',
       [`${prefix}--multi-select--readonly`]: readOnly,
-      [`${prefix}--autoalign`]: autoAlign,
+      [`${prefix}--autoalign`]: enableFloatingStyles,
       [`${prefix}--multi-select--selectall`]: selectAll,
     });
 
@@ -689,9 +699,9 @@ const MultiSelect = React.forwardRef(
     const menuProps = useMemo(
       () =>
         getMenuProps({
-          ref: autoAlign ? refs.setFloating : null,
+          ref: enableFloatingStyles ? refs.setFloating : null,
         }),
-      [autoAlign, getMenuProps, refs.setFloating]
+      [enableFloatingStyles, getMenuProps, refs.setFloating]
     );
 
     return (
@@ -729,7 +739,7 @@ const MultiSelect = React.forwardRef(
           )}
           <div
             className={multiSelectFieldWrapperClasses}
-            ref={autoAlign ? refs.setReference : null}>
+            ref={enableFloatingStyles ? refs.setReference : null}>
             {selectedItems.length > 0 && (
               <ListBox.Selection
                 readOnly={readOnly}