@procore/data-table 13.3.1 → 13.3.2

package/dist/modern/index.mjs

@@ -1,6 +1,6 @@
 import React82, { useState, forwardRef, useContext, useRef, useImperativeHandle, useLayoutEffect, useCallback, useMemo, memo, useEffect, Component as Component$1, createElement } from 'react';
 import { startsWith, equals, omit, isNil, mergeDeepLeft, intersection, isEmpty, groupBy } from 'ramda';
-import { useI18nContext, Select, Input, UNSAFE_useOverlayTriggerContext, Card, Flex, Box, SegmentedController, Calendar, useDateTime, OverlayTrigger, isEventSource, DateInput, UNSAFE_isValidYearRange, DateSelect, PillSelect, TextArea, UNSAFE_useMenuImperativeControlNavigation, UNSAFE_Menu, UNSAFE_menuItemsWrapperAttribute, Spinner, FlexList, Typography, Tooltip, Button, Required, colors, Form, spacing, Switch, Panel, UNSAFE_StyledFilterTokenLabel, typographyWeights, SelectButton, StyledSelectButton, StyledSelectButtonLabel, StyledButton, StyledSelectArrow, Popover, UNSAFE_StyledSuperSelectTrigger, UNSAFE_StyledSuperSelectLabel, UNSAFE_StyledSuperSelectArrow, DateTimeProvider, useField, Label, UNSAFE_mergeRefs, ContactItem as ContactItem$1, ToggleButton, Pagination, Link, AvatarStack, Pill, Typeahead, UNSAFE_FilterToken, useI18n, I18nContext, UNSAFE_SuperSelect, Avatar, MultiSelect, H3, Checkbox, useVisibility, UNSAFE_useSuperSelectContext, DropdownFlyout, StyledDropdownFlyoutLabel, StyledDropdownFlyoutExpandIcon, EmptyState as EmptyState$1 } from '@procore/core-react';
+import { useI18nContext, Select, Input, UNSAFE_useOverlayTriggerContext, Card, Flex, Box, SegmentedController, Calendar, useDateTime, OverlayTrigger, isEventSource, DateInput, UNSAFE_isValidYearRange, DateSelect, PillSelect, TextArea, UNSAFE_useMenuImperativeControlNavigation, UNSAFE_Menu, UNSAFE_menuItemsWrapperAttribute, Spinner as Spinner$1, FlexList, Typography, Tooltip, Button, Required, colors, Form, spacing, Switch, Panel, UNSAFE_StyledFilterTokenLabel, typographyWeights, SelectButton, StyledSelectButton, StyledSelectButtonLabel, StyledButton, StyledSelectArrow, Popover, UNSAFE_StyledSuperSelectTrigger, UNSAFE_StyledSuperSelectLabel, UNSAFE_StyledSuperSelectArrow, DateTimeProvider, useField, Label, UNSAFE_mergeRefs, ContactItem as ContactItem$1, ToggleButton, Pagination, Link, AvatarStack, Pill, Typeahead, UNSAFE_FilterToken, useI18n, I18nContext, UNSAFE_SuperSelect, Avatar, MultiSelect, H3, Checkbox, useVisibility, UNSAFE_useSuperSelectContext, DropdownFlyout, StyledDropdownFlyoutLabel, StyledDropdownFlyoutExpandIcon, EmptyState as EmptyState$1 } from '@procore/core-react';
 import classnames from 'classnames/bind';
 import { Error as Error$1, Grip, Pencil, Building, Calendar as Calendar$1, CaretsInVertical, CaretsOutVertical, CaretsOutVerticalWithLine, CaretsIn, CaretsOut, CaretDown, CaretRight, ArrowDown, ArrowUp } from '@procore/core-icons';
 import classNames from 'classnames';
@@ -996,6 +996,41 @@ var rowHeightMap = {
   [rowSize.md]: "medium",
   [rowSize.lg]: "large"
 };
+var Spinner = ({
+  loading,
+  ...rest
+}) => /* @__PURE__ */ React82.createElement(Box, { "data-qa": loading ? "loading" : "loaded", display: "flex-row", flex: 1 }, /* @__PURE__ */ React82.createElement(Spinner$1, { loading, size: "md", centered: true, ...rest }));
+function isNestedColumnDefinition(Column2) {
+  return "children" in Column2;
+}
+function isNestedColDef(colDef) {
+  if (!colDef.groupId) {
+    return false;
+  }
+  return "groupId" in colDef;
+}
+function getLeafColDefs(columnDefinitions) {
+  return columnDefinitions.flatMap((colDef) => {
+    return isNestedColDef(colDef) ? getLeafColDefs(colDef.children) : [colDef];
+  });
+}
+function isFirstColumn(params) {
+  const displayedColumns = params?.columnApi?.getAllDisplayedColumns() || [];
+  return displayedColumns.length ? displayedColumns[0] === params.column : false;
+}
+function getCheckboxColumn(columnApi) {
+  const isGrouping = columnApi?.getRowGroupColumns?.()?.length !== 0;
+  if (isGrouping) {
+    return null;
+  }
+  const displayedColumns = columnApi?.getAllDisplayedColumns() || [];
+  return displayedColumns.length ? displayedColumns[0] : null;
+}
+function removeEventListenerFromGrid(event, listener, gridApi) {
+  if (!gridApi || gridApi?.destroyCalled)
+    return;
+  gridApi?.removeEventListener(event, listener);
+}
 
 // src/styles.scss
 var styles_exports = {};
@@ -53792,7 +53827,7 @@ var LocationFilterOverlay = React82.forwardRef(
           getLabel3(item)
         );
       })),
-      loading && /* @__PURE__ */ React82.createElement(UNSAFE_Menu.Footer, null, /* @__PURE__ */ React82.createElement(Flex, { justifyContent: "center" }, /* @__PURE__ */ React82.createElement(Spinner, { size: "sm" })))
+      loading && /* @__PURE__ */ React82.createElement(UNSAFE_Menu.Footer, null, /* @__PURE__ */ React82.createElement(Flex, { justifyContent: "center" }, /* @__PURE__ */ React82.createElement(Spinner$1, { size: "sm" })))
     ));
   }
 );
@@ -53995,8 +54030,16 @@ function useFilterState({
     }
     return () => {
       listenersEnabled.current = false;
-      gridApi?.removeEventListener("filterChanged", filterEventFunction);
-      gridApi?.removeEventListener("rowDataUpdated", filterEventFunction);
+      removeEventListenerFromGrid(
+        "filterChanged",
+        filterEventFunction,
+        gridApi
+      );
+      removeEventListenerFromGrid(
+        "rowDataUpdated",
+        filterEventFunction,
+        gridApi
+      );
     };
   }, [gridApi, columnDefinitions]);
   return { allAvailableFilters, changeFilterValue, filterableColumns };
@@ -54478,36 +54521,6 @@ var ServerSideFilter = ({
     )
   );
 };
-var Spinner2 = ({
-  loading,
-  ...rest
-}) => /* @__PURE__ */ React82.createElement(Box, { "data-qa": loading ? "loading" : "loaded", display: "flex-row", flex: 1 }, /* @__PURE__ */ React82.createElement(Spinner, { loading, size: "md", centered: true, ...rest }));
-function isNestedColumnDefinition(Column2) {
-  return "children" in Column2;
-}
-function isNestedColDef(colDef) {
-  if (!colDef.groupId) {
-    return false;
-  }
-  return "groupId" in colDef;
-}
-function getLeafColDefs(columnDefinitions) {
-  return columnDefinitions.flatMap((colDef) => {
-    return isNestedColDef(colDef) ? getLeafColDefs(colDef.children) : [colDef];
-  });
-}
-function isFirstColumn(params) {
-  const displayedColumns = params?.columnApi?.getAllDisplayedColumns() || [];
-  return displayedColumns.length ? displayedColumns[0] === params.column : false;
-}
-function getCheckboxColumn(columnApi) {
-  const isGrouping = columnApi?.getRowGroupColumns?.()?.length !== 0;
-  if (isGrouping) {
-    return null;
-  }
-  const displayedColumns = columnApi?.getAllDisplayedColumns() || [];
-  return displayedColumns.length ? displayedColumns[0] : null;
-}
 
 // src/utils/transformers.ts
 var emptyObject = {};
@@ -55205,7 +55218,7 @@ function ClientSideRowCheckbox(props) {
   useEffect(() => {
     props.api.addEventListener("rowSelected", handleRowSelection);
     return () => {
-      props.api.removeEventListener("rowSelected", handleRowSelection);
+      removeEventListenerFromGrid("rowSelected", handleRowSelection, props.api);
     };
   }, []);
   return /* @__PURE__ */ React82.createElement(Box, { paddingRight: "md" }, /* @__PURE__ */ React82.createElement(
@@ -55316,7 +55329,11 @@ function GroupCaret(props) {
     props.api.addEventListener("rowGroupOpened", handleRowGroupToggle);
     return () => {
       mounted.current = false;
-      props.api.removeEventListener("rowGroupOpened", handleRowGroupToggle);
+      removeEventListenerFromGrid(
+        "rowGroupOpened",
+        handleRowGroupToggle,
+        props.api
+      );
     };
   }, []);
   return /* @__PURE__ */ React82.createElement(Box, { className: AutoGroupCell_default.expandableCaret }, props.node.isExpandable() && /* @__PURE__ */ React82.createElement(
@@ -55342,7 +55359,11 @@ var AutoGroupCellRenderer = (props) => {
   useEffect(() => {
     props.api.addEventListener("cellValueChanged", handleCellValueChange);
     return () => {
-      props.api.removeEventListener("cellValueChanged", handleCellValueChange);
+      removeEventListenerFromGrid(
+        "cellValueChanged",
+        handleCellValueChange,
+        props.api
+      );
     };
   }, []);
   const adjustedLevel = props.node.footer ? props.node.level + 1 : props.node.level;
@@ -56193,7 +56214,7 @@ var GenericHeaderRenderer = (props) => {
     props.api.addEventListener("columnMoved", calculatePosition);
     calculatePosition();
     return () => {
-      props.api.removeEventListener("columnMoved", calculatePosition);
+      removeEventListenerFromGrid("columnMoved", calculatePosition, props.api);
     };
   }, []);
   const onSortToggled = (event) => {
@@ -56270,7 +56291,11 @@ var GenericHeaderRenderer = (props) => {
     };
     props.api?.addEventListener("expandOrCollapseAll", onExpandAllChange);
     return () => {
-      props.api?.removeEventListener("expandOrCollapseAll", onExpandAllChange);
+      removeEventListenerFromGrid(
+        "expandOrCollapseAll",
+        onExpandAllChange,
+        props.api
+      );
     };
   }, [props.api]);
   React82.useEffect(() => {
@@ -56361,7 +56386,11 @@ var GenericHeaderRenderer = (props) => {
     props.api.addEventListener(selectionChanged, onSelectionChanged);
     onSelectionChanged();
     return () => {
-      props.api.removeEventListener(selectionChanged, onSelectionChanged);
+      removeEventListenerFromGrid(
+        selectionChanged,
+        onSelectionChanged,
+        props.api
+      );
     };
   }, [isFirstColumn2, selectAll]);
   const { ref, isOverflowing, recalculate } = useOverflowObserver();
@@ -56370,8 +56399,8 @@ var GenericHeaderRenderer = (props) => {
     props.api.addEventListener("cellMouseOver", handler);
     props.api.addEventListener("columnResized", handler);
     return () => {
-      props.api.removeEventListener("cellMouseOver", handler);
-      props.api.removeEventListener("columnResized", handler);
+      removeEventListenerFromGrid("cellMouseOver", handler, props.api);
+      removeEventListenerFromGrid("columnResized", handler, props.api);
     };
   }, []);
   React82.useEffect(() => {
@@ -56397,8 +56426,16 @@ var GenericHeaderRenderer = (props) => {
     props.api.addEventListener(selectionChanged, updateSelectAllCheckbox);
     props.api.addEventListener(paginationChanged, updateSelectAllCheckbox);
     return () => {
-      props.api.removeEventListener(selectionChanged, updateSelectAllCheckbox);
-      props.api.removeEventListener(paginationChanged, updateSelectAllCheckbox);
+      removeEventListenerFromGrid(
+        selectionChanged,
+        updateSelectAllCheckbox,
+        props.api
+      );
+      removeEventListenerFromGrid(
+        paginationChanged,
+        updateSelectAllCheckbox,
+        props.api
+      );
     };
   }, [isFirstColumn2, props.selectionSSREnabled, props.api, onSSDR]);
   const toggleSelectAll = React82.useCallback(() => {
@@ -56606,7 +56643,7 @@ function CustomLoader(params) {
         level: params.node.level,
         lastRowInGroup: params.node === groupNodes.last
       }
-    ), /* @__PURE__ */ React82.createElement(Spinner2, { loading: true, size: "sm" }), /* @__PURE__ */ React82.createElement(Box, { paddingLeft: "sm" }, /* @__PURE__ */ React82.createElement(Typography, { color: "gray15", italic: true, intent: "small" }, I18n.t(
+    ), /* @__PURE__ */ React82.createElement(Spinner, { loading: true, size: "sm" }), /* @__PURE__ */ React82.createElement(Box, { paddingLeft: "sm" }, /* @__PURE__ */ React82.createElement(Typography, { color: "gray15", italic: true, intent: "small" }, I18n.t(
       `dataTable.loading.${params.node === groupNodes.first ? "initial" : "secondary"}`
     ))));
   }
@@ -56616,7 +56653,7 @@ var DefaultFrameworkComponents = {
   agColumnHeader: GenericHeaderRenderer,
   agLoadingCellRenderer: CustomLoader,
   agColumnGroupHeader: GenericColumnGroupHeader,
-  loadingOverlayRenderer: Spinner2,
+  loadingOverlayRenderer: Spinner,
   emptyStateRenderer: EmptyState
 };
 function getFrameworkComponents(headerMenuConfig, onSelectAll, selectionSSREnabled) {
@@ -58753,12 +58790,18 @@ var useRowSelectionState = () => {
   const { gridApi, rowSelectionRef } = useInternalTableContext();
   React82.useEffect(() => {
     function selectionEventFunction() {
-      setSelectedRows(getSelectedRows({ gridApi, rowSelectionRef }));
+      if (!gridApi?.destroyCalled) {
+        setSelectedRows(getSelectedRows({ gridApi, rowSelectionRef }));
+      }
     }
     selectionEventFunction();
     gridApi?.addEventListener("selectionChanged", selectionEventFunction);
     return () => {
-      gridApi?.removeEventListener("selectionChanged", selectionEventFunction);
+      removeEventListenerFromGrid(
+        "selectionChanged",
+        selectionEventFunction,
+        gridApi
+      );
     };
   }, [gridApi]);
   return selectedRows;
@@ -76259,7 +76302,7 @@ function personOptionRenderer(editorParams) {
   };
 }
 function LoadingField() {
-  return /* @__PURE__ */ React82.createElement(Box, { marginTop: "md" }, /* @__PURE__ */ React82.createElement(Spinner, { size: "sm", loading: true }));
+  return /* @__PURE__ */ React82.createElement(Box, { marginTop: "md" }, /* @__PURE__ */ React82.createElement(Spinner$1, { size: "sm", loading: true }));
 }
 function RadioList({
   getId: getId3,
@@ -76587,7 +76630,7 @@ var BulkEditInput = (props) => {
         );
       } else {
         console.warn(
-          `@procore/labs-data-table: Unable to find the field for ${props.editor}. Will fallback to a Form.Text.`
+          `@procore/data-table: Unable to find the field for ${props.editor}. Will fallback to a Form.Text.`
         );
         return defaultField;
       }
@@ -77048,9 +77091,10 @@ var ConfigurationColumns = React82.forwardRef(
         onDisplayedColumnsChanged
       );
       return () => {
-        gridApi?.removeEventListener(
+        removeEventListenerFromGrid(
           "displayedColumnsChanged",
-          onDisplayedColumnsChanged
+          onDisplayedColumnsChanged,
+          gridApi
         );
       };
     }, [gridApi, colDefs]);
@@ -77895,9 +77939,10 @@ var RowGroupSelector = ({ localeText }) => {
       onDisplayedColumnsChanged
     );
     return () => {
-      gridApi?.removeEventListener(
+      removeEventListenerFromGrid(
         "displayedColumnsChanged",
-        onDisplayedColumnsChanged
+        onDisplayedColumnsChanged,
+        gridApi
       );
     };
   }, [gridApi]);
@@ -81550,7 +81595,9 @@ var Table = (props) => {
         maxHeight = nodeHeight;
       }
     });
-    gridApi?.setHeaderHeight(maxHeight);
+    if (!gridApi?.destroyCalled) {
+      gridApi?.setHeaderHeight(maxHeight);
+    }
   }
   React82.useEffect(() => {
     if (gridApi && columnApi && internalTableContext.tableRef?.current) {
@@ -81561,7 +81608,11 @@ var Table = (props) => {
       gridApi.addEventListener("firstDataRendered", setHeaderHeight);
       setHeaderHeight();
       return () => {
-        gridApi.removeEventListener("firstDataRendered", setHeaderHeight);
+        removeEventListenerFromGrid(
+          "firstDataRendered",
+          setHeaderHeight,
+          gridApi
+        );
       };
     }
     return;
@@ -81793,6 +81844,8 @@ var Table = (props) => {
   const onDisplayedColumnsChanged = React82.useCallback(
     (event) => {
       if (props.onRowDragEnd) {
+        if (event.columnApi?.destroyCalled)
+          return;
         const firstVisibleColumn = event.columnApi.getAllGridColumns().find((col) => col.isVisible());
         if (!firstVisibleColumn?.getColDef().rowDrag) {
           adjustRowDragIcon(event.api, event.columnApi);
@@ -82086,7 +82139,7 @@ var Table = (props) => {
     props.UNSAFE_internalAGGridOverrides ?? {}
   );
   return /* @__PURE__ */ React82.createElement(
-    Spinner2,
+    Spinner,
     {
       loading: loadingStatus.loading,
       label: loadingStatus.message,
@@ -82253,7 +82306,11 @@ var TablePagination = ({
     }
     gridApi?.addEventListener("paginationChanged", onPaginationChanged);
     return () => {
-      gridApi?.removeEventListener("paginationChanged", onPaginationChanged);
+      removeEventListenerFromGrid(
+        "paginationChanged",
+        onPaginationChanged,
+        gridApi
+      );
     };
   }, []);
   return /* @__PURE__ */ React82.createElement(
@@ -82813,7 +82870,7 @@ var LocationQuickFilterOverlay = React82.forwardRef(
           getLabel3(item)
         );
       })),
-      loading && /* @__PURE__ */ React82.createElement(UNSAFE_Menu.Footer, null, /* @__PURE__ */ React82.createElement(Flex, { justifyContent: "center" }, /* @__PURE__ */ React82.createElement(Spinner, { size: "sm" })))
+      loading && /* @__PURE__ */ React82.createElement(UNSAFE_Menu.Footer, null, /* @__PURE__ */ React82.createElement(Flex, { justifyContent: "center" }, /* @__PURE__ */ React82.createElement(Spinner$1, { size: "sm" })))
     ));
   }
 );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@procore/data-table",
-  "version": "13.3.1",
+  "version": "13.3.2",
   "description": "Complex data grid built on top of ag-grid, with DST components and styles.",
   "main": "dist/legacy/index.js",
   "module": "dist/legacy/index.mjs",