DataGrid - Focused cell appearance is applied to non-editable CheckBox cells on mouse clicks (DevExpress#30922)

Author: Raushen

e2e/testcafe-devextreme/tests/dataGrid/common/focus/focus.ts

@@ -298,3 +298,22 @@ test('DataGrid - FocusedRowChanged event isnt raised when the push API is used t
     },
   }));
 });
+
+test('DataGrid - Focused cell appearance is applied to non-editable CheckBox cells on mouse clicks (T1282082)', async (t) => {
+  const grid = new DataGrid(GRID_SELECTOR);
+
+  await t
+    .click(grid.getDataCell(0, 0).element)
+    .click(grid.getDataCell(0, 1).element)
+    .click(grid.getDataCell(0, 0).element)
+    .expect(grid.getDataCell(0, 0).isFocused)
+    .notOk();
+}).before(async () => createWidget('dxDataGrid', {
+  dataSource: [
+    { BoolOne: false, BoolTwo: false },
+  ],
+  columns: [
+    'BoolOne',
+    'BoolTwo',
+  ],
+}));

packages/devextreme/js/__internal/grids/grid_core/keyboard_navigation/m_keyboard_navigation.ts

@@ -1363,6 +1363,7 @@ export class KeyboardNavigationController extends modules.ViewController {
   private _clickTargetCellHandler(event, $cell) {
     const column = this._getColumnByCellElement($cell);
     const isCellEditMode = this._isCellEditMode();
+    const isEditing = this._editingController.isEditing();
 
     this.setCellFocusType();
 
@@ -1397,14 +1398,18 @@ export class KeyboardNavigationController extends modules.ViewController {
         const $target = event
           && $(event.target).closest(`${NON_FOCUSABLE_ELEMENTS_SELECTOR}, td`);
         const skipFocusEvent = $target && $target.not($cell).is(NON_FOCUSABLE_ELEMENTS_SELECTOR);
-        const isEditor = !!column && !column.command && $cell.hasClass(EDITOR_CELL_CLASS);
-        const isDisabled = !isEditor && (!args.isHighlighted || skipFocusEvent);
+        const isEditCell = !column?.command
+          && isEditing
+          && $cell.hasClass(EDITOR_CELL_CLASS);
+
+        const isDisabled = !isEditCell
+          && (!args.isHighlighted || skipFocusEvent);
         this._focus($cell, isDisabled, skipFocusEvent);
       }
     } else {
       this.setRowFocusType();
       this.setFocusedRowIndex(args.prevRowIndex);
-      if (this._editingController.isEditing() && isCellEditMode) {
+      if (isEditing && isCellEditMode) {
         this._closeEditCell();
       }
     }