@skyux/ag-grid 4.12.0 → 4.12.4

package/CHANGELOG.md

@@ -1,3 +1,20 @@
+# 4.12.4 (2022-01-04)
+
+- Fixed an issue where the first row would not render. [#182](https://github.com/blackbaud/skyux-ag-grid/pull/182)
+
+# 4.12.3 (2022-01-03)
+
+- Added support for wrapped text in custom components. [#177](https://github.com/blackbaud/skyux-ag-grid/pull/177)
+- Fixed an error with the validator popover change detection. [#180](https://github.com/blackbaud/skyux-ag-grid/pull/180)
+
+# 4.12.2 (2021-12-02)
+
+- Added bug fixes and features from the `master` branch. [#173](https://github.com/blackbaud/skyux-ag-grid/pull/173)
+
+# 4.12.1 (2021-11-10)
+
+- Added bug fixes and features from the `master` branch. [#165](https://github.com/blackbaud/skyux-ag-grid/pull/165)
+
 # 4.12.0 (2021-11-04)
 
 - Added bug fixes and features from the `master` branch. [#161](https://github.com/blackbaud/skyux-ag-grid/pull/161)

package/bundles/skyux-ag-grid.umd.js

@@ -885,6 +885,7 @@
             return fallback;
         };
     }
+    var rowNodeId = -1;
     /**
      * A service that provides default styling and behavior for agGrids in SKY UX SPAs.
      */
@@ -1036,6 +1037,20 @@
                     'sky-ag-grid-cell-renderer-currency-validator': SkyAgGridCellRendererCurrencyValidatorComponent,
                     'sky-ag-grid-cell-renderer-validator-tooltip': SkyAgGridCellRendererValidatorTooltipComponent
                 },
+                getRowNodeId: function (data) {
+                    if ('id' in data && data.id !== undefined) {
+                        return "" + data.id;
+                    }
+                    return "" + rowNodeId--;
+                },
+                getRowClass: function (params) {
+                    if (params.node.id) {
+                        return "sky-ag-grid-row-" + params.node.id;
+                    }
+                    else {
+                        return undefined;
+                    }
+                },
                 headerHeight: ((_p = (_o = this.currentTheme) === null || _o === void 0 ? void 0 : _o.theme) === null || _p === void 0 ? void 0 : _p.name) === 'modern' ? 60 : 37,
                 icons: {
                     sortDescending: this.getIconTemplate('caret-down'),
@@ -1488,13 +1503,30 @@
                 agGrid.api.sizeColumnsToFit();
             });
             agGrid.columnMoved
+                .pipe(operators.takeUntil(this.ngUnsubscribe), operators.filter(function (event) {
+                return ![
+                    'gridInitializing',
+                    'uiColumnResized',
+                    'uiColumnDragged',
+                    'api'
+                ].includes(event.source);
+            }))
+                .subscribe(function (value) {
+                _this.updateColumnsInCurrentDataState(value.columnApi);
+            });
+            var currentColumns;
+            agGrid.dragStarted
                 .pipe(operators.takeUntil(this.ngUnsubscribe))
-                .subscribe(function (event) {
-                var columnOrder = agGrid.columnApi.getAllDisplayedVirtualColumns().map(function (col) { return col.getColDef().colId; });
-                if (event.source !== 'api') {
-                    var viewState = _this.currentDataState.getViewStateById(_this.viewConfig.id);
-                    viewState.displayedColumnIds = columnOrder;
-                    _this.dataManagerSvc.updateDataState(_this.currentDataState.addOrUpdateView(_this.viewConfig.id, viewState), _this.viewConfig.id);
+                .subscribe(function (value) {
+                currentColumns = _this.getColumnOrder(value.columnApi);
+            });
+            agGrid.dragStopped
+                .pipe(operators.takeUntil(this.ngUnsubscribe))
+                .subscribe(function (value) {
+                var newColumns = _this.getColumnOrder(value.columnApi);
+                var hasChanged = currentColumns.findIndex(function (colId, i) { return colId !== newColumns[i]; });
+                if (hasChanged > -1) {
+                    _this.updateColumnsInCurrentDataState(value.columnApi);
                 }
             });
             agGrid.rowSelected
@@ -1538,6 +1570,17 @@
                 _this.dataManagerSvc.updateDataState(_this.currentDataState, _this.viewConfig.id);
             });
         };
+        SkyAgGridDataManagerAdapterDirective.prototype.updateColumnsInCurrentDataState = function (columnApi) {
+            var columnOrder = this.getColumnOrder(columnApi);
+            var viewState = this.currentDataState.getViewStateById(this.viewConfig.id);
+            viewState.displayedColumnIds = columnOrder;
+            this.dataManagerSvc.updateDataState(this.currentDataState.addOrUpdateView(this.viewConfig.id, viewState), this.viewConfig.id);
+        };
+        SkyAgGridDataManagerAdapterDirective.prototype.getColumnOrder = function (columnApi) {
+            return columnApi
+                .getAllDisplayedVirtualColumns()
+                .map(function (col) { return col.getColDef().colId; });
+        };
         SkyAgGridDataManagerAdapterDirective.prototype.displayColumns = function (dataState) {
             var e_1, _a;
             var agGrid = this.currentAgGrid;
@@ -1639,7 +1682,7 @@
             var _this = this;
             this.zone.run(function () {
                 _this.indicatorShouldShow = false;
-                _this.changeDetector.markForCheck();
+                _this.changeDetector.detectChanges();
             });
         };
         SkyAgGridCellValidatorTooltipComponent.prototype.hidePopover = function () {
@@ -1649,7 +1692,7 @@
             var _this = this;
             this.zone.run(function () {
                 _this.indicatorShouldShow = true;
-                _this.changeDetector.markForCheck();
+                _this.changeDetector.detectChanges();
             });
         };
         SkyAgGridCellValidatorTooltipComponent.prototype.showPopover = function () {
@@ -1928,7 +1971,7 @@
             return this.rowDeleteConfigs.find(function (rowDelete) { return rowDelete.id === row.id; });
         };
         SkyAgGridRowDeleteDirective.prototype.affixToRow = function (affixer, id) {
-            var rowElement = this.elementRef.nativeElement.querySelector('[row-id="' + id + '"] div[aria-colindex="1"]');
+            var rowElement = this.elementRef.nativeElement.querySelector("\n      [row-id=\"" + id + "\"] div[aria-colindex=\"1\"],\n      .ag-row.sky-ag-grid-row-" + id + " div[aria-colindex=\"1\"]\n    ");
             // This covers cases where AG Grid places the column index on an inner element (such as with the `enableCellTextSelection` option)
             // This appears to have been fixed by AG Grid in version 24 and so it is not testable after that version.
             /* istanbul ignore if */