@linzjs/step-ag-grid 22.2.0 → 22.2.1

package/dist/step-ag-grid.esm.js

@@ -2337,9 +2337,13 @@ const useGridContextMenu = ({ contextMenuSelectRow, contextMenu: ContextMenu, })
  * AgGrid checkbox select does not pass clicks within cell but not on the checkbox to checkbox.
  * This passes the event to the checkbox when you click anywhere in the cell.
  */
-const clickInputWhenContainingCellClicked = ({ data, event, colDef }) => {
+const clickInputWhenContainingCellClicked = (params) => {
+    const { data, event, colDef } = params;
     if (!data || !event)
         return;
+    if (fnOrVar(colDef.editable, params) === false) {
+        return;
+    }
     const element = event.target;
     // Already handled
     if (["BUTTON", "INPUT"].includes(element?.tagName) && element.closest(".ag-cell-inline-editing"))
@@ -2350,25 +2354,16 @@ const clickInputWhenContainingCellClicked = ({ data, event, colDef }) => {
     const colId = colDef.colId;
     if (!colId)
         return;
-    const clickInput = (cnt) => {
+    const clickInput = () => {
         const cell = row.querySelector(`[col-id='${colId}']`);
         if (!cell)
             return;
         const input = cell.querySelector("input, button");
-        if (!input) {
-            return;
-        }
-        // When clicking on a cell that is not editing, the cell changes to editing and the input/button ref becomes invalid
-        // So wait until the cell is in edit mode before sending the click
-        if (!input.ownerDocument.contains(input)) {
-            if (cnt !== 0) {
-                setTimeout(() => clickInput(cnt - 1));
-            }
+        if (!input)
             return;
-        }
         input?.dispatchEvent(event);
     };
-    setTimeout(() => clickInput(20), 10);
+    setTimeout(clickInput, 20);
 };
 
 /**