eru-grid 0.0.53 → 0.0.54

package/fesm2022/eru-grid.mjs

@@ -1288,7 +1288,10 @@ class PivotTransformService {
         const enableColumnSubtotals = gridConfiguration?.config?.enableColumnSubtotals ?? false;
         const enableGrandTotal = gridConfiguration?.config?.enableGrandTotal ?? false;
         const enableColumnGrandTotal = gridConfiguration?.config?.enableColumnGrandTotal ?? false;
-        if (enableRowSubtotals || enableColumnSubtotals || enableGrandTotal || enableColumnGrandTotal) {
+        // Nothing to total when there are no rows — a grand total of an empty set
+        // would render a row of zeros under a result that has no data.
+        if (pivotRows.length > 0 &&
+            (enableRowSubtotals || enableColumnSubtotals || enableGrandTotal || enableColumnGrandTotal)) {
             pivotRows = this.addSubtotals(pivotRows, configuration, pivotColumns, gridConfiguration, sourceData);
         }
         // Step 5: Generate column definitions (including subtotal columns)
@@ -3896,6 +3899,11 @@ class EruGridStore {
     // Pivot-specific signals
     _pivotConfiguration = signal(null, ...(ngDevMode ? [{ debugName: "_pivotConfiguration" }] : []));
     _pivotResult = signal(null, ...(ngDevMode ? [{ debugName: "_pivotResult" }] : []));
+    /**
+     * Widths for generated pivot leaves that have no field to hold them.
+     * Keyed by leaf name — see updateColumnWidth.
+     */
+    _pivotLeafWidths = signal(new Map(), ...(ngDevMode ? [{ debugName: "_pivotLeafWidths" }] : []));
     _drilldown = signal(null, ...(ngDevMode ? [{ debugName: "_drilldown" }] : []));
     _actionClick = signal(null, ...(ngDevMode ? [{ debugName: "_actionClick" }] : []));
     _columnResize = signal(null, ...(ngDevMode ? [{ debugName: "_columnResize" }] : []));
@@ -3985,6 +3993,7 @@ class EruGridStore {
     // Pivot-specific readonly signals
     pivotConfiguration = this._pivotConfiguration.asReadonly();
     pivotResult = this._pivotResult.asReadonly();
+    pivotLeafWidths = this._pivotLeafWidths.asReadonly();
     drilldown = this._drilldown.asReadonly();
     actionClick = this._actionClick.asReadonly();
     columnResize = this._columnResize.asReadonly();
@@ -4401,8 +4410,20 @@ class EruGridStore {
             });
             return;
         }
-        const columns = this.columns().map(col => col.name === owner ? { ...col, field_size: field_size } : col);
-        this._columns.set(columns);
+        const columns = this.columns();
+        if (!columns.some(col => col.name === owner)) {
+            // A pivot dimension the grid has no column for — the author put a field
+            // in the pivot's rows that is not in the column set. There is no owner to
+            // hold the width, so it is held against the leaf name. Without this the
+            // resize reached the cells (the directive styles them directly) but
+            // nothing else: the next change detection re-asserted the old width, and
+            // the table total, which is summed from the leaves, never moved.
+            const next = new Map(this._pivotLeafWidths());
+            next.set(owner, field_size);
+            this._pivotLeafWidths.set(next);
+            return;
+        }
+        this._columns.set(columns.map(col => (col.name === owner ? { ...col, field_size: field_size } : col)));
     }
     setCellValueChange(change) {
         this._cellValueChange.set(change);
@@ -4721,16 +4742,16 @@ class EruGridStore {
         }
         if (!pivotConfig)
             return;
-        // No source rows: publish an EMPTY pivot rather than returning and leaving
-        // the previous result on screen. Bailing here is what kept the last
-        // filter's rows — and their subtotals — rendered after a query came back
-        // with nothing, since the pivot view reads _pivotResult and not the row
-        // stores that had already been cleared.
-        if (sourceData.length === 0) {
-            if (this._pivotResult() !== null)
-                this._pivotResult.set(null);
-            return;
-        }
+        // No source rows still goes through the transform, producing a pivot with
+        // the row-dimension columns and no data rows.
+        //
+        // Two things this must NOT do. Bailing out leaves the previous filter's
+        // rows and subtotals on screen, since the pivot view reads _pivotResult and
+        // not the row stores that were already cleared. Publishing null empties it
+        // but drops the pivot's own column definitions, so displayColumns() falls
+        // back to the raw source columns and an empty pivot reads as a table. The
+        // transform on an empty set yields neither: column values and measures
+        // cannot be derived from nothing, so only the row dimensions remain.
         this.setLoading(true);
         this.setError(null);
         try {
@@ -14584,7 +14605,6 @@ class EruGridComponent {
         return !!row && this.gridStore.activeBoardRow() === row;
     }
     initialMinHeight = 400;
-    initialTotalWidth = 0;
     viewport;
     groupsViewport;
     groupsScrollContainerEl;
@@ -14889,10 +14909,21 @@ class EruGridComponent {
         this.viewport?.getRenderedRange();
         this.firstTr = this.viewport?.getRenderedRange()?.start || 0;
     }
+    /**
+     * The sum of the rendered column widths — what every table in the grid is
+     * forced to via `--table-total-width`.
+     *
+     * A signal, not a value captured at init. A resize writes the new width to
+     * the column (or, in pivot mode, to the measure the leaf came from), and the
+     * cells and `<col>`s follow it; a cached total does not, so the table stayed
+     * at its old width while its columns changed underneath — leaving the header
+     * and body tables wider or narrower than the columns they contain, and the
+     * horizontal scroll range wrong. Pivot mode also changes its leaf SET as the
+     * data changes, so even without a resize the total goes stale.
+     */
+    totalTableWidth = computed(() => this.getLeafColumns().reduce((total, column) => total + (column?.field_size || 0), 0), ...(ngDevMode ? [{ debugName: "totalTableWidth" }] : []));
     getTotalTableWidth() {
-        const columns = this.getLeafColumns();
-        let totalWidth = columns.reduce((total, column) => total + column.field_size, 0);
-        return totalWidth;
+        return this.totalTableWidth();
     }
     /* getContainerHeight(): number {
       const element = this.rowContainer?.nativeElement;
@@ -15120,7 +15151,6 @@ class EruGridComponent {
                 this.initializeColumnWidths();
                 this.firstDataRowIndex.set(this.maxDepth() + 1);
                 this.initialMinHeight = this.getMinHeightPx();
-                this.initialTotalWidth = this.getTotalTableWidth();
                 this.columnsInitialized.set(true);
             }
         });
@@ -15322,7 +15352,6 @@ class EruGridComponent {
         // initializeGroups() moved to effect in constructor to handle timing
         this.firstDataRowIndex.set(this.maxDepth() + 1);
         this.initialMinHeight = this.getMinHeightPx();
-        this.initialTotalWidth = this.getTotalTableWidth();
     }
     ngAfterViewInit() {
         this.applyTokens();
@@ -15419,12 +15448,12 @@ class EruGridComponent {
         return this.initialMinHeight;
     }
     getInitialTotalWidth() {
-        return this.initialTotalWidth;
+        return this.totalTableWidth();
     }
     applyCdkWidth() {
         const element = this.rowContainer?.nativeElement;
         if (element) {
-            return element.clientWidth > this.initialTotalWidth;
+            return element.clientWidth > this.totalTableWidth();
         }
         return false;
     }
@@ -15828,6 +15857,7 @@ class EruGridComponent {
         const leaves = pivotResult.headerStructure.leafColumns;
         const aggregations = this.gridStore.configuration()?.pivot?.aggregations || [];
         const sourceColumns = this.gridStore.columns() || [];
+        const leafWidths = this.gridStore.pivotLeafWidths();
         // Carry configuration from the field a leaf came from onto the leaf itself.
         // The pivot transform builds leaves with a handful of hard-coded keys — it
         // types every aggregated column as plain 'number' whatever the measure says,
@@ -15841,15 +15871,20 @@ class EruGridComponent {
             const owner = leaf.aggregationFunction
                 ? aggregations.find(a => a.name === this.designTargetFor(leaf))
                 : sourceColumns.find(c => c.name === leaf.name);
-            if (!owner)
-                return leaf;
             const patch = {};
-            for (const key of EruGridComponent.PIVOT_INHERITED_KEYS) {
-                const value = owner[key];
-                // `false` is meaningful (enableDrilldown), blank/absent is not.
-                if (value !== undefined && value !== null && value !== '')
-                    patch[key] = value;
+            if (owner) {
+                for (const key of EruGridComponent.PIVOT_INHERITED_KEYS) {
+                    const value = owner[key];
+                    // `false` is meaningful (enableDrilldown), blank/absent is not.
+                    if (value !== undefined && value !== null && value !== '')
+                        patch[key] = value;
+                }
             }
+            // A resize of a leaf with no owning field is held against the leaf name
+            // instead — see EruGridStore.updateColumnWidth.
+            const width = leafWidths.get(leaf.name);
+            if (typeof width === 'number' && width > 0)
+                patch.field_size = width;
             return Object.keys(patch).length ? { ...leaf, ...patch } : leaf;
         });
     }