Merge pull request #1384 from ghiscoding/chore/before-edit-cell

chore: cancel edit by calling `preventDefault()` w/`onBeforeEditCell`

Author: ghiscoding

docs/column-functionalities/Editors.md

@@ -534,7 +534,7 @@ With that in mind and the code from the SO answer, we end up with the following
 #### Component
 ```ts
   verifyCellIsEditableBeforeEditing(e, args): boolean {
-    // your logic here should return true/false if it's editable or not
+    // you can call `event.preventDefault()` to make cell not editable
     // args contains the dataContext and other Slickgrid arguments
   }
 ```

docs/grid-functionalities/Composite-Editor-Modal.md

@@ -564,8 +564,8 @@ handleOnBeforeEditCell(event) {
 
   if (column && item) {
     if (!checkItemIsEditable(item, column, grid)) {
-      event.preventDefault();
-      eventData.stopImmediatePropagation();
+      event.preventDefault(); // OR eventData.preventDefault();
+      return false;
     }
   }
   return false;
@@ -600,8 +600,8 @@ handleOnBeforeEditCell(event) {
 
   if (column && item) {
     if (!checkItemIsEditable(item, column, grid, target )) {
-      event.preventDefault();
-      eventData.stopImmediatePropagation();
+      event.preventDefault(); // OR eventData.preventDefault();
+      return false;
     }
   }
   return false;

src/app/examples/grid-composite-editor.component.ts

@@ -527,7 +527,7 @@ export class GridCompositeEditorComponent implements OnDestroy, OnInit {
 
     if (column && item) {
       if (!checkItemIsEditable(item, column, grid)) {
-        e.stopImmediatePropagation();
+        e.preventDefault(); // OR eventData.preventDefault();
         return false;
       }
     }

src/app/examples/grid-resize-by-content.component.ts

@@ -427,8 +427,8 @@ export class GridResizeByContentComponent implements OnInit {
 
     if (column && item) {
       if (!checkItemIsEditable(item, column, grid)) {
-        // event.preventDefault();
-        e.stopImmediatePropagation();
+        e.preventDefault(); // OR eventData.preventDefault();
+        return false;
       }
     }
     return false;