ag-grid-community 33.2.2 → 33.2.4

package/dist/package/main.esm.mjs

@@ -254,7 +254,6 @@ function _getAriaCheckboxStateName(translate, state) {
 
 // packages/ag-grid-community/src/utils/browser.ts
 var isSafari;
-var isChrome;
 var isFirefox;
 var isMacOs;
 var isIOS;
@@ -267,13 +266,6 @@ function _isBrowserSafari() {
   }
   return isSafari;
 }
-function _isBrowserChrome() {
-  if (isChrome === void 0) {
-    const win = window;
-    isChrome = !!win.chrome && (!!win.chrome.webstore || !!win.chrome.runtime) || /Chrome/.test(navigator.userAgent) && /Google Inc/.test(navigator.vendor);
-  }
-  return isChrome;
-}
 function _isBrowserFirefox() {
   if (isFirefox === void 0) {
     isFirefox = /(firefox)/i.test(navigator.userAgent);
@@ -362,7 +354,6 @@ function _isInvisibleScrollbar() {
 }
 
 // packages/ag-grid-community/src/utils/dom.ts
-var rtlNegativeScroll;
 function _radioCssClass(element, elementClass, otherElementClass) {
   const parent = element.parentElement;
   let sibling = parent && parent.firstChild;
@@ -495,46 +486,16 @@ function _getElementRectWithOffset(el) {
     bottom: offsetElementRect.bottom + (borderBottomWidth || 0)
   };
 }
-function _isRtlNegativeScroll() {
-  if (typeof rtlNegativeScroll === "boolean") {
-    return rtlNegativeScroll;
-  }
-  const template = document.createElement("div");
-  template.style.direction = "rtl";
-  template.style.width = "10px";
-  template.style.height = "5px";
-  template.style.position = "fixed";
-  template.style.top = "0px";
-  template.style.overflow = "hidden";
-  template.dir = "rtl";
-  template.innerHTML = /* html */
-  `<div style="width: 20px">
-            <span style="display: inline-block; width: 1px"></span>
-            <span style="display: inline-block; width: 1px"></span>
-        </div>`;
-  document.body.appendChild(template);
-  template.scrollLeft = 1;
-  rtlNegativeScroll = Math.floor(template.scrollLeft) === 0;
-  document.body.removeChild(template);
-  return rtlNegativeScroll;
-}
 function _getScrollLeft(element, rtl) {
   let scrollLeft = element.scrollLeft;
   if (rtl) {
     scrollLeft = Math.abs(scrollLeft);
-    if (_isBrowserChrome() && !_isRtlNegativeScroll()) {
-      scrollLeft = element.scrollWidth - element.getBoundingClientRect().width - scrollLeft;
-    }
   }
   return scrollLeft;
 }
 function _setScrollLeft(element, value, rtl) {
   if (rtl) {
-    if (_isRtlNegativeScroll()) {
-      value *= -1;
-    } else if (_isBrowserSafari() || _isBrowserChrome()) {
-      value = element.scrollWidth - element.getBoundingClientRect().width - value;
-    }
+    value *= -1;
   }
   element.scrollLeft = value;
 }
@@ -895,7 +856,7 @@ function _waitUntil(condition, callback, timeout = 100, timeoutMessage) {
 }
 
 // packages/ag-grid-community/src/version.ts
-var VERSION = "33.2.2";
+var VERSION = "33.2.4";
 
 // packages/ag-grid-community/src/validation/logging.ts
 var MAX_URL_LENGTH = 2e3;
@@ -9683,7 +9644,7 @@ var GridBodyScrollFeature = class extends BeanStub {
   shouldBlockHorizontalScroll(scrollTo) {
     const clientWidth = this.centerRowsCtrl.getCenterWidth();
     const { scrollWidth } = this.centerRowsCtrl.eViewport;
-    if (this.enableRtl && _isRtlNegativeScroll()) {
+    if (this.enableRtl) {
       if (scrollTo > 0) {
         return true;
       }
@@ -9725,7 +9686,7 @@ var GridBodyScrollFeature = class extends BeanStub {
     const minScrollLeft = 0;
     const maxScrollLeft = this.centerRowsCtrl.eViewport.scrollWidth - this.centerRowsCtrl.getCenterWidth();
     if (!fromAlignedGridsService && this.shouldBlockScrollUpdate(1 /* Horizontal */, hScrollPosition)) {
-      if (this.enableRtl && _isRtlNegativeScroll()) {
+      if (this.enableRtl) {
         hScrollPosition = hScrollPosition > 0 ? 0 : maxScrollLeft;
       } else {
         hScrollPosition = Math.min(Math.max(hScrollPosition, minScrollLeft), maxScrollLeft);
@@ -11308,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;
@@ -27443,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) {
@@ -30839,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));
@@ -30876,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) {
@@ -31188,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.2",
+  "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.2"
+    "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.2",
+  "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.2"
+    "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/utils/dom.d.ts

@@ -48,7 +48,6 @@ export declare function _getElementRectWithOffset(el: HTMLElement): {
     right: number;
     bottom: number;
 };
-export declare function _isRtlNegativeScroll(): boolean;
 export declare function _getScrollLeft(element: HTMLElement, rtl: boolean): number;
 export declare function _setScrollLeft(element: HTMLElement, value: number, rtl: boolean): void;
 export declare function _clearElement(el: HTMLElement): void;

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

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

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ag-grid-community",
-  "version": "33.2.2",
+  "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.2"
+    "ag-charts-types": "11.2.4"
   },
   "devDependencies": {
     "source-map-loader": "^5.0.0",