@homebound/beam 2.313.0 → 2.313.2

package/dist/components/Table/GridTableApi.d.ts

@@ -68,8 +68,13 @@ export declare class GridTableApiImpl<R extends Kinded> implements GridTableApi<
     init(persistCollapse: string | undefined, virtuosoRef: MutableRefObject<VirtuosoHandle | null>): void;
     scrollToIndex(index: number): void;
     getSelectedRowIds(kind?: string): string[];
+    private getSelectedRowIdsImpl;
     getSelectedRows(kind?: string): any;
+    private getSelectedRowsImpl;
     getVisibleRows(kind?: string): any;
+    private getVisibleRowsImpl;
+    getVisibleRowIds(kind?: string): any;
+    private getVisibleRowIdsImpl;
     clearSelections(id?: string): void;
     setActiveRowId(id: string | undefined): void;
     setActiveCellId(id: string | undefined): void;

package/dist/components/Table/GridTableApi.js

@@ -1,6 +1,8 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.GridTableApiImpl = exports.useGridTableApi = void 0;
+const mobx_1 = require("mobx");
+const mobx_utils_1 = require("mobx-utils");
 const react_1 = require("react");
 const index_1 = require("../index");
 const TableState_1 = require("./utils/TableState");
@@ -31,6 +33,12 @@ class GridTableApiImpl {
         this.virtuosoRef = { current: null };
         // This instance gets spread into each row's GridRowApi, so bind the methods up-front
         bindMethods(this);
+        // Memoize these so that if the user is creating new `data` instances on every render, they
+        // can use `getSelectedRowIds` to observer a stable list of `[pi:1, pi:2]`, etc.
+        this.getVisibleRowsImpl = (0, mobx_utils_1.computedFn)(this.getVisibleRowsImpl, { equals: mobx_1.comparer.shallow });
+        this.getVisibleRowIdsImpl = (0, mobx_utils_1.computedFn)(this.getVisibleRowIdsImpl, { equals: mobx_1.comparer.shallow });
+        this.getSelectedRowsImpl = (0, mobx_utils_1.computedFn)(this.getSelectedRowsImpl, { equals: mobx_1.comparer.shallow });
+        this.getSelectedRowIdsImpl = (0, mobx_utils_1.computedFn)(this.getSelectedRowIdsImpl, { equals: mobx_1.comparer.shallow });
     }
     /** Called once by the GridTable when it takes ownership of this api instance. */
     init(persistCollapse, virtuosoRef) {
@@ -44,15 +52,33 @@ class GridTableApiImpl {
         this.virtuosoRef.current && this.virtuosoRef.current.scrollToIndex(index);
     }
     getSelectedRowIds(kind) {
-        return this.getSelectedRows(kind).map((row) => row.id);
+        return this.getSelectedRowIdsImpl(kind !== null && kind !== void 0 ? kind : undefined);
+    }
+    // impl with required param for computedFn
+    getSelectedRowIdsImpl(kind) {
+        return this.tableState.selectedRows.filter((rs) => !kind || rs.kind === kind).map((rs) => rs.row.id);
     }
-    // The `any` is not great, but getting the overload to handle the optional kind is annoying
     getSelectedRows(kind) {
-        return this.tableState.selectedRows.filter((row) => !kind || row.kind === kind);
+        return this.getSelectedRowsImpl(kind !== null && kind !== void 0 ? kind : undefined);
+    }
+    // impl with required param for computedFn
+    getSelectedRowsImpl(kind) {
+        return this.tableState.selectedRows.filter((rs) => !kind || rs.kind === kind).map((rs) => rs.row);
     }
     // The `any` is not great, but getting the overload to handle the optional kind is annoying
     getVisibleRows(kind) {
-        return this.tableState.visibleRows.filter((row) => !kind || row.kind === kind);
+        return this.getVisibleRowsImpl(kind !== null && kind !== void 0 ? kind : undefined);
+    }
+    // impl with required param for computedFn
+    getVisibleRowsImpl(kind) {
+        return this.tableState.visibleRows.filter((row) => !kind || row.kind === kind).map((rs) => rs.row);
+    }
+    getVisibleRowIds(kind) {
+        return this.getVisibleRowIdsImpl(kind !== null && kind !== void 0 ? kind : undefined);
+    }
+    // impl with required param for computedFn
+    getVisibleRowIdsImpl(kind) {
+        return this.tableState.visibleRows.filter((row) => !kind || row.kind === kind).map((rs) => rs.row.id);
     }
     clearSelections(id) {
         this.tableState.selectRow("header", false);

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

@@ -65,7 +65,7 @@ export declare class TableState<R extends Kinded> {
      * Returns selected data rows (non-header, non-totals, etc.), ignoring rows that
      * have `row.selectable !== false`.
      */
-    get selectedRows(): GridDataRow<R>[];
+    get selectedRows(): RowState<R>[];
     /** Returns kept group row, with the latest kept children, if any. */
     get keptRowGroup(): GridDataRow<R>;
     /** Returns kept rows, i.e. those that were user-selected but then client-side or server-side filtered. */

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

@@ -179,9 +179,7 @@ class TableState {
      * have `row.selectable !== false`.
      */
     get selectedRows() {
-        return this.rowStates.allStates
-            .filter((rs) => rs.isSelected && !utils_1.reservedRowKinds.includes(rs.row.kind))
-            .map((rs) => rs.row);
+        return this.rowStates.allStates.filter((rs) => rs.isSelected && !utils_1.reservedRowKinds.includes(rs.row.kind));
     }
     /** Returns kept group row, with the latest kept children, if any. */
     get keptRowGroup() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@homebound/beam",
-  "version": "2.313.0",
+  "version": "2.313.2",
   "author": "Homebound",
   "license": "MIT",
   "main": "dist/index.js",
@@ -116,6 +116,7 @@
     "jest-watch-typeahead": "^2.2.2",
     "mobx": "^6.8.0",
     "mobx-react": "^7.6.0",
+    "mobx-utils": "^6.0.8",
     "prettier": "^2.8.4",
     "prettier-plugin-organize-imports": "^3.2.2",
     "react": "^18.2.0",