ag-grid-community 33.2.3 → 33.2.4

package/dist/package/main.esm.mjs

@@ -856,7 +856,7 @@ function _waitUntil(condition, callback, timeout = 100, timeoutMessage) {
 }
 
 // packages/ag-grid-community/src/version.ts
-var VERSION = "33.2.3";
+var VERSION = "33.2.4";
 
 // packages/ag-grid-community/src/validation/logging.ts
 var MAX_URL_LENGTH = 2e3;
@@ -11269,6 +11269,9 @@ var RowCtrl = class extends BeanStub {
       this.allRowGuis.forEach(callback);
     }
   }
+  isRowRendered() {
+    return this.allRowGuis.length > 0;
+  }
   onRowHeightChanged(gui) {
     if (this.rowNode.rowHeight == null) {
       return;
@@ -27404,13 +27407,14 @@ var FocusService = class extends BeanStub {
   // grid cell will still be focused as far as the grid is concerned,
   // however the browser focus will have moved somewhere else.
   getFocusCellToUseAfterRefresh() {
-    if (this.gos.get("suppressFocusAfterRefresh") || !this.focusedCell) {
+    const { gos, focusedCell } = this;
+    if (gos.get("suppressFocusAfterRefresh") || gos.get("suppressCellFocus") || !focusedCell) {
       return null;
     }
     if (!this.doesRowOrCellHaveBrowserFocus()) {
       return null;
     }
-    return this.focusedCell;
+    return focusedCell;
   }
   getFocusHeaderToUseAfterRefresh() {
     if (this.gos.get("suppressFocusAfterRefresh") || !this.focusedHeader) {
@@ -30800,33 +30804,44 @@ var RowRenderer = class extends BeanStub {
       this.allRowCtrls = liveList;
     }
   }
-  isCellRendered(rowIndex, column) {
+  /**
+   * Checks if the cell is rendered or not. Also returns true if row ctrl is present but has not rendered
+   * cells yet.
+   * @returns true if cellCtrl is present, or if the row is present but has not rendered rows yet
+   */
+  isCellBeingRendered(rowIndex, column) {
     const rowCtrl = this.rowCtrlsByRowIndex[rowIndex];
-    if (!column) {
+    if (!column || !rowCtrl) {
       return !!rowCtrl;
     }
-    if (rowCtrl && rowCtrl.isFullWidth()) {
+    if (rowCtrl.isFullWidth()) {
       return true;
     }
     const spannedCell = this.beans.spannedRowRenderer?.getCellByPosition({ rowIndex, column, rowPinned: null });
-    if (spannedCell) {
-      return true;
-    }
-    return !!rowCtrl?.getCellCtrl(column);
+    return !!spannedCell || !!rowCtrl.getCellCtrl(column) || !rowCtrl.isRowRendered();
   }
   /**
    * Notifies all row and cell controls of any change in focused cell.
    * @param event cell focused event
    */
+  updateCellFocus(event) {
+    this.getAllCellCtrls().forEach((cellCtrl) => cellCtrl.onCellFocused(event));
+    this.getFullWidthRowCtrls().forEach((rowCtrl) => rowCtrl.onFullWidthRowFocused(event));
+  }
+  /**
+   * Called when a new cell is focused in the grid
+   * - if the focused cell isn't rendered; re-draw rows to dry to render it
+   * - subsequently updates all cell and row controls with the new focused cell
+   * @param event cell focused event
+   */
   onCellFocusChanged(event) {
     if (event && event.rowIndex != null && !event.rowPinned) {
       const col = this.beans.colModel.getCol(event.column) ?? void 0;
-      if (!this.isCellRendered(event.rowIndex, col)) {
+      if (!this.isCellBeingRendered(event.rowIndex, col)) {
         this.redraw();
       }
     }
-    this.getAllCellCtrls().forEach((cellCtrl) => cellCtrl.onCellFocused(event));
-    this.getFullWidthRowCtrls().forEach((rowCtrl) => rowCtrl.onFullWidthRowFocused(event));
+    this.updateCellFocus(event);
   }
   onSuppressCellFocusChanged(suppressCellFocus) {
     this.getAllCellCtrls().forEach((cellCtrl) => cellCtrl.onSuppressCellFocusChanged(suppressCellFocus));
@@ -30837,10 +30852,8 @@ var RowRenderer = class extends BeanStub {
   // all active cells.
   registerCellEventListeners() {
     this.addManagedEventListeners({
-      cellFocused: (event) => {
-        this.onCellFocusChanged(event);
-      },
-      cellFocusCleared: () => this.onCellFocusChanged(),
+      cellFocused: (event) => this.onCellFocusChanged(event),
+      cellFocusCleared: () => this.updateCellFocus(),
       flashCells: (event) => {
         const { cellFlashSvc } = this.beans;
         if (cellFlashSvc) {
@@ -31149,7 +31162,7 @@ var RowRenderer = class extends BeanStub {
       return;
     }
     if (!focusSvc.doesRowOrCellHaveBrowserFocus()) {
-      this.onCellFocusChanged(
+      this.updateCellFocus(
         _addGridCommonParams(this.gos, {
           ...cellToFocus,
           forceBrowserFocus: true,

package/dist/package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ag-grid-community",
-  "version": "33.2.3",
+  "version": "33.2.4",
   "description": "Advanced Data Grid / Data Table supporting Javascript / Typescript / React / Angular / Vue",
   "main": "./dist/package/main.cjs.js",
   "types": "./dist/types/src/main.d.ts",
@@ -88,7 +88,7 @@
   ],
   "homepage": "https://www.ag-grid.com/",
   "dependencies": {
-    "ag-charts-types": "11.2.3"
+    "ag-charts-types": "11.2.4"
   },
   "devDependencies": {
     "source-map-loader": "^5.0.0",

package/dist/types/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ag-grid-community",
-  "version": "33.2.3",
+  "version": "33.2.4",
   "description": "Advanced Data Grid / Data Table supporting Javascript / Typescript / React / Angular / Vue",
   "main": "./dist/package/main.cjs.js",
   "types": "./dist/types/src/main.d.ts",
@@ -88,7 +88,7 @@
   ],
   "homepage": "https://www.ag-grid.com/",
   "dependencies": {
-    "ag-charts-types": "11.2.3"
+    "ag-charts-types": "11.2.4"
   },
   "devDependencies": {
     "source-map-loader": "^5.0.0",

package/dist/types/src/rendering/row/rowCtrl.d.ts

@@ -165,6 +165,7 @@ export declare class RowCtrl extends BeanStub<RowCtrlEvent> {
     resetHoveredStatus(el?: HTMLElement): void;
     private roundRowTopToBounds;
     forEachGui(gui: RowGui | undefined, callback: (gui: RowGui) => void): void;
+    isRowRendered(): boolean;
     protected onRowHeightChanged(gui?: RowGui): void;
     destroyFirstPass(suppressAnimation?: boolean): void;
     destroySecondPass(): void;

package/dist/types/src/rendering/rowRenderer.d.ts

@@ -49,11 +49,23 @@ export declare class RowRenderer extends BeanStub implements NamedBean {
     getStickyTopRowCtrls(): RowCtrl[];
     getStickyBottomRowCtrls(): RowCtrl[];
     private updateAllRowCtrls;
-    private isCellRendered;
+    /**
+     * Checks if the cell is rendered or not. Also returns true if row ctrl is present but has not rendered
+     * cells yet.
+     * @returns true if cellCtrl is present, or if the row is present but has not rendered rows yet
+     */
+    private isCellBeingRendered;
     /**
      * Notifies all row and cell controls of any change in focused cell.
      * @param event cell focused event
      */
+    private updateCellFocus;
+    /**
+     * Called when a new cell is focused in the grid
+     * - if the focused cell isn't rendered; re-draw rows to dry to render it
+     * - subsequently updates all cell and row controls with the new focused cell
+     * @param event cell focused event
+     */
     private onCellFocusChanged;
     private onSuppressCellFocusChanged;
     private registerCellEventListeners;

package/dist/types/src/version.d.ts

@@ -1 +1 @@
-export declare const VERSION = "33.2.3";
+export declare const VERSION = "33.2.4";

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ag-grid-community",
-  "version": "33.2.3",
+  "version": "33.2.4",
   "description": "Advanced Data Grid / Data Table supporting Javascript / Typescript / React / Angular / Vue",
   "main": "./dist/package/main.cjs.js",
   "types": "./dist/types/src/main.d.ts",
@@ -88,7 +88,7 @@
   ],
   "homepage": "https://www.ag-grid.com/",
   "dependencies": {
-    "ag-charts-types": "11.2.3"
+    "ag-charts-types": "11.2.4"
   },
   "devDependencies": {
     "source-map-loader": "^5.0.0",