fix(updateItem): call grid.updateRow instead of grid.invalidateRow

- triggering invalidateRow will destroy any opened editor, so we should not call it and the correct method to use after updating a row is to use grid.updateRow
- SlickGrid author mentioned the following: You would typically use invalidateRows()/invalidateRow() followed by render() to update the data. The updateRow()/updateCell() are there in case you absolutely have to keep the existing DOM objects instead of recreating them.

Author: Ghislain Beaulac

src/app/modules/angular-slickgrid/components/angular-slickgrid.component.ts

@@ -472,12 +472,7 @@ export class AngularSlickgridComponent implements AfterViewInit, OnDestroy, OnIn
 
     if (dataView && grid) {
       this._eventHandler.subscribe(dataView.onRowCountChanged, (e: any, args: any) => {
-        grid.updateRowCount();
-        grid.render();
-      });
-      this._eventHandler.subscribe(dataView.onRowsChanged, (e: any, args: any) => {
-        grid.invalidateRows(args.rows);
-        grid.render();
+        grid.invalidate();
       });
     }
 

src/app/modules/angular-slickgrid/services/grid.service.ts

@@ -242,8 +242,7 @@ export class GridService {
       throw new Error('We could not find SlickGrid Grid, DataView objects');
     }
 
-    const row = 0;
-    this._dataView.insertItem(row, item);
+    this._dataView.insertItem(0, item); // insert at index 0
 
     if (!shouldResortGrid) {
       this._grid.scrollRowIntoView(0); // scroll to row 0
@@ -445,20 +444,21 @@ export class GridService {
     if (itemId === undefined) {
       throw new Error(`Cannot update a row without a valid "id"`);
     }
-    const row = this._dataView.getRowById(itemId);
+    const rowNumber = this._dataView.getRowById(itemId);
 
-    if (!item || row === undefined) {
+    if (!item || rowNumber === undefined) {
       throw new Error(`Could not find the item in the grid or it's associated "id"`);
     }
 
     const gridIdx = this._dataView.getIdxById(itemId);
     if (gridIdx !== undefined) {
       // Update the item itself inside the dataView
       this._dataView.updateItem(itemId, item);
+      this._grid.updateRow(rowNumber);
 
       // highlight the row we just updated, if defined
       if (shouldHighlightRow) {
-        this.highlightRow(row, 1500);
+        this.highlightRow(rowNumber, 1500);
       }
 
       // do we want to trigger an event after updating the item