@genesislcap/pbc-reporting-ui 15.20.1 → 15.20.2-FUI-0.3

package/dist/custom-elements.json

@@ -4395,6 +4395,48 @@
         }
       ]
     },
+    {
+      "kind": "javascript-module",
+      "path": "src/old/utils/row-selection.ts",
+      "declarations": [
+        {
+          "kind": "variable",
+          "name": "USER_TOGGLE_MARKER",
+          "type": {
+            "text": "string"
+          },
+          "default": "'__reportingUserCheckboxToggle'",
+          "description": "Set on the row node while a user checkbox click is applied. @internal"
+        },
+        {
+          "kind": "variable",
+          "name": "USER_CHECKBOX_SOURCE",
+          "type": {
+            "text": "string"
+          },
+          "default": "'checkboxSelected'",
+          "description": "Selection source reported for a click on the custom row checkbox. @internal"
+        }
+      ],
+      "exports": [
+        {
+          "kind": "js",
+          "name": "USER_TOGGLE_MARKER",
+          "declaration": {
+            "name": "USER_TOGGLE_MARKER",
+            "module": "src/old/utils/row-selection.ts"
+          }
+        },
+        {
+          "kind": "js",
+          "name": "USER_CHECKBOX_SOURCE",
+          "declaration": {
+            "name": "USER_CHECKBOX_SOURCE",
+            "module": "src/old/utils/row-selection.ts"
+          }
+        }
+      ]
+    },
     {
       "kind": "javascript-module",
       "path": "src/new/components/clone-report-config-modal/clone-report-config-modal.styles.ts",
@@ -6924,6 +6966,15 @@
                 "text": "boolean"
               }
             },
+            {
+              "kind": "field",
+              "name": "boundOnSelectionChanged",
+              "type": {
+                "text": "() => void"
+              },
+              "privacy": "private",
+              "description": "Kept so the listener registered in `init` can actually be removed in `destroy`."
+            },
             {
               "kind": "method",
               "name": "isDisabled",

package/dist/dts/new/main/edit-config/col-filters/col-filters-grid.helpers.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"col-filters-grid.helpers.d.ts","sourceRoot":"","sources":["../../../../../../src/new/main/edit-config/col-filters/col-filters-grid.helpers.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAQ3D,OAAO,EAAE,aAAa,EAAE,MAAM,0BAA0B,CAAC;AAEzD;;GAEG;AACH,wBAAgB,0BAA0B,CAAC,KAAK,EAAE,gBAAgB,CAAC,aAAa,CAAC,GAAG,IAAI,CAqBvF"}
+{"version":3,"file":"col-filters-grid.helpers.d.ts","sourceRoot":"","sources":["../../../../../../src/new/main/edit-config/col-filters/col-filters-grid.helpers.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAS3D,OAAO,EAAE,aAAa,EAAE,MAAM,0BAA0B,CAAC;AAEzD;;GAEG;AACH,wBAAgB,0BAA0B,CAAC,KAAK,EAAE,gBAAgB,CAAC,aAAa,CAAC,GAAG,IAAI,CAwBvF"}

package/dist/dts/new/main/edit-config/col-rename-alias/col-rename-alias-grid.helpers.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"col-rename-alias-grid.helpers.d.ts","sourceRoot":"","sources":["../../../../../../src/new/main/edit-config/col-rename-alias/col-rename-alias-grid.helpers.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,eAAe,EACf,SAAS,EACT,mBAAmB,EACnB,gBAAgB,EACjB,MAAM,yBAAyB,CAAC;AAGjC,OAAO,EACL,eAAe,EACf,WAAW,EAIZ,MAAM,WAAW,CAAC;AACnB,OAAO,EAAE,wBAAwB,EAAE,MAAM,+BAA+B,CAAC;AAEzE;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,wBAAwB,GAAG,wBAAwB,CAU3F;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAClC,CAAC,SAAS,IAAI,CAAC,WAAW,EAAE,eAAe,CAAC,UAAU,GAAG,eAAe,CAAC,UAAU,CAAC,EACpF,kBAAkB,EAAE,SAAS,EAAE,gBAAgB,GAAE,SAAc,IACvD,QAAQ,eAAe,CAAC,CAAC,CAAC,KAAG,SAAS,CAM/C;AAED;;GAEG;AACH,eAAO,MAAM,0BAA0B,GAAI,UAAU,MAAM,GAAG,MACpD,OAAO,MAAM,EAAE,QAAQ,mBAAmB,CAAC,wBAAwB,CAAC,SAc7E,CAAC;AAEF;;GAEG;AACH,wBAAgB,gCAAgC,CAC9C,KAAK,EAAE,gBAAgB,CAAC,wBAAwB,CAAC,GAChD,IAAI,CAiBN"}
+{"version":3,"file":"col-rename-alias-grid.helpers.d.ts","sourceRoot":"","sources":["../../../../../../src/new/main/edit-config/col-rename-alias/col-rename-alias-grid.helpers.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,eAAe,EACf,SAAS,EACT,mBAAmB,EACnB,gBAAgB,EACjB,MAAM,yBAAyB,CAAC;AAIjC,OAAO,EACL,eAAe,EACf,WAAW,EAIZ,MAAM,WAAW,CAAC;AACnB,OAAO,EAAE,wBAAwB,EAAE,MAAM,+BAA+B,CAAC;AAEzE;;GAEG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,wBAAwB,GAAG,wBAAwB,CAU3F;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAClC,CAAC,SAAS,IAAI,CAAC,WAAW,EAAE,eAAe,CAAC,UAAU,GAAG,eAAe,CAAC,UAAU,CAAC,EACpF,kBAAkB,EAAE,SAAS,EAAE,gBAAgB,GAAE,SAAc,IACvD,QAAQ,eAAe,CAAC,CAAC,CAAC,KAAG,SAAS,CAM/C;AAED;;GAEG;AACH,eAAO,MAAM,0BAA0B,GAAI,UAAU,MAAM,GAAG,MACpD,OAAO,MAAM,EAAE,QAAQ,mBAAmB,CAAC,wBAAwB,CAAC,SAc7E,CAAC;AAEF;;GAEG;AACH,wBAAgB,gCAAgC,CAC9C,KAAK,EAAE,gBAAgB,CAAC,wBAAwB,CAAC,GAChD,IAAI,CAsBN"}

package/dist/dts/old/components/renderers/row-select.renderer.d.ts

@@ -9,6 +9,8 @@ import { FoundationElement } from '@genesislcap/web-core';
 export declare class RowSelectRenderer extends FoundationElement implements ICellRendererComp {
     params: ICellRendererParams & BooleanRendererParams;
     checked: boolean;
+    /** Kept so the listener registered in `init` can actually be removed in `destroy`. */
+    private boundOnSelectionChanged;
     isDisabled(data: any): boolean;
     init(params: ICellRendererParams): void;
     onSelectionChanged(): void;

package/dist/dts/old/components/renderers/row-select.renderer.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"row-select.renderer.d.ts","sourceRoot":"","sources":["../../../../../src/old/components/renderers/row-select.renderer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAC;AACjF,OAAO,EAAE,qBAAqB,EAAuB,MAAM,uCAAuC,CAAC;AACnG,OAAO,EAAyB,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAEjF;;;;GAIG;AACH,qBAAa,iBAAkB,SAAQ,iBAAkB,YAAW,iBAAiB;IAE5E,MAAM,EAAE,mBAAmB,GAAG,qBAAqB,CAAC;IAG3D,OAAO,EAAE,OAAO,CAAC;IAEV,UAAU,CAAC,IAAI,KAAA,GAAG,OAAO;IAWzB,IAAI,CAAC,MAAM,EAAE,mBAAmB;IAShC,kBAAkB;IAMlB,aAAa,CAAC,CAAC,KAAA;IAKf,MAAM,IAAI,WAAW;IAIrB,OAAO;IAOP,OAAO,CAAC,MAAM,EAAE,mBAAmB;IAK1C,QAAQ;IAIR,mBAAmB;CAGpB;AAED;;;GAGG;AACH,eAAO,MAAM,yBAAyB,iDAsCrC,CAAC;AAEF;;;;;GAKG;AACH,eAAO,MAAM,8BAA8B,GAAI,qBAAsB,2EAYpE,CAAC;AAEF;;;;;;GAMG;AACH,eAAO,MAAM,6BAA6B;;;;;;;;4BAIxC,CAAC"}
+{"version":3,"file":"row-select.renderer.d.ts","sourceRoot":"","sources":["../../../../../src/old/components/renderers/row-select.renderer.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,mBAAmB,EAAE,MAAM,yBAAyB,CAAC;AACjF,OAAO,EAAE,qBAAqB,EAAuB,MAAM,uCAAuC,CAAC;AACnG,OAAO,EAAyB,iBAAiB,EAAE,MAAM,uBAAuB,CAAC;AAGjF;;;;GAIG;AACH,qBAAa,iBAAkB,SAAQ,iBAAkB,YAAW,iBAAiB;IAE5E,MAAM,EAAE,mBAAmB,GAAG,qBAAqB,CAAC;IAG3D,OAAO,EAAE,OAAO,CAAC;IAEjB,sFAAsF;IACtF,OAAO,CAAC,uBAAuB,CAAa;IAErC,UAAU,CAAC,IAAI,KAAA,GAAG,OAAO;IAWzB,IAAI,CAAC,MAAM,EAAE,mBAAmB;IAUhC,kBAAkB;IAMlB,aAAa,CAAC,CAAC,KAAA;IAUf,MAAM,IAAI,WAAW;IAIrB,OAAO;IAMP,OAAO,CAAC,MAAM,EAAE,mBAAmB;IAK1C,QAAQ;IAIR,mBAAmB;CAGpB;AAED;;;GAGG;AACH,eAAO,MAAM,yBAAyB,iDAsCrC,CAAC;AAEF;;;;;GAKG;AACH,eAAO,MAAM,8BAA8B,GAAI,qBAAsB,2EAYpE,CAAC;AAEF;;;;;;GAMG;AACH,eAAO,MAAM,6BAA6B;;;;;;;;4BAIxC,CAAC"}

package/dist/dts/old/utils/row-selection.d.ts

@@ -0,0 +1,31 @@
+import { IRowNode, RowSelectedEvent } from '@ag-grid-community/core';
+/** Set on the row node while a user checkbox click is applied. @internal */
+export declare const USER_TOGGLE_MARKER = "__reportingUserCheckboxToggle";
+/** Selection source reported for a click on the custom row checkbox. @internal */
+export declare const USER_CHECKBOX_SOURCE = "checkboxSelected";
+/**
+ * True while a user checkbox click is being applied through the node marker fallback.
+ * @internal
+ */
+export declare function isUserCheckboxToggle(node: unknown): boolean;
+/**
+ * True when a selection change came from the user rather than from a programmatic
+ * `setSelected()`, which defaults to source `'api'`.
+ * @internal
+ */
+export declare function isUserSelectionEvent(event: RowSelectedEvent): boolean;
+/**
+ * Apply a user checkbox toggle so the resulting `rowSelected` event stays distinguishable from a
+ * programmatic re-sync.
+ *
+ * @remarks
+ * Grid-option handlers such as `onRowSelected` are invoked asynchronously: the event is queued
+ * through a `setTimeout`, because `rowSelected` is not one of the grid's always-synchronous
+ * events. A marker set on the node and cleared in a `finally` block is therefore already gone
+ * before the handler reads it. The selection source travels on the event itself, so pass it
+ * explicitly. `setNodesSelected` takes a single object argument, so the source keeps its position
+ * across grid versions, unlike the positional `source` on `RowNode.setSelected`.
+ * @internal
+ */
+export declare function selectNodeAsUser(node: IRowNode, newValue: boolean, api?: unknown): void;
+//# sourceMappingURL=row-selection.d.ts.map

package/dist/dts/old/utils/row-selection.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"row-selection.d.ts","sourceRoot":"","sources":["../../../../src/old/utils/row-selection.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,gBAAgB,EAAE,MAAM,yBAAyB,CAAC;AAErE,4EAA4E;AAC5E,eAAO,MAAM,kBAAkB,kCAAkC,CAAC;AAElE,kFAAkF;AAClF,eAAO,MAAM,oBAAoB,qBAAqB,CAAC;AAkBvD;;;GAGG;AACH,wBAAgB,oBAAoB,CAAC,IAAI,EAAE,OAAO,GAAG,OAAO,CAM3D;AAED;;;;GAIG;AACH,wBAAgB,oBAAoB,CAAC,KAAK,EAAE,gBAAgB,GAAG,OAAO,CAErE;AAED;;;;;;;;;;;;GAYG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,QAAQ,EAAE,QAAQ,EAAE,OAAO,EAAE,GAAG,CAAC,EAAE,OAAO,GAAG,IAAI,CAevF"}

package/dist/esm/new/main/edit-config/col-filters/col-filters-grid.helpers.js

@@ -1,3 +1,4 @@
+import { isUserSelectionEvent } from '../../../../old/utils/row-selection';
 import { actions, selectors } from '../../../store';
 import { GridColumnNames, datasourceNameFromDisplay, lookupColumnIsIncluded, shouldDisableColumn, } from '../shared';
 /**
@@ -7,6 +8,9 @@ export function rowSelectionChangedHandler(event) {
     var _a, _b;
     if (!event.data)
         return;
+    // Ignore programmatic re-selection (`setSelected()` defaults to source `'api'`).
+    if (!isUserSelectionEvent(event))
+        return;
     // gate event listener to only trigger if the data is actually changed by the user
     // (return if setting the same value as in the store, e.g. when first loading
     // the grid data)

package/dist/esm/new/main/edit-config/col-rename-alias/col-rename-alias-grid.helpers.js

@@ -1,4 +1,5 @@
 import { capitalCase } from 'change-case';
+import { isUserSelectionEvent } from '../../../../old/utils/row-selection';
 import { actions, selectors } from '../../../store';
 import { GridColumnNames, datasourceNameFromDisplay, lookupColumnIsRenamed, shouldDisableColumn, } from '../shared';
 /**
@@ -48,6 +49,11 @@ export function renameRowSelectionChangedHandler(event) {
     var _a, _b;
     if (!event.data)
         return;
+    // Ignore programmatic re-selection: `onRowDataUpdated` re-syncs through a bare `setSelected()`
+    // (source `'api'`), and AG Grid re-applies its own selection when the stepper step is revisited.
+    // Neither is a user edit, so neither may write back to `COLUMN_RENAMES`.
+    if (!isUserSelectionEvent(event))
+        return;
     // gate event listener to only trigger if the data is actually changed by the user
     // (return if setting the same value as in the store, e.g. when first loading
     // the grid data)

package/dist/esm/old/components/renderers/row-select.renderer.js

@@ -1,6 +1,7 @@
 import { __decorate } from "tslib";
 import { EDITED_COLUMN_FIELD } from '@genesislcap/foundation-zero-grid-pro';
 import { css, html, observable, FoundationElement } from '@genesislcap/web-core';
+import { selectNodeAsUser } from '../../utils/row-selection';
 /**
  * The AG Row Select Renderer element.
  * @public
@@ -26,7 +27,8 @@ export class RowSelectRenderer extends FoundationElement {
         this.checked = !!params.value;
         const { ariaLabelField } = params.context;
         this.ariaLabel = `Select row for ${params.data[ariaLabelField]}. Press Space to toggle selection.`;
-        this.params.api.addEventListener('selectionChanged', this.onSelectionChanged.bind(this));
+        this.boundOnSelectionChanged = this.onSelectionChanged.bind(this);
+        this.params.api.addEventListener('selectionChanged', this.boundOnSelectionChanged);
     }
     onSelectionChanged() {
         var _a;
@@ -35,16 +37,23 @@ export class RowSelectRenderer extends FoundationElement {
         }
     }
     changeHandler(e) {
+        var _a, _b, _c;
         this.checked = e.target.checked;
-        this.params.node.setSelected(this.checked);
+        const node = (_a = this.params) === null || _a === void 0 ? void 0 : _a.node;
+        if (!node)
+            return;
+        // FAST re-emits `change` for programmatic `checked` writes, so only treat this as a user
+        // toggle when the checkbox and the node actually disagree.
+        if (this.checked === ((_b = node.isSelected()) !== null && _b !== void 0 ? _b : false))
+            return;
+        selectNodeAsUser(node, this.checked, (_c = this.params) === null || _c === void 0 ? void 0 : _c.api);
     }
     getGui() {
         return this;
     }
     destroy() {
         if (this.params && this.params.api) {
-            // oxlint-disable-next-line unicorn/no-invalid-remove-event-listener -- symmetric to addEventListener(.bind)
-            this.params.api.removeEventListener('selectionChanged', this.onSelectionChanged.bind(this));
+            this.params.api.removeEventListener('selectionChanged', this.boundOnSelectionChanged);
         }
     }
     refresh(params) {

package/dist/esm/old/utils/row-selection.js

@@ -0,0 +1,64 @@
+/** Set on the row node while a user checkbox click is applied. @internal */
+export const USER_TOGGLE_MARKER = '__reportingUserCheckboxToggle';
+/** Selection source reported for a click on the custom row checkbox. @internal */
+export const USER_CHECKBOX_SOURCE = 'checkboxSelected';
+/**
+ * AG Grid's own UI selection sources, plus the source this module reports for the custom row
+ * checkbox. Programmatic sources (`api`, `apiSelectAll`) are deliberately absent.
+ * @internal
+ */
+const USER_SELECTION_SOURCES = new Set([
+    'uiSelectAll',
+    'uiSelectAllFiltered',
+    'uiSelectAllCurrentPage',
+    'checkboxSelected',
+    'rowClicked',
+    'spaceKey',
+    'spacePressed', // v29 name for 'spaceKey'
+    'keyboardSelectAll', // v36 Ctrl+A; absent from v29's source union
+]);
+/**
+ * True while a user checkbox click is being applied through the node marker fallback.
+ * @internal
+ */
+export function isUserCheckboxToggle(node) {
+    return (typeof node === 'object' &&
+        node !== null &&
+        node[USER_TOGGLE_MARKER] === true);
+}
+/**
+ * True when a selection change came from the user rather than from a programmatic
+ * `setSelected()`, which defaults to source `'api'`.
+ * @internal
+ */
+export function isUserSelectionEvent(event) {
+    return USER_SELECTION_SOURCES.has(event.source) || isUserCheckboxToggle(event.node);
+}
+/**
+ * Apply a user checkbox toggle so the resulting `rowSelected` event stays distinguishable from a
+ * programmatic re-sync.
+ *
+ * @remarks
+ * Grid-option handlers such as `onRowSelected` are invoked asynchronously: the event is queued
+ * through a `setTimeout`, because `rowSelected` is not one of the grid's always-synchronous
+ * events. A marker set on the node and cleared in a `finally` block is therefore already gone
+ * before the handler reads it. The selection source travels on the event itself, so pass it
+ * explicitly. `setNodesSelected` takes a single object argument, so the source keeps its position
+ * across grid versions, unlike the positional `source` on `RowNode.setSelected`.
+ * @internal
+ */
+export function selectNodeAsUser(node, newValue, api) {
+    const setNodesSelected = api === null || api === void 0 ? void 0 : api.setNodesSelected;
+    if (typeof setNodesSelected === 'function') {
+        setNodesSelected.call(api, { nodes: [node], newValue, source: USER_CHECKBOX_SOURCE });
+        return;
+    }
+    // Grids without `setNodesSelected`: mark the node and clear the mark on a later task, so the
+    // marker outlives an asynchronous dispatch of the selection event.
+    const marked = node;
+    marked[USER_TOGGLE_MARKER] = true;
+    node.setSelected(newValue);
+    setTimeout(() => {
+        delete marked[USER_TOGGLE_MARKER];
+    }, 0);
+}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@genesislcap/pbc-reporting-ui",
   "description": "Genesis PBC Reporting UI",
-  "version": "15.20.1",
+  "version": "15.20.2-FUI-0.3",
   "license": "SEE LICENSE IN license.txt",
   "main": "dist/esm/index.js",
   "types": "dist/dts/index.d.ts",
@@ -86,32 +86,32 @@
     "@ag-grid-community/core": "29.2.0"
   },
   "dependencies": {
-    "@genesislcap/foundation-comms": "15.20.1",
-    "@genesislcap/foundation-entity-management": "15.20.1",
-    "@genesislcap/foundation-events": "15.20.1",
-    "@genesislcap/foundation-forms": "15.20.1",
-    "@genesislcap/foundation-logger": "15.20.1",
-    "@genesislcap/foundation-notifications": "15.20.1",
-    "@genesislcap/foundation-ui": "15.20.1",
-    "@genesislcap/foundation-user": "15.20.1",
-    "@genesislcap/foundation-utils": "15.20.1",
-    "@genesislcap/foundation-zero": "15.20.1",
-    "@genesislcap/foundation-zero-grid-pro": "15.20.1",
-    "@genesislcap/grid-pro": "15.20.1",
-    "@genesislcap/rapid-design-system": "15.20.1",
-    "@genesislcap/rapid-grid-pro": "15.20.1",
-    "@genesislcap/web-core": "15.20.1",
+    "@genesislcap/foundation-comms": "15.20.2-FUI-0.3",
+    "@genesislcap/foundation-entity-management": "15.20.2-FUI-0.3",
+    "@genesislcap/foundation-events": "15.20.2-FUI-0.3",
+    "@genesislcap/foundation-forms": "15.20.2-FUI-0.3",
+    "@genesislcap/foundation-logger": "15.20.2-FUI-0.3",
+    "@genesislcap/foundation-notifications": "15.20.2-FUI-0.3",
+    "@genesislcap/foundation-ui": "15.20.2-FUI-0.3",
+    "@genesislcap/foundation-user": "15.20.2-FUI-0.3",
+    "@genesislcap/foundation-utils": "15.20.2-FUI-0.3",
+    "@genesislcap/foundation-zero": "15.20.2-FUI-0.3",
+    "@genesislcap/foundation-zero-grid-pro": "15.20.2-FUI-0.3",
+    "@genesislcap/grid-pro": "15.20.2-FUI-0.3",
+    "@genesislcap/rapid-design-system": "15.20.2-FUI-0.3",
+    "@genesislcap/rapid-grid-pro": "15.20.2-FUI-0.3",
+    "@genesislcap/web-core": "15.20.2-FUI-0.3",
     "@reduxjs/toolkit": "^2.2.5",
     "@types/node": "^22.10.2",
     "change-case": "^5.4.3",
     "lodash": "^4.17.21"
   },
   "devDependencies": {
-    "@genesislcap/eslint-config": "15.20.1",
-    "@genesislcap/foundation-testing": "15.20.1",
-    "@genesislcap/genx": "15.20.1",
-    "@genesislcap/prettier-config": "15.20.1",
-    "@genesislcap/stylelint-config": "15.20.1",
+    "@genesislcap/eslint-config": "15.20.2-FUI-0.3",
+    "@genesislcap/foundation-testing": "15.20.2-FUI-0.3",
+    "@genesislcap/genx": "15.20.2-FUI-0.3",
+    "@genesislcap/prettier-config": "15.20.2-FUI-0.3",
+    "@genesislcap/stylelint-config": "15.20.2-FUI-0.3",
     "@types/json-schema": "^7.0.11",
     "playwright-lighthouse": "^4.0.0",
     "rimraf": "^5.0.0"