@itwin/itwinui-react 3.17.2 → 3.17.4

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # Changelog
 
+## 3.17.4
+
+### Patch Changes
+
+- [#2473](https://github.com/iTwin/iTwinUI/pull/2473): Fixed scrolling on touch devices when using the `enableVirtualization` prop in `Tree` and `ComboBox`.
+- [#2466](https://github.com/iTwin/iTwinUI/pull/2466): Fix `Tab`'s stripe positioning when `Tabs.TabList` overflows.
+- [#2469](https://github.com/iTwin/iTwinUI/pull/2469): Fixed `Tabs` bug where orientation related styles of nested tabs were sometimes incorrect.
+
+## 3.17.3
+
+### Patch Changes
+
+- [#2452](https://github.com/iTwin/iTwinUI/pull/2452): Fixed bug where nested `MenuItem`s in `Tile.MoreOptions` were not closing when clicked.
+- [#2455](https://github.com/iTwin/iTwinUI/pull/2455): Submenus will now correctly portal to the same place as their parent menu (E.g. `DropdownMenu`, `SplitButton`)
+- [#2457](https://github.com/iTwin/iTwinUI/pull/2457): Fixed bug in `ExpandableBlock` where cursor was `pointer` even when it was disabled.
+
 ## 3.17.2
 
 ### Patch Changes

package/DEV-cjs/core/ComboBox/ComboBox.js

@@ -371,27 +371,23 @@ const ComboBox = _react.forwardRef((props, forwardedRef) => {
           disabled: inputProps?.disabled,
           ...rest,
         },
-        _react.createElement(
-          _react.Fragment,
-          null,
-          _react.createElement(_ComboBoxInput.ComboBoxInput, {
-            value: inputValue,
-            disabled: inputProps?.disabled,
-            ...inputProps,
-            onChange: handleOnInput,
-            selectTags: isMultipleEnabled(selectedIndexes, multiple)
-              ? selectedIndexes
-                  ?.map((index) => {
-                    let item = options[index];
-                    return _react.createElement(_SelectTag.SelectTag, {
-                      key: item.label,
-                      label: item.label,
-                    });
-                  })
-                  .filter(Boolean)
-              : void 0,
-          }),
-        ),
+        _react.createElement(_ComboBoxInput.ComboBoxInput, {
+          value: inputValue,
+          disabled: inputProps?.disabled,
+          ...inputProps,
+          onChange: handleOnInput,
+          selectTags: isMultipleEnabled(selectedIndexes, multiple)
+            ? selectedIndexes
+                ?.map((index) => {
+                  let item = options[index];
+                  return _react.createElement(_SelectTag.SelectTag, {
+                    key: item.label,
+                    label: item.label,
+                  });
+                })
+                .filter(Boolean)
+            : void 0,
+        }),
         _react.createElement(_ComboBoxEndIcon.ComboBoxEndIcon, {
           ...endIconProps,
           disabled: inputProps?.disabled,

package/DEV-cjs/core/ComboBox/ComboBoxMenu.js

@@ -65,8 +65,12 @@ const VirtualizedComboBoxMenu = (props) => {
     [filteredOptions, getMenuItem, children, virtualizer.measureElement],
   );
   return _react.createElement(
-    _react.Fragment,
-    null,
+    'div',
+    {
+      style: {
+        display: 'contents',
+      },
+    },
     _react.createElement(
       _index.ShadowRoot,
       {

package/DEV-cjs/core/DropdownMenu/DropdownMenu.js

@@ -2,11 +2,23 @@
 Object.defineProperty(exports, '__esModule', {
   value: true,
 });
-Object.defineProperty(exports, 'DropdownMenu', {
-  enumerable: true,
-  get: function () {
+function _export(target, all) {
+  for (var name in all)
+    Object.defineProperty(target, name, {
+      enumerable: true,
+      get: all[name],
+    });
+}
+_export(exports, {
+  DropdownMenu: function () {
     return DropdownMenu;
   },
+  DropdownMenuCloseOnClickContext: function () {
+    return DropdownMenuCloseOnClickContext;
+  },
+  DropdownMenuContext: function () {
+    return DropdownMenuContext;
+  },
 });
 const _interop_require_wildcard = require('@swc/helpers/_/_interop_require_wildcard');
 const _react = /*#__PURE__*/ _interop_require_wildcard._(require('react'));
@@ -42,34 +54,50 @@ const DropdownMenuContent = _react.forwardRef((props, forwardedRef) => {
     visibleProp,
     onVisibleChange,
   );
+  let close = _react.useCallback(() => {
+    setVisible(false);
+  }, [setVisible]);
   let menuContent = _react.useMemo(() => {
-    if ('function' == typeof menuItems)
-      return menuItems(() => setVisible(false));
+    if ('function' == typeof menuItems) return menuItems(close);
     return menuItems;
-  }, [menuItems, setVisible]);
+  }, [close, menuItems]);
+  let dropdownMenuContextValue = _react.useMemo(
+    () => ({
+      close,
+    }),
+    [close],
+  );
   return _react.createElement(
-    _Menu.Menu,
+    DropdownMenuContext.Provider,
     {
-      trigger: children,
-      onKeyDown: (0, _index.mergeEventHandlers)(props.onKeyDown, (e) => {
-        if (e.defaultPrevented) return;
-        if ('Tab' === e.key) setVisible(false);
-      }),
-      role: role,
-      ref: forwardedRef,
-      portal: portal,
-      popoverProps: _react.useMemo(
-        () => ({
-          placement,
-          matchWidth,
-          visible,
-          onVisibleChange: setVisible,
-          middleware,
-        }),
-        [matchWidth, middleware, placement, setVisible, visible],
-      ),
-      ...rest,
+      value: dropdownMenuContextValue,
     },
-    menuContent,
+    _react.createElement(
+      _Menu.Menu,
+      {
+        trigger: children,
+        onKeyDown: (0, _index.mergeEventHandlers)(props.onKeyDown, (e) => {
+          if (e.defaultPrevented) return;
+          if ('Tab' === e.key) setVisible(false);
+        }),
+        role: role,
+        ref: forwardedRef,
+        portal: portal,
+        popoverProps: _react.useMemo(
+          () => ({
+            placement,
+            matchWidth,
+            visible,
+            onVisibleChange: setVisible,
+            middleware,
+          }),
+          [matchWidth, middleware, placement, setVisible, visible],
+        ),
+        ...rest,
+      },
+      menuContent,
+    ),
   );
 });
+const DropdownMenuContext = _react.createContext(void 0);
+const DropdownMenuCloseOnClickContext = _react.createContext(void 0);

package/DEV-cjs/core/Menu/Menu.js

@@ -16,6 +16,9 @@ _export(exports, {
   MenuContext: function () {
     return MenuContext;
   },
+  MenuPortalContext: function () {
+    return MenuPortalContext;
+  },
 });
 const _interop_require_default = require('@swc/helpers/_/_interop_require_default');
 const _interop_require_wildcard = require('@swc/helpers/_/_interop_require_wildcard');
@@ -31,11 +34,13 @@ const Menu = _react.forwardRef((props, ref) => {
     className,
     trigger,
     positionReference,
-    portal = true,
+    portal: portalProp,
     popoverProps: popoverPropsProp,
     children,
     ...rest
   } = props;
+  let menuPortalContext = _react.useContext(MenuPortalContext);
+  let portal = portalProp ?? menuPortalContext;
   let tree = (0, _react1.useFloatingTree)();
   let nodeId = (0, _react1.useFloatingNodeId)();
   let parentId = (0, _react1.useFloatingParentNodeId)();
@@ -134,35 +139,39 @@ const Menu = _react.forwardRef((props, ref) => {
     () => void 0,
     () => void 0,
   );
-  let popoverGetItemProps = ({ focusableItemIndex, userProps }) =>
-    getItemProps({
-      ...userProps,
-      tabIndex:
-        null != activeIndex &&
-        activeIndex >= 0 &&
-        null != focusableItemIndex &&
-        focusableItemIndex >= 0 &&
-        activeIndex === focusableItemIndex
-          ? 0
-          : -1,
-      onFocus: (0, _index.mergeEventHandlers)(userProps?.onFocus, () => {
-        queueMicrotask(() => {
-          setHasFocusedNodeInSubmenu(true);
-        });
-        tree?.events.emit('onNodeFocused', {
-          nodeId: nodeId,
-          parentId: parentId,
-        });
+  let popoverGetItemProps = _react.useCallback(
+    ({ focusableItemIndex, userProps }) =>
+      getItemProps({
+        ...userProps,
+        tabIndex:
+          null != activeIndex &&
+          activeIndex >= 0 &&
+          null != focusableItemIndex &&
+          focusableItemIndex >= 0 &&
+          activeIndex === focusableItemIndex
+            ? 0
+            : -1,
+        onFocus: (0, _index.mergeEventHandlers)(userProps?.onFocus, () => {
+          queueMicrotask(() => {
+            setHasFocusedNodeInSubmenu(true);
+          });
+          tree?.events.emit('onNodeFocused', {
+            nodeId: nodeId,
+            parentId: parentId,
+          });
+        }),
+        onMouseEnter: (0, _index.mergeEventHandlers)(
+          userProps?.onMouseEnter,
+          (event) => {
+            if (null != focusableItemIndex && focusableItemIndex >= 0)
+              setActiveIndex(focusableItemIndex);
+            if (event.target === event.currentTarget)
+              event.currentTarget.focus();
+          },
+        ),
       }),
-      onMouseEnter: (0, _index.mergeEventHandlers)(
-        userProps?.onMouseEnter,
-        (event) => {
-          if (null != focusableItemIndex && focusableItemIndex >= 0)
-            setActiveIndex(focusableItemIndex);
-          if (event.target === event.currentTarget) event.currentTarget.focus();
-        },
-      ),
-    });
+    [activeIndex, getItemProps, nodeId, parentId, tree?.events],
+  );
   let reference = (0, _index.cloneElementWithRef)(trigger, (triggerChild) =>
     getReferenceProps(
       popover.getReferenceProps({
@@ -201,28 +210,38 @@ const Menu = _react.forwardRef((props, ref) => {
     _react.createElement(
       MenuContext.Provider,
       {
-        value: {
-          popoverGetItemProps,
-          focusableElements,
-        },
+        value: _react.useMemo(
+          () => ({
+            popoverGetItemProps,
+            focusableElements,
+          }),
+          [focusableElements, popoverGetItemProps],
+        ),
       },
       _react.createElement(
-        _Popover.PopoverOpenContext.Provider,
+        MenuPortalContext.Provider,
         {
-          value: popover.open,
+          value: portal,
         },
-        reference,
+        _react.createElement(
+          _Popover.PopoverOpenContext.Provider,
+          {
+            value: popover.open,
+          },
+          reference,
+        ),
+        null != tree
+          ? _react.createElement(
+              _react1.FloatingNode,
+              {
+                id: nodeId,
+              },
+              floating,
+            )
+          : floating,
       ),
-      null != tree
-        ? _react.createElement(
-            _react1.FloatingNode,
-            {
-              id: nodeId,
-            },
-            floating,
-          )
-        : floating,
     ),
   );
 });
 const MenuContext = _react.createContext(void 0);
+const MenuPortalContext = _react.createContext(void 0);

package/DEV-cjs/core/Menu/MenuItem.js

@@ -17,6 +17,7 @@ const _ListItem = require('../List/ListItem.js');
 const _classnames = /*#__PURE__*/ _interop_require_default._(
   require('classnames'),
 );
+const _DropdownMenu = require('../DropdownMenu/DropdownMenu.js');
 const MenuItem = _react.forwardRef((props, forwardedRef) => {
   let {
     className,
@@ -41,6 +42,10 @@ const MenuItem = _react.forwardRef((props, forwardedRef) => {
       'Passing a non-empty submenuItems array and onClick to MenuItem at the same time is not supported. This is because when a non empty submenuItems array is passed, clicking the MenuItem toggles the submenu visibility.',
     );
   let parentMenu = _react.useContext(_Menu.MenuContext);
+  let dropdownMenu = _react.useContext(_DropdownMenu.DropdownMenuContext);
+  let shouldCloseMenuOnClick = _react.useContext(
+    _DropdownMenu.DropdownMenuCloseOnClickContext,
+  );
   let menuItemRef = _react.useRef(null);
   let submenuId = (0, _index.useId)();
   let popoverProps = _react.useMemo(
@@ -59,6 +64,7 @@ const MenuItem = _react.forwardRef((props, forwardedRef) => {
   );
   let onClick = () => {
     if (disabled) return;
+    if (shouldCloseMenuOnClick) dropdownMenu?.close();
     onClickProp?.(value);
   };
   let handlers = {

package/DEV-cjs/core/Tabs/Tabs.js

@@ -84,7 +84,9 @@ const TabsWrapper = _react.forwardRef((props, ref) => {
 TabsWrapper.displayName = 'Tabs.Wrapper';
 const TabList = _react.forwardRef((props, ref) => {
   let { className, children, ...rest } = props;
-  let { type, hasSublabel, color } = (0, _index.useSafeContext)(TabsContext);
+  let { type, hasSublabel, color, orientation } = (0, _index.useSafeContext)(
+    TabsContext,
+  );
   let isClient = (0, _index.useIsClient)();
   let tablistRef = _react.useRef(null);
   let [tablistSizeRef, tabsWidth] = (0, _index.useContainerWidth)(
@@ -110,6 +112,7 @@ const TabList = _react.forwardRef((props, ref) => {
         },
         className,
       ),
+      'data-iui-orientation': orientation,
       role: 'tablist',
       ref: refs,
       ...rest,
@@ -156,10 +159,12 @@ const Tab = _react.forwardRef((props, forwardedRef) => {
     let updateStripe = () => {
       let currentTabRect = tabRef.current?.getBoundingClientRect();
       let tabslistRect = tablistRef.current?.getBoundingClientRect();
+      let currentTabLeftIncludingScroll =
+        (currentTabRect?.x ?? 0) + (tablistRef.current?.scrollLeft ?? 0);
       let tabsStripePosition =
         null != currentTabRect && null != tabslistRect
           ? {
-              horizontal: currentTabRect.x - tabslistRect.x,
+              horizontal: currentTabLeftIncludingScroll - tabslistRect.x,
               vertical: currentTabRect.y - tabslistRect.y,
             }
           : {

package/DEV-cjs/core/Tile/Tile.js

@@ -211,41 +211,39 @@ const TileMoreOptions = _react.forwardRef((props, forwardedRef) => {
   let { className, children = [], buttonProps, ...rest } = props;
   let [isMenuVisible, setIsMenuVisible] = _react.useState(false);
   return _react.createElement(
-    _index.Box,
+    _DropdownMenu.DropdownMenuCloseOnClickContext.Provider,
     {
-      className: (0, _classnames.default)(
-        'iui-tile-more-options',
-        {
-          'iui-visible': isMenuVisible,
-        },
-        className,
-      ),
-      ref: forwardedRef,
-      ...rest,
+      value: true,
     },
     _react.createElement(
-      _DropdownMenu.DropdownMenu,
+      _index.Box,
       {
-        onVisibleChange: setIsMenuVisible,
-        menuItems: (close) =>
-          children?.map((option) =>
-            _react.cloneElement(option, {
-              onClick: (value) => {
-                close();
-                option.props.onClick?.(value);
-              },
-            }),
-          ),
+        className: (0, _classnames.default)(
+          'iui-tile-more-options',
+          {
+            'iui-visible': isMenuVisible,
+          },
+          className,
+        ),
+        ref: forwardedRef,
+        ...rest,
       },
       _react.createElement(
-        _IconButton.IconButton,
+        _DropdownMenu.DropdownMenu,
         {
-          styleType: 'borderless',
-          size: 'small',
-          'aria-label': 'More options',
-          ...buttonProps,
+          onVisibleChange: setIsMenuVisible,
+          menuItems: children,
         },
-        _react.createElement(_index.SvgMore, null),
+        _react.createElement(
+          _IconButton.IconButton,
+          {
+            styleType: 'borderless',
+            size: 'small',
+            'aria-label': 'More options',
+            ...buttonProps,
+          },
+          _react.createElement(_index.SvgMore, null),
+        ),
       ),
     ),
   );

package/DEV-cjs/core/Tree/Tree.js

@@ -217,29 +217,37 @@ const VirtualizedTree = _react.forwardRef(
         ref: (0, _index.useMergedRefs)(ref, parentRef),
       },
       _react.createElement(
-        _index.ShadowRoot,
+        'div',
         {
-          css: virtualizerCss,
+          style: {
+            display: 'contents',
+          },
         },
         _react.createElement(
-          'div',
+          _index.ShadowRoot,
           {
-            'data-iui-virtualizer': 'root',
-            style: {
-              minBlockSize: virtualizer.getTotalSize(),
-            },
+            css: virtualizerCss,
           },
-          _react.createElement('slot', null),
-        ),
-      ),
-      _react.createElement(
-        _react.Fragment,
-        null,
-        virtualizer
-          .getVirtualItems()
-          .map((virtualItem) =>
-            itemRenderer(virtualItem.index, virtualItem, virtualizer),
+          _react.createElement(
+            'div',
+            {
+              'data-iui-virtualizer': 'root',
+              style: {
+                minBlockSize: virtualizer.getTotalSize(),
+              },
+            },
+            _react.createElement('slot', null),
           ),
+        ),
+        _react.createElement(
+          _react.Fragment,
+          null,
+          virtualizer
+            .getVirtualItems()
+            .map((virtualItem) =>
+              itemRenderer(virtualItem.index, virtualItem, virtualizer),
+            ),
+        ),
       ),
     );
   },

package/DEV-cjs/styles.js

@@ -1,6 +1,6 @@
 'use strict';
 Object.defineProperty(exports, Symbol.toStringTag, { value: 'Module' });
-const e = '3.17.2';
+const e = '3.17.4';
 const u = new Proxy(
   {},
   {

package/DEV-esm/core/ComboBox/ComboBox.js

@@ -367,27 +367,23 @@ export const ComboBox = React.forwardRef((props, forwardedRef) => {
           disabled: inputProps?.disabled,
           ...rest,
         },
-        React.createElement(
-          React.Fragment,
-          null,
-          React.createElement(ComboBoxInput, {
-            value: inputValue,
-            disabled: inputProps?.disabled,
-            ...inputProps,
-            onChange: handleOnInput,
-            selectTags: isMultipleEnabled(selectedIndexes, multiple)
-              ? selectedIndexes
-                  ?.map((index) => {
-                    let item = options[index];
-                    return React.createElement(SelectTag, {
-                      key: item.label,
-                      label: item.label,
-                    });
-                  })
-                  .filter(Boolean)
-              : void 0,
-          }),
-        ),
+        React.createElement(ComboBoxInput, {
+          value: inputValue,
+          disabled: inputProps?.disabled,
+          ...inputProps,
+          onChange: handleOnInput,
+          selectTags: isMultipleEnabled(selectedIndexes, multiple)
+            ? selectedIndexes
+                ?.map((index) => {
+                  let item = options[index];
+                  return React.createElement(SelectTag, {
+                    key: item.label,
+                    label: item.label,
+                  });
+                })
+                .filter(Boolean)
+            : void 0,
+        }),
         React.createElement(ComboBoxEndIcon, {
           ...endIconProps,
           disabled: inputProps?.disabled,

package/DEV-esm/core/ComboBox/ComboBoxMenu.js

@@ -59,8 +59,12 @@ let VirtualizedComboBoxMenu = (props) => {
     [filteredOptions, getMenuItem, children, virtualizer.measureElement],
   );
   return React.createElement(
-    React.Fragment,
-    null,
+    'div',
+    {
+      style: {
+        display: 'contents',
+      },
+    },
     React.createElement(
       ShadowRoot,
       {

package/DEV-esm/core/DropdownMenu/DropdownMenu.js

@@ -31,34 +31,50 @@ let DropdownMenuContent = React.forwardRef((props, forwardedRef) => {
     visibleProp,
     onVisibleChange,
   );
+  let close = React.useCallback(() => {
+    setVisible(false);
+  }, [setVisible]);
   let menuContent = React.useMemo(() => {
-    if ('function' == typeof menuItems)
-      return menuItems(() => setVisible(false));
+    if ('function' == typeof menuItems) return menuItems(close);
     return menuItems;
-  }, [menuItems, setVisible]);
+  }, [close, menuItems]);
+  let dropdownMenuContextValue = React.useMemo(
+    () => ({
+      close,
+    }),
+    [close],
+  );
   return React.createElement(
-    Menu,
+    DropdownMenuContext.Provider,
     {
-      trigger: children,
-      onKeyDown: mergeEventHandlers(props.onKeyDown, (e) => {
-        if (e.defaultPrevented) return;
-        if ('Tab' === e.key) setVisible(false);
-      }),
-      role: role,
-      ref: forwardedRef,
-      portal: portal,
-      popoverProps: React.useMemo(
-        () => ({
-          placement,
-          matchWidth,
-          visible,
-          onVisibleChange: setVisible,
-          middleware,
-        }),
-        [matchWidth, middleware, placement, setVisible, visible],
-      ),
-      ...rest,
+      value: dropdownMenuContextValue,
     },
-    menuContent,
+    React.createElement(
+      Menu,
+      {
+        trigger: children,
+        onKeyDown: mergeEventHandlers(props.onKeyDown, (e) => {
+          if (e.defaultPrevented) return;
+          if ('Tab' === e.key) setVisible(false);
+        }),
+        role: role,
+        ref: forwardedRef,
+        portal: portal,
+        popoverProps: React.useMemo(
+          () => ({
+            placement,
+            matchWidth,
+            visible,
+            onVisibleChange: setVisible,
+            middleware,
+          }),
+          [matchWidth, middleware, placement, setVisible, visible],
+        ),
+        ...rest,
+      },
+      menuContent,
+    ),
   );
 });
+export const DropdownMenuContext = React.createContext(void 0);
+export const DropdownMenuCloseOnClickContext = React.createContext(void 0);