@datarobot/design-system 29.2.1 → 29.3.0

package/cjs/table-react/components/table.js

@@ -147,7 +147,9 @@ function Table({
     }
     // Component width depends on Vertical scroll
   }, [columnSizing, width, isVerticalScrollPresent]);
-  const decoratedColumns = (0, _useColumns.useColumns)(columns, enableCheckBoxes, width - (isVerticalScrollPresent ? _types.COLUMN_SIZE.SCROLL_OFFSET : 0), visibility, columnSizing, isRowExpandEnabled, !!getRowActions, localizationProps?.tableBody);
+  const decoratedColumns = (0, _useColumns.useColumns)(columns, enableCheckBoxes, width - (isVerticalScrollPresent ? _types.COLUMN_SIZE.SCROLL_OFFSET : 0), visible || visibility,
+  // we do not update internal state when external one is updated so visibility might fall behind
+  columnSizing, isRowExpandEnabled, !!getRowActions, localizationProps?.tableBody);
   const onChangeSize = fn => {
     const nextData = typeof fn === 'function' ? fn(columnSizing) : fn;
     const targetData = typeof fn === 'function' ? fn({}) : fn;

package/cjs/table-react/hooks/useInnerStateFallback.js

@@ -20,7 +20,7 @@ function useInnerStateFallback(onExternalStateUpdate, externalState, decoratorFn
   const onChangeState = fn => {
     // we are setting innerState and outer  externalState as the same object "nextData"
     // it means that their links should be equal. If their links are not equal, it means externalState was altered by the user
-    // thus expandedRows should take precedence over inner expandedState that has fallen behind
+    // thus external state should take precedence over inner state that has fallen behind
     const currentState = externalState && externalState !== innerState ? externalState : innerState;
     const nextData = typeof fn === 'function' ? fn(currentState) : fn;
     const nextDecoratedData = decoratorFn(nextData);

package/esm/table-react/components/table.js

@@ -139,7 +139,9 @@ export function Table({
     }
     // Component width depends on Vertical scroll
   }, [columnSizing, width, isVerticalScrollPresent]);
-  const decoratedColumns = useColumns(columns, enableCheckBoxes, width - (isVerticalScrollPresent ? COLUMN_SIZE.SCROLL_OFFSET : 0), visibility, columnSizing, isRowExpandEnabled, !!getRowActions, localizationProps?.tableBody);
+  const decoratedColumns = useColumns(columns, enableCheckBoxes, width - (isVerticalScrollPresent ? COLUMN_SIZE.SCROLL_OFFSET : 0), visible || visibility,
+  // we do not update internal state when external one is updated so visibility might fall behind
+  columnSizing, isRowExpandEnabled, !!getRowActions, localizationProps?.tableBody);
   const onChangeSize = fn => {
     const nextData = typeof fn === 'function' ? fn(columnSizing) : fn;
     const targetData = typeof fn === 'function' ? fn({}) : fn;

package/esm/table-react/hooks/useInnerStateFallback.js

@@ -14,7 +14,7 @@ export function useInnerStateFallback(onExternalStateUpdate, externalState, deco
   const onChangeState = fn => {
     // we are setting innerState and outer  externalState as the same object "nextData"
     // it means that their links should be equal. If their links are not equal, it means externalState was altered by the user
-    // thus expandedRows should take precedence over inner expandedState that has fallen behind
+    // thus external state should take precedence over inner state that has fallen behind
     const currentState = externalState && externalState !== innerState ? externalState : innerState;
     const nextData = typeof fn === 'function' ? fn(currentState) : fn;
     const nextDecoratedData = decoratorFn(nextData);

package/js/bundle/bundle.js

@@ -78004,7 +78004,9 @@ function Table(_ref) {
     }
     // Component width depends on Vertical scroll
   }, [columnSizing, width, isVerticalScrollPresent]);
-  var decoratedColumns = (0,_hooks_useColumns__WEBPACK_IMPORTED_MODULE_9__.useColumns)(columns, enableCheckBoxes, width - (isVerticalScrollPresent ? _types__WEBPACK_IMPORTED_MODULE_18__.COLUMN_SIZE.SCROLL_OFFSET : 0), visibility, columnSizing, isRowExpandEnabled, !!getRowActions, localizationProps === null || localizationProps === void 0 ? void 0 : localizationProps.tableBody);
+  var decoratedColumns = (0,_hooks_useColumns__WEBPACK_IMPORTED_MODULE_9__.useColumns)(columns, enableCheckBoxes, width - (isVerticalScrollPresent ? _types__WEBPACK_IMPORTED_MODULE_18__.COLUMN_SIZE.SCROLL_OFFSET : 0), visible || visibility,
+  // we do not update internal state when external one is updated so visibility might fall behind
+  columnSizing, isRowExpandEnabled, !!getRowActions, localizationProps === null || localizationProps === void 0 ? void 0 : localizationProps.tableBody);
   var onChangeSize = function onChangeSize(fn) {
     var nextData = typeof fn === 'function' ? fn(columnSizing) : fn;
     var targetData = typeof fn === 'function' ? fn({}) : fn;
@@ -78498,7 +78500,7 @@ function useInnerStateFallback(onExternalStateUpdate, externalState) {
   var onChangeState = function onChangeState(fn) {
     // we are setting innerState and outer  externalState as the same object "nextData"
     // it means that their links should be equal. If their links are not equal, it means externalState was altered by the user
-    // thus expandedRows should take precedence over inner expandedState that has fallen behind
+    // thus external state should take precedence over inner state that has fallen behind
     var currentState = externalState && externalState !== innerState ? externalState : innerState;
     var nextData = typeof fn === 'function' ? fn(currentState) : fn;
     var nextDecoratedData = decoratorFn(nextData);