@homebound/beam 2.302.0 → 2.302.2

package/dist/components/Table/GridTable.js

@@ -232,7 +232,9 @@ function GridTable(props) {
     if (tooManyClientSideRows) {
         visibleDataRows = visibleDataRows.slice(0, filterMaxRows + keptSelectedRows.length);
     }
-    tableState.setMatchedRows(filteredRowIds);
+    (0, react_1.useEffect)(() => {
+        tableState.setMatchedRows(filteredRowIds);
+    }, [tableState, filteredRowIds]);
     // Push back to the caller a way to ask us where a row is.
     const { rowLookup } = props;
     if (rowLookup) {

package/dist/components/Table/utils/RowState.d.ts

@@ -48,4 +48,16 @@ export declare class RowState {
     get isKept(): boolean;
     private get inferSelectedState();
     private get visibleChildren();
+    /**
+     * Returns whether this row should act like a parent.
+     *
+     * This means "has children" and "does not have inferSelectedState: false"
+     * set. I.e. `inferSelectedState: false` allows a parent with children to
+     * still act as its own selectable identity.
+     *
+     * We also check `children.length > 0`, because sometimes pages will calc a
+     * row's children as `children = someList.map(...)`, and if the list is empty,
+     * they want the row to be selectable.
+     */
+    private get isParent();
 }

package/dist/components/Table/utils/RowState.js

@@ -37,7 +37,7 @@ class RowState {
      */
     get isSelected() {
         // We consider group rows selected if all of their children are selected.
-        if (this.children && this.inferSelectedState)
+        if (this.isParent)
             return this.selectedState === "checked";
         return this.selected;
     }
@@ -46,7 +46,7 @@ class RowState {
         // Parent `selectedState` is special b/c it does not directly depend on the parent's own selected-ness,
         // but instead depends on the current visible children. I.e. a parent might be "selected", but then the
         // client-side filter changes, a child reappears, and we need to transition to partial-ness.
-        if (this.children && this.inferSelectedState) {
+        if (this.isParent) {
             // Use visibleChildren b/c if filters are hiding some of our children, we still want to show fully selected
             const children = this.visibleChildren;
             const allChecked = children.every((child) => child.selectedState === "checked");
@@ -107,8 +107,7 @@ class RowState {
         return (this.selected &&
             // Headers, totals, etc., do not need keeping
             !src_1.reservedRowKinds.includes(this.row.kind) &&
-            // Parents don't need keeping, unless they're actually real rows
-            !(this.children && this.inferSelectedState) &&
+            !this.isParent &&
             (!this.isMatched || this.wasRemoved));
     }
     get inferSelectedState() {
@@ -121,6 +120,20 @@ class RowState {
             return (_a = this.children) !== null && _a !== void 0 ? _a : [];
         return (_c = (_b = this.children) === null || _b === void 0 ? void 0 : _b.filter((c) => c.isMatched === true)) !== null && _c !== void 0 ? _c : [];
     }
+    /**
+     * Returns whether this row should act like a parent.
+     *
+     * This means "has children" and "does not have inferSelectedState: false"
+     * set. I.e. `inferSelectedState: false` allows a parent with children to
+     * still act as its own selectable identity.
+     *
+     * We also check `children.length > 0`, because sometimes pages will calc a
+     * row's children as `children = someList.map(...)`, and if the list is empty,
+     * they want the row to be selectable.
+     */
+    get isParent() {
+        return !!this.children && this.children.length > 0 && this.inferSelectedState;
+    }
     /** Pretty toString. */
     [Symbol.for("nodejs.util.inspect.custom")]() {
         return `RowState ${this.row.kind}-${this.row.id}`;

package/dist/components/Table/utils/TableState.d.ts

@@ -22,7 +22,7 @@ export type SelectedState = "checked" | "unchecked" | "partial";
 export declare class TableState {
     private readonly collapsedRows;
     private persistCollapse;
-    rows: GridDataRow<any>[];
+    private rows;
     private readonly rowStates;
     activeRowId: string | undefined;
     activeCellId: string | undefined;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@homebound/beam",
-  "version": "2.302.0",
+  "version": "2.302.2",
   "author": "Homebound",
   "license": "MIT",
   "main": "dist/index.js",