@itwin/itwinui-react 3.15.3 → 3.15.4

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

@@ -1,105 +1,124 @@
 import * as React from 'react';
 import cx from 'classnames';
 import {
-  useMergedRefs,
-  useOverflow,
   SvgChevronRight,
   Box,
+  OverflowContainer,
   useWarningLogger,
 } from '../../utils/index.js';
 import { Button } from '../Buttons/Button.js';
 import { Anchor } from '../Typography/Anchor.js';
 let BreadcrumbsComponent = React.forwardRef((props, ref) => {
   let {
-    children: items,
-    currentIndex = items.length - 1,
+    children: childrenProp,
+    currentIndex = React.Children.count(childrenProp) - 1,
     separator,
     overflowButton,
     className,
     ...rest
   } = props;
-  let [overflowRef, visibleCount] = useOverflow(items);
-  let refs = useMergedRefs(overflowRef, ref);
+  let items = React.useMemo(
+    () => React.Children.toArray(childrenProp),
+    [childrenProp],
+  );
   return React.createElement(
     Box,
     {
       as: 'nav',
       className: cx('iui-breadcrumbs', className),
-      ref: refs,
+      ref: ref,
       'aria-label': 'Breadcrumb',
       ...rest,
     },
     React.createElement(
-      Box,
+      OverflowContainer,
       {
         as: 'ol',
+        itemsCount: items.length,
         className: 'iui-breadcrumbs-list',
       },
-      visibleCount > 1 &&
+      React.createElement(
+        BreadcrumbContent,
+        {
+          currentIndex: currentIndex,
+          overflowButton: overflowButton,
+          separator: separator,
+        },
+        items,
+      ),
+    ),
+  );
+});
+BreadcrumbsComponent.displayName = 'Breadcrumbs';
+let BreadcrumbContent = (props) => {
+  let { children: items, currentIndex, overflowButton, separator } = props;
+  let { visibleCount } = OverflowContainer.useContext();
+  return React.createElement(
+    React.Fragment,
+    null,
+    visibleCount > 1 &&
+      React.createElement(
+        React.Fragment,
+        null,
+        React.createElement(ListItem, {
+          item: items[0],
+          isActive: 0 === currentIndex,
+        }),
+        React.createElement(Separator, {
+          separator: separator,
+        }),
+      ),
+    items.length - visibleCount > 0 &&
+      React.createElement(
+        React.Fragment,
+        null,
         React.createElement(
-          React.Fragment,
-          null,
-          React.createElement(ListItem, {
-            item: items[0],
-            isActive: 0 === currentIndex,
-          }),
-          React.createElement(Separator, {
-            separator: separator,
-          }),
+          Box,
+          {
+            as: 'li',
+            className: 'iui-breadcrumbs-item',
+          },
+          overflowButton
+            ? overflowButton(visibleCount)
+            : React.createElement(
+                Box,
+                {
+                  as: 'span',
+                  className: 'iui-breadcrumbs-content',
+                },
+                '…',
+              ),
         ),
-      items.length - visibleCount > 0 &&
-        React.createElement(
+        React.createElement(Separator, {
+          separator: separator,
+        }),
+      ),
+    items
+      .slice(
+        visibleCount > 1 ? items.length - visibleCount + 1 : items.length - 1,
+      )
+      .map((_, _index) => {
+        let index =
+          visibleCount > 1
+            ? 1 + (items.length - visibleCount) + _index
+            : items.length - 1;
+        return React.createElement(
           React.Fragment,
-          null,
-          React.createElement(
-            Box,
-            {
-              as: 'li',
-              className: 'iui-breadcrumbs-item',
-            },
-            overflowButton
-              ? overflowButton(visibleCount)
-              : React.createElement(
-                  Box,
-                  {
-                    as: 'span',
-                    className: 'iui-breadcrumbs-content',
-                  },
-                  '…',
-                ),
-          ),
-          React.createElement(Separator, {
-            separator: separator,
+          {
+            key: index,
+          },
+          React.createElement(ListItem, {
+            item: items[index],
+            isActive: currentIndex === index,
           }),
-        ),
-      items
-        .slice(
-          visibleCount > 1 ? items.length - visibleCount + 1 : items.length - 1,
-        )
-        .map((_, _index) => {
-          let index =
-            visibleCount > 1
-              ? 1 + (items.length - visibleCount) + _index
-              : items.length - 1;
-          return React.createElement(
-            React.Fragment,
-            {
-              key: index,
-            },
-            React.createElement(ListItem, {
-              item: items[index],
-              isActive: currentIndex === index,
+          index < items.length - 1 &&
+            React.createElement(Separator, {
+              separator: separator,
             }),
-            index < items.length - 1 &&
-              React.createElement(Separator, {
-                separator: separator,
-              }),
-          );
-        }),
-    ),
+        );
+      }),
   );
-});
-BreadcrumbsComponent.displayName = 'Breadcrumbs';
+};
 let ListItem = ({ item, isActive }) => {
   let children = item;
   let logWarning = useWarningLogger();

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

@@ -1,6 +1,6 @@
 import * as React from 'react';
 import cx from 'classnames';
-import { useOverflow, useMergedRefs, Box } from '../../utils/index.js';
+import { Box, OverflowContainer } from '../../utils/index.js';
 import {
   Composite,
   CompositeItem,
@@ -93,14 +93,12 @@ let OverflowGroup = React.forwardRef((props, forwardedRef) => {
     () => React.Children.toArray(childrenProp).filter(Boolean),
     [childrenProp],
   );
-  let [overflowRef, visibleCount] = useOverflow(
-    items,
-    !overflowButton,
-    orientation,
-  );
   return React.createElement(
-    BaseGroup,
+    OverflowContainer,
     {
+      as: BaseGroup,
+      itemsCount: items.length,
+      overflowOrientation: orientation,
       orientation: orientation,
       ...rest,
       className: cx(
@@ -110,27 +108,34 @@ let OverflowGroup = React.forwardRef((props, forwardedRef) => {
         },
         props.className,
       ),
-      ref: useMergedRefs(forwardedRef, overflowRef),
+      ref: forwardedRef,
     },
-    (() => {
-      if (!(visibleCount < items.length)) return items;
-      let overflowStart =
-        'start' === overflowPlacement
-          ? items.length - visibleCount
-          : visibleCount - 1;
-      return React.createElement(
-        React.Fragment,
-        null,
-        overflowButton &&
-          'start' === overflowPlacement &&
-          overflowButton(overflowStart),
-        'start' === overflowPlacement
-          ? items.slice(overflowStart + 1)
-          : items.slice(0, Math.max(0, overflowStart)),
-        overflowButton &&
-          'end' === overflowPlacement &&
-          overflowButton(overflowStart),
-      );
-    })(),
+    React.createElement(OverflowGroupContent, {
+      overflowButton: overflowButton,
+      overflowPlacement: overflowPlacement,
+      items: items,
+    }),
   );
 });
+let OverflowGroupContent = (props) => {
+  let { overflowButton, overflowPlacement, items } = props;
+  let { visibleCount } = OverflowContainer.useContext();
+  let overflowStart =
+    'start' === overflowPlacement
+      ? items.length - visibleCount
+      : visibleCount - 1;
+  if (!(visibleCount < items.length)) return items;
+  return React.createElement(
+    React.Fragment,
+    null,
+    overflowButton &&
+      'start' === overflowPlacement &&
+      overflowButton(overflowStart),
+    'start' === overflowPlacement
+      ? items.slice(overflowStart + 1)
+      : items.slice(0, Math.max(0, overflowStart)),
+    overflowButton &&
+      'end' === overflowPlacement &&
+      overflowButton(overflowStart),
+  );
+};

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

@@ -113,12 +113,14 @@ export const ComboBox = React.forwardRef((props, forwardedRef) => {
         setFocusedIndex(selectedIndexes ?? -1);
     } else {
       setFocusedIndex(-1);
-      if (!isMultipleEnabled(selectedIndexes, multiple))
-        setInputValue(
-          selectedIndexes >= 0
-            ? optionsRef.current[selectedIndexes]?.label ?? ''
-            : '',
-        );
+      isMultipleEnabled(selectedIndexes, multiple)
+        ? setInputValue('')
+        : setInputValue(
+            selectedIndexes >= 0
+              ? optionsRef.current[selectedIndexes]?.label ?? ''
+              : '',
+          );
+      setIsInputDirty(false);
     }
   }, [isOpen, multiple, optionsRef, selectedIndexes]);
   let previousOptions = React.useRef(options);
@@ -216,6 +218,7 @@ export const ComboBox = React.forwardRef((props, forwardedRef) => {
             .filter(Boolean)
             .join(', '),
         );
+        setInputValue('');
       } else {
         setSelectedIndexes(__originalIndex);
         hide();

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

@@ -11,6 +11,7 @@ import {
 } from '../../utils/index.js';
 import { IconButton } from '../Buttons/IconButton.js';
 import { TimePicker } from '../TimePicker/TimePicker.js';
+import { PopoverInitialFocusContext } from '../Popover/Popover.js';
 let isSameDay = (a, b) =>
   a &&
   b &&
@@ -175,6 +176,11 @@ export const DatePicker = React.forwardRef((props, forwardedRef) => {
         currentDate.getFullYear(),
     );
   }, [date, setMonthAndYear, startDate, endDate, enableRangeSelect]);
+  let popoverInitialFocusContext = React.useContext(PopoverInitialFocusContext);
+  React.useLayoutEffect(() => {
+    if (setFocus && popoverInitialFocusContext)
+      popoverInitialFocusContext.setInitialFocus(-1);
+  }, [popoverInitialFocusContext, setFocus]);
   let days = React.useMemo(() => {
     let offsetToFirst = new Date(
       displayedYear,
@@ -499,10 +505,12 @@ export const DatePicker = React.forwardRef((props, forwardedRef) => {
                   role: 'option',
                   tabIndex: isSameDay(weekDay, focusedDay) ? 0 : -1,
                   'aria-disabled': isDisabled ? 'true' : void 0,
-                  ref: (element) =>
-                    isSameDay(weekDay, focusedDay) &&
-                    needFocus.current &&
-                    element?.focus(),
+                  ref: (element) => {
+                    if (isSameDay(weekDay, focusedDay) && needFocus.current)
+                      setTimeout(() => {
+                        element?.focus();
+                      });
+                  },
                   ...dayProps,
                   className: cx(getDayClass(weekDay), dayProps?.className),
                 },

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

@@ -33,7 +33,9 @@ import {
   useMergedRefs,
 } from '../../utils/index.js';
 import { usePortalTo } from '../../utils/components/Portal.js';
+import { ThemeProvider } from '../ThemeProvider/ThemeProvider.js';
 export const PopoverOpenContext = React.createContext(void 0);
+export const PopoverInitialFocusContext = React.createContext(void 0);
 export const usePopover = (options) => {
   let {
     placement = 'bottom-start',
@@ -231,7 +233,12 @@ export const Popover = React.forwardRef((props, forwardedRef) => {
     role: 'dialog',
     middleware,
   });
-  let popoverRef = useMergedRefs(popover.refs.setFloating, forwardedRef);
+  let [popoverElement, setPopoverElement] = React.useState();
+  let popoverRef = useMergedRefs(
+    popover.refs.setFloating,
+    forwardedRef,
+    setPopoverElement,
+  );
   let triggerId = `${useId()}-trigger`;
   let hasAriaLabel = !!props['aria-labelledby'] || !!props['aria-label'];
   useLayoutEffect(() => {
@@ -239,6 +246,13 @@ export const Popover = React.forwardRef((props, forwardedRef) => {
     popover.refs.setPositionReference(positionReference);
     return () => void popover.refs.setPositionReference(null);
   }, [popover.refs, positionReference]);
+  let [initialFocus, setInitialFocus] = React.useState();
+  let initialFocusContextValue = React.useMemo(
+    () => ({
+      setInitialFocus,
+    }),
+    [],
+  );
   return React.createElement(
     React.Fragment,
     null,
@@ -255,32 +269,46 @@ export const Popover = React.forwardRef((props, forwardedRef) => {
     ),
     popover.open
       ? React.createElement(
-          PopoverPortal,
+          PopoverInitialFocusContext.Provider,
           {
-            portal: portal,
+            value: initialFocusContextValue,
           },
           React.createElement(
-            FloatingFocusManager,
+            PopoverPortal,
             {
-              context: popover.context,
-              modal: false,
+              portal: portal,
             },
             React.createElement(
-              Box,
+              ThemeProvider,
               {
-                className: cx(
+                portalContainer: popoverElement,
+              },
+              React.createElement(DisplayContents, null),
+              React.createElement(
+                FloatingFocusManager,
+                {
+                  context: popover.context,
+                  modal: false,
+                  initialFocus: initialFocus,
+                },
+                React.createElement(
+                  Box,
                   {
-                    'iui-popover-surface': applyBackground,
+                    className: cx(
+                      {
+                        'iui-popover-surface': applyBackground,
+                      },
+                      className,
+                    ),
+                    'aria-labelledby': hasAriaLabel
+                      ? void 0
+                      : popover.refs.domReference.current?.id,
+                    ...popover.getFloatingProps(rest),
+                    ref: popoverRef,
                   },
-                  className,
+                  content,
                 ),
-                'aria-labelledby': hasAriaLabel
-                  ? void 0
-                  : popover.refs.domReference.current?.id,
-                ...popover.getFloatingProps(rest),
-                ref: popoverRef,
-              },
-              content,
+              ),
             ),
           ),
         )

package/DEV-esm/core/Select/SelectTagContainer.js

@@ -1,26 +1,37 @@
 import * as React from 'react';
 import cx from 'classnames';
-import { useOverflow, useMergedRefs, Box } from '../../utils/index.js';
 import { SelectTag } from './SelectTag.js';
+import { OverflowContainer } from '../../utils/index.js';
 export const SelectTagContainer = React.forwardRef((props, ref) => {
-  let { tags, className, ...rest } = props;
-  let [containerRef, visibleCount] = useOverflow(tags);
-  let refs = useMergedRefs(ref, containerRef);
+  let { tags: tagsProp, className, ...rest } = props;
+  let tags = React.useMemo(() => React.Children.toArray(tagsProp), [tagsProp]);
   return React.createElement(
-    Box,
+    OverflowContainer,
     {
+      itemsCount: tags.length,
       className: cx('iui-select-tag-container', className),
-      ref: refs,
+      ref: ref,
       ...rest,
     },
+    React.createElement(SelectTagContainerContent, {
+      ...props,
+      tags: tags,
+    }),
+  );
+});
+let SelectTagContainerContent = (props) => {
+  let { tags } = props;
+  let { visibleCount } = OverflowContainer.useContext();
+  return React.createElement(
+    React.Fragment,
+    null,
+    visibleCount < tags.length ? tags.slice(0, visibleCount - 1) : tags,
     React.createElement(
-      React.Fragment,
+      OverflowContainer.OverflowNode,
       null,
-      visibleCount < tags.length ? tags.slice(0, visibleCount - 1) : tags,
-      visibleCount < tags.length &&
-        React.createElement(SelectTag, {
-          label: `+${tags.length - visibleCount + 1} item(s)`,
-        }),
+      React.createElement(SelectTag, {
+        label: `+${tags.length - visibleCount + 1} item(s)`,
+      }),
     ),
   );
-});
+};

package/DEV-esm/core/Table/ColumnHeader.js

@@ -5,40 +5,39 @@ import {
   LineClamp,
   SvgSortDown,
   SvgSortUp,
+  useMergedRefs,
 } from '../../utils/index.js';
-import { SELECTION_CELL_ID } from './columns/index.js';
 import { FilterToggle } from './filters/FilterToggle.js';
-import { getCellStyle, getSubRowStyle, getStickyStyle } from './utils.js';
+import {
+  getCellStyle,
+  getSubRowStyle,
+  getStickyStyle,
+  TableInstanceContext,
+} from './utils.js';
 import cx from 'classnames';
-export const ColumnHeader = (props) => {
+export const ColumnHeader = React.forwardRef((props, forwardedRef) => {
   let {
-    columnRefs,
     column,
-    index,
     areFiltersSet,
-    hasAnySubRows,
-    headers,
-    state,
-    data,
     isResizable,
     columnResizeMode,
     enableColumnReordering,
     density,
-    visibleColumns,
+    columnHasExpanders,
+    isLast,
+    isTableEmpty,
     ...rest
   } = props;
   let isHeaderDirectClick = React.useRef(false);
+  let instance = React.useContext(TableInstanceContext);
   let COLUMN_MIN_WIDTHS = {
     default: 72,
     withExpander: 108,
   };
   let showFilterButton = (column) =>
-    (0 !== data.length || areFiltersSet) && column.canFilter && !!column.Filter;
-  let showSortButton = (column) => 0 !== data.length && column.canSort;
+    (!isTableEmpty || areFiltersSet) && column.canFilter && !!column.Filter;
+  let showSortButton = (column) => !isTableEmpty && column.canSort;
   let { onClick, ...restSortProps } = column.getSortByToggleProps();
-  let columnHasExpanders =
-    hasAnySubRows &&
-    index === headers.findIndex((c) => c.id !== SELECTION_CELL_ID);
   if ([void 0, 0].includes(column.minWidth)) {
     column.minWidth = columnHasExpanders
       ? COLUMN_MIN_WIDTHS.withExpander
@@ -58,12 +57,12 @@ export const ColumnHeader = (props) => {
       column.columnClassName,
     ),
     style: {
-      ...getCellStyle(column, !!state.isTableResizing),
+      ...getCellStyle(column, !!instance?.state.isTableResizing),
       ...(columnHasExpanders &&
         getSubRowStyle({
           density,
         })),
-      ...getStickyStyle(column, visibleColumns),
+      ...getStickyStyle(column, instance?.visibleColumns ?? []),
       flexWrap: 'wrap',
       columnGap: 'var(--iui-size-xs)',
     },
@@ -75,14 +74,14 @@ export const ColumnHeader = (props) => {
       ...rest,
       key: columnProps.key,
       title: void 0,
-      ref: React.useCallback(
-        (el) => {
-          if (el) {
-            columnRefs.current[column.id] = el;
-            column.resizeWidth = el.getBoundingClientRect().width;
-          }
-        },
-        [column, columnRefs],
+      ref: useMergedRefs(
+        React.useCallback(
+          (el) => {
+            if (el) column.resizeWidth = el.getBoundingClientRect().width;
+          },
+          [column],
+        ),
+        forwardedRef,
       ),
       onMouseDown: () => {
         isHeaderDirectClick.current = true;
@@ -153,7 +152,7 @@ export const ColumnHeader = (props) => {
         ),
       isResizable &&
         column.isResizerVisible &&
-        (index !== headers.length - 1 || 'expand' === columnResizeMode) &&
+        (!isLast || 'expand' === columnResizeMode) &&
         React.createElement(
           Box,
           {
@@ -172,17 +171,17 @@ export const ColumnHeader = (props) => {
           slot: 'resizers',
         }),
       'left' === column.sticky &&
-        state.sticky.isScrolledToRight &&
+        instance?.state.sticky.isScrolledToRight &&
         React.createElement(Box, {
           className: 'iui-table-cell-shadow-right',
           slot: 'shadows',
         }),
       'right' === column.sticky &&
-        state.sticky.isScrolledToLeft &&
+        instance?.state.sticky.isScrolledToLeft &&
         React.createElement(Box, {
           className: 'iui-table-cell-shadow-left',
           slot: 'shadows',
         }),
     ),
   );
-};
+});

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

@@ -24,7 +24,7 @@ import {
   useLatestRef,
   useVirtualScroll,
 } from '../../utils/index.js';
-import { TableColumnsContext } from './utils.js';
+import { TableInstanceContext } from './utils.js';
 import { TableRowMemoized } from './TableRowMemoized.js';
 import { customFilterFunctions } from './filters/customFilterFunctions.js';
 import {
@@ -299,7 +299,6 @@ export const Table = (props) => {
     gotoPage,
     setPageSize,
     flatHeaders,
-    visibleColumns,
     setGlobalFilter,
   } = instance;
   let headerGroups = _headerGroups;
@@ -310,6 +309,10 @@ export const Table = (props) => {
       "Table's `columns` prop should not have a top-level `Header` or sub-columns. They are only allowed to be passed for backwards compatibility.\n See https://github.com/iTwin/iTwinUI/wiki/iTwinUI-react-v2-migration-guide#breaking-changes",
     );
   }
+  if (subComponent && data.some((item) => !!item.subRows?.length))
+    logWarning(
+      'Passing both `subComponent` and `data` with `subRows` is not supported. There are features designed for `subRows` that are not compatible with `subComponent` and vice versa.',
+    );
   let ariaDataAttributes = Object.entries(rest).reduce(
     (result, [key, value]) => {
       if (key.startsWith('data-') || key.startsWith('aria-'))
@@ -547,9 +550,9 @@ export const Table = (props) => {
     updateStickyState();
   }, []);
   return React.createElement(
-    TableColumnsContext.Provider,
+    TableInstanceContext.Provider,
     {
-      value: instance.columns,
+      value: instance,
     },
     React.createElement(
       Box,
@@ -608,19 +611,23 @@ export const Table = (props) => {
                 return React.createElement(ColumnHeader, {
                   ...dragAndDropProps,
                   key: dragAndDropProps.key || column.id || index,
-                  columnRefs: columnRefs,
                   column: column,
-                  index: index,
                   areFiltersSet: areFiltersSet,
-                  hasAnySubRows: hasAnySubRows,
-                  headers: headerGroup.headers,
-                  state: state,
-                  data: data,
+                  columnHasExpanders:
+                    hasAnySubRows &&
+                    index ===
+                      headerGroup.headers.findIndex(
+                        (c) => c.id !== SELECTION_CELL_ID,
+                      ),
+                  isLast: index === headerGroup.headers.length - 1,
+                  isTableEmpty: 0 === data.length,
                   isResizable: isResizable,
                   columnResizeMode: columnResizeMode,
                   enableColumnReordering: enableColumnReordering,
                   density: density,
-                  visibleColumns: visibleColumns,
+                  ref: (el) => {
+                    if (el) columnRefs.current[column.id] = el;
+                  },
                 });
               }),
             ),