@itwin/itwinui-react 3.14.0 → 3.14.1

package/cjs/core/Table/Table.js

@@ -35,7 +35,6 @@ const singleRowSelectedAction = 'singleRowSelected';
 const shiftRowSelectedAction = 'shiftRowSelected';
 const tableResizeStartAction = 'tableResizeStart';
 const tableResizeEndAction = 'tableResizeEnd';
-const logWarning = (0, _index.createWarningLogger)();
 const flattenColumns = (columns) => {
   let flatColumns = [];
   columns.forEach((column) => {
@@ -282,6 +281,7 @@ const Table = (props) => {
     setGlobalFilter,
   } = instance;
   let headerGroups = _headerGroups;
+  let logWarning = (0, _index.useWarningLogger)();
   if (1 === columns.length && 'columns' in columns[0])
     headerGroups = _headerGroups.slice(1);
   let ariaDataAttributes = Object.entries(rest).reduce(
@@ -571,7 +571,7 @@ const Table = (props) => {
                 let dragAndDropProps = column.getDragAndDropProps();
                 return _react.createElement(_ColumnHeader.ColumnHeader, {
                   ...dragAndDropProps,
-                  key: dragAndDropProps.key,
+                  key: dragAndDropProps.key || column.id || index,
                   columnRefs: columnRefs,
                   column: column,
                   index: index,

package/cjs/styles.js

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

package/cjs/utils/functions/dev.d.ts

@@ -1,17 +1,2 @@
 declare const isUnitTest: boolean;
-/**
- * Returns a function that can be used to log one-time warnings in dev environments.
- *
- * **Note**: The actual log call should be wrapped in a check against `process.env.NODE_ENV === 'development'`
- * to ensure that it is removed from the production build output (by SWC).
- * Read more about the [`NODE_ENV` convention](https://nodejs.org/en/learn/getting-started/nodejs-the-difference-between-development-and-production).
- *
- * @example
- * const logWarning = createWarningLogger();
- *
- * if (process.env.NODE_ENV === 'development') {
- *   logWarning("please don't use this")
- * }
- */
-declare const createWarningLogger: () => (message: string) => void;
-export { isUnitTest, createWarningLogger };
+export { isUnitTest };

package/cjs/utils/functions/dev.js

@@ -2,18 +2,9 @@
 Object.defineProperty(exports, '__esModule', {
   value: true,
 });
-function _export(target, all) {
-  for (var name in all)
-    Object.defineProperty(target, name, {
-      enumerable: true,
-      get: all[name],
-    });
-}
-_export(exports, {
-  createWarningLogger: function () {
-    return createWarningLogger;
-  },
-  isUnitTest: function () {
+Object.defineProperty(exports, 'isUnitTest', {
+  enumerable: true,
+  get: function () {
     return isUnitTest;
   },
 });
@@ -24,4 +15,3 @@ const isMocha =
     `${globalThis.beforeEach}`.replace(/\s/g, '');
 const isVitest = void 0 !== globalThis.__vitest_index__;
 const isUnitTest = isJest || isVitest || isMocha;
-const createWarningLogger = () => () => {};

package/cjs/utils/hooks/index.d.ts

@@ -14,3 +14,4 @@ export * from './useId.js';
 export * from './useControlledState.js';
 export * from './useSyncExternalStore.js';
 export * from './useVirtualScroll.js';
+export * from './useWarningLogger.js';

package/cjs/utils/hooks/index.js

@@ -19,3 +19,4 @@ _export_star._(require('./useId.js'), exports);
 _export_star._(require('./useControlledState.js'), exports);
 _export_star._(require('./useSyncExternalStore.js'), exports);
 _export_star._(require('./useVirtualScroll.js'), exports);
+_export_star._(require('./useWarningLogger.js'), exports);

package/cjs/utils/hooks/useWarningLogger.d.ts

@@ -0,0 +1,15 @@
+/**
+ * Returns a function that can be used to log one-time warnings in dev environments.
+ *
+ * **Note**: The actual log call should be wrapped in a check against `process.env.NODE_ENV === 'development'`
+ * to ensure that it is removed from the production build output (by SWC).
+ * Read more about the [`NODE_ENV` convention](https://nodejs.org/en/learn/getting-started/nodejs-the-difference-between-development-and-production).
+ *
+ * @example
+ * const logWarning = useWarningLogger();
+ *
+ * if (process.env.NODE_ENV === 'development') {
+ *   logWarning("please don't use this")
+ * }
+ */
+export declare const useWarningLogger: () => (message: string) => void;

package/cjs/utils/hooks/useWarningLogger.js

@@ -0,0 +1,17 @@
+'use strict';
+Object.defineProperty(exports, '__esModule', {
+  value: true,
+});
+Object.defineProperty(exports, 'useWarningLogger', {
+  enumerable: true,
+  get: function () {
+    return useWarningLogger;
+  },
+});
+const _interop_require_wildcard = require('@swc/helpers/_/_interop_require_wildcard');
+const _react = _interop_require_wildcard._(require('react'));
+const _dev = require('../functions/dev.js');
+const _React = _react;
+const ReactInternals =
+  _React.__SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED;
+const useWarningLogger = () => () => {};

package/esm/core/Breadcrumbs/Breadcrumbs.js

@@ -5,11 +5,10 @@ import {
   useOverflow,
   SvgChevronRight,
   Box,
-  createWarningLogger,
+  useWarningLogger,
 } from '../../utils/index.js';
 import { Button } from '../Buttons/Button.js';
 import { Anchor } from '../Typography/Anchor.js';
-let logWarning = createWarningLogger();
 let BreadcrumbsComponent = React.forwardRef((props, ref) => {
   let {
     children: items,
@@ -102,6 +101,7 @@ let BreadcrumbsComponent = React.forwardRef((props, ref) => {
 });
 let ListItem = ({ item, isActive }) => {
   let children = item;
+  let logWarning = useWarningLogger();
   if (
     children?.type === 'span' ||
     children?.type === 'a' ||

package/esm/core/Buttons/IconButton.js

@@ -1,11 +1,10 @@
 import cx from 'classnames';
 import * as React from 'react';
-import { Box, ButtonBase, createWarningLogger } from '../../utils/index.js';
+import { Box, ButtonBase, useWarningLogger } from '../../utils/index.js';
 import { Tooltip } from '../Tooltip/Tooltip.js';
 import { VisuallyHidden } from '../VisuallyHidden/VisuallyHidden.js';
 import { ButtonGroupContext } from '../ButtonGroup/ButtonGroup.js';
 import { PopoverOpenContext } from '../Popover/Popover.js';
-let logWarning = createWarningLogger();
 export const IconButton = React.forwardRef((props, ref) => {
   let {
     isActive,
@@ -21,6 +20,7 @@ export const IconButton = React.forwardRef((props, ref) => {
   } = props;
   let buttonGroupOrientation = React.useContext(ButtonGroupContext);
   let hasPopoverOpen = React.useContext(PopoverOpenContext);
+  let logWarning = useWarningLogger();
   let button = React.createElement(
     ButtonBase,
     {

package/esm/core/Menu/MenuItem.js

@@ -3,12 +3,11 @@ import {
   SvgCaretRightSmall,
   useMergedRefs,
   useId,
-  createWarningLogger,
+  useWarningLogger,
 } from '../../utils/index.js';
 import { Menu, MenuContext } from './Menu.js';
 import { ListItem } from '../List/ListItem.js';
 import cx from 'classnames';
-let logWarning = createWarningLogger();
 export const MenuItem = React.forwardRef((props, forwardedRef) => {
   let {
     className,
@@ -27,6 +26,7 @@ export const MenuItem = React.forwardRef((props, forwardedRef) => {
     subMenuItems = [],
     ...rest
   } = props;
+  let logWarning = useWarningLogger();
   let parentMenu = React.useContext(MenuContext);
   let menuItemRef = React.useRef(null);
   let submenuId = useId();

package/esm/core/Popover/Popover.js

@@ -32,7 +32,6 @@ 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 usePopover = (options) => {
   let {
@@ -224,12 +223,7 @@ export const Popover = React.forwardRef((props, forwardedRef) => {
     role: 'dialog',
     middleware,
   });
-  let [popoverElement, setPopoverElement] = React.useState();
-  let popoverRef = useMergedRefs(
-    popover.refs.setFloating,
-    forwardedRef,
-    setPopoverElement,
-  );
+  let popoverRef = useMergedRefs(popover.refs.setFloating, forwardedRef);
   let triggerId = `${useId()}-trigger`;
   let hasAriaLabel = !!props['aria-labelledby'] || !!props['aria-label'];
   useLayoutEffect(() => {
@@ -258,34 +252,27 @@ export const Popover = React.forwardRef((props, forwardedRef) => {
             portal: portal,
           },
           React.createElement(
-            ThemeProvider,
+            FloatingFocusManager,
             {
-              portalContainer: popoverElement,
+              context: popover.context,
+              modal: false,
             },
-            React.createElement(DisplayContents, null),
             React.createElement(
-              FloatingFocusManager,
+              Box,
               {
-                context: popover.context,
-                modal: false,
+                className: cx(
+                  {
+                    'iui-popover-surface': applyBackground,
+                  },
+                  className,
+                ),
+                'aria-labelledby': hasAriaLabel
+                  ? void 0
+                  : popover.refs.domReference.current?.id,
+                ...popover.getFloatingProps(rest),
+                ref: popoverRef,
               },
-              React.createElement(
-                Box,
-                {
-                  className: cx(
-                    {
-                      'iui-popover-surface': applyBackground,
-                    },
-                    className,
-                  ),
-                  'aria-labelledby': hasAriaLabel
-                    ? void 0
-                    : popover.refs.domReference.current?.id,
-                  ...popover.getFloatingProps(rest),
-                  ref: popoverRef,
-                },
-                content,
-              ),
+              content,
             ),
           ),
         )

package/esm/core/Table/Table.js

@@ -18,7 +18,7 @@ import {
   useResizeObserver,
   useLayoutEffect,
   Box,
-  createWarningLogger,
+  useWarningLogger,
   ShadowRoot,
   useMergedRefs,
   useLatestRef,
@@ -52,7 +52,6 @@ let singleRowSelectedAction = 'singleRowSelected';
 let shiftRowSelectedAction = 'shiftRowSelected';
 export const tableResizeStartAction = 'tableResizeStart';
 let tableResizeEndAction = 'tableResizeEnd';
-let logWarning = createWarningLogger();
 let flattenColumns = (columns) => {
   let flatColumns = [];
   columns.forEach((column) => {
@@ -292,6 +291,7 @@ export const Table = (props) => {
     setGlobalFilter,
   } = instance;
   let headerGroups = _headerGroups;
+  let logWarning = useWarningLogger();
   if (1 === columns.length && 'columns' in columns[0])
     headerGroups = _headerGroups.slice(1);
   let ariaDataAttributes = Object.entries(rest).reduce(
@@ -578,7 +578,7 @@ export const Table = (props) => {
                 let dragAndDropProps = column.getDragAndDropProps();
                 return React.createElement(ColumnHeader, {
                   ...dragAndDropProps,
-                  key: dragAndDropProps.key,
+                  key: dragAndDropProps.key || column.id || index,
                   columnRefs: columnRefs,
                   column: column,
                   index: index,

package/esm/styles.js

@@ -1,4 +1,4 @@
-const t = '3.14.0';
+const t = '3.14.1';
 const u = new Proxy(
   {},
   {

package/esm/utils/functions/dev.d.ts

@@ -1,17 +1,2 @@
 declare const isUnitTest: boolean;
-/**
- * Returns a function that can be used to log one-time warnings in dev environments.
- *
- * **Note**: The actual log call should be wrapped in a check against `process.env.NODE_ENV === 'development'`
- * to ensure that it is removed from the production build output (by SWC).
- * Read more about the [`NODE_ENV` convention](https://nodejs.org/en/learn/getting-started/nodejs-the-difference-between-development-and-production).
- *
- * @example
- * const logWarning = createWarningLogger();
- *
- * if (process.env.NODE_ENV === 'development') {
- *   logWarning("please don't use this")
- * }
- */
-declare const createWarningLogger: () => (message: string) => void;
-export { isUnitTest, createWarningLogger };
+export { isUnitTest };

package/esm/utils/functions/dev.js

@@ -5,5 +5,4 @@ let isMocha =
     `${globalThis.beforeEach}`.replace(/\s/g, '');
 let isVitest = void 0 !== globalThis.__vitest_index__;
 let isUnitTest = isJest || isVitest || isMocha;
-let createWarningLogger = () => () => {};
-export { isUnitTest, createWarningLogger };
+export { isUnitTest };

package/esm/utils/hooks/index.d.ts

@@ -14,3 +14,4 @@ export * from './useId.js';
 export * from './useControlledState.js';
 export * from './useSyncExternalStore.js';
 export * from './useVirtualScroll.js';
+export * from './useWarningLogger.js';

package/esm/utils/hooks/index.js

@@ -14,3 +14,4 @@ export * from './useId.js';
 export * from './useControlledState.js';
 export * from './useSyncExternalStore.js';
 export * from './useVirtualScroll.js';
+export * from './useWarningLogger.js';

package/esm/utils/hooks/useWarningLogger.d.ts

@@ -0,0 +1,15 @@
+/**
+ * Returns a function that can be used to log one-time warnings in dev environments.
+ *
+ * **Note**: The actual log call should be wrapped in a check against `process.env.NODE_ENV === 'development'`
+ * to ensure that it is removed from the production build output (by SWC).
+ * Read more about the [`NODE_ENV` convention](https://nodejs.org/en/learn/getting-started/nodejs-the-difference-between-development-and-production).
+ *
+ * @example
+ * const logWarning = useWarningLogger();
+ *
+ * if (process.env.NODE_ENV === 'development') {
+ *   logWarning("please don't use this")
+ * }
+ */
+export declare const useWarningLogger: () => (message: string) => void;

package/esm/utils/hooks/useWarningLogger.js

@@ -0,0 +1,5 @@
+import * as React from 'react';
+import { isUnitTest } from '../functions/dev.js';
+let _React = React;
+let ReactInternals = _React.__SECRET_INTERNALS_DO_NOT_USE_OR_YOU_WILL_BE_FIRED;
+export const useWarningLogger = () => () => {};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@itwin/itwinui-react",
-  "version": "3.14.0",
+  "version": "3.14.1",
   "author": "Bentley Systems",
   "license": "MIT",
   "type": "module",