@mui/x-data-grid 7.29.2 → 7.29.3

package/CHANGELOG.md

@@ -3,6 +3,44 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## 7.29.3
+
+_May 8, 2025_
+
+We'd like to offer a big thanks to the 3 contributors who made this release possible. Here are some highlights ✨:
+
+- 🐞 Bugfixes
+
+Team members who have contributed to this release:
+@arminmeh, @LukasTy, and @MBilalShafi.
+
+<!--/ HIGHLIGHT_ABOVE_SEPARATOR /-->
+
+### Data Grid
+
+#### `@mui/x-data-grid@7.29.3`
+
+- [DataGrid] Ignore `preProcessEditCellProps` for non-editable columns when starting a row update (#17734) @arminmeh
+- [DataGrid] Avoid applying row selection propagation on filtered rows (#17742) @MBilalShafi
+
+#### `@mui/x-data-grid-pro@7.29.3` [![pro](https://mui.com/r/x-pro-svg)](https://mui.com/r/x-pro-svg-link 'Pro plan')
+
+Same changes as in `@mui/x-data-grid@7.29.3`.
+
+#### `@mui/x-data-grid-premium@7.29.3` [![premium](https://mui.com/r/x-premium-svg)](https://mui.com/r/x-premium-svg-link 'Premium plan')
+
+Same changes as in `@mui/x-data-grid-pro@7.29.3`.
+
+### Date and Time Pickers
+
+#### `@mui/x-date-pickers@7.29.3`
+
+- [DateTimePicker] Fix focus behavior on desktop variant (#17730) @LukasTy
+
+#### `@mui/x-date-pickers-pro@7.29.3` [![pro](https://mui.com/r/x-pro-svg)](https://mui.com/r/x-pro-svg-link 'Pro plan')
+
+Same changes as in `@mui/x-date-pickers@7.29.3`.
+
 ## 7.29.2
 
 _May 1, 2025_

package/hooks/features/editing/useGridRowEditing.js

@@ -11,7 +11,7 @@ import { GridEditModes, GridRowModes } from "../../../models/gridEditRowModel.js
 import { useGridApiMethod } from "../../utils/useGridApiMethod.js";
 import { gridEditRowsStateSelector, gridRowIsEditingSelector } from "./gridEditingSelectors.js";
 import { isPrintableKey, isPasteShortcut } from "../../../utils/keyboardUtils.js";
-import { gridColumnFieldsSelector, gridVisibleColumnFieldsSelector } from "../columns/gridColumnsSelector.js";
+import { gridColumnDefinitionsSelector, gridVisibleColumnFieldsSelector } from "../columns/gridColumnsSelector.js";
 import { gridRowsLookupSelector } from "../rows/gridRowsSelector.js";
 import { deepClone } from "../../../utils/utils.js";
 import { GridRowEditStopReasons, GridRowEditStartReasons } from "../../../models/params/gridRowParams.js";
@@ -313,8 +313,9 @@ export const useGridRowEditing = (apiRef, props) => {
       initialValue
     } = params;
     const row = apiRef.current.getRow(id);
-    const columnFields = gridColumnFieldsSelector(apiRef);
-    const newProps = columnFields.reduce((acc, field) => {
+    const columns = gridColumnDefinitionsSelector(apiRef);
+    const newProps = columns.reduce((acc, col) => {
+      const field = col.field;
       const cellParams = apiRef.current.getCellParams(id, field);
       if (!cellParams.isEditable) {
         return acc;
@@ -331,7 +332,7 @@ export const useGridRowEditing = (apiRef, props) => {
       acc[field] = {
         value: newValue,
         error: false,
-        isProcessingProps: !!column.preProcessEditCellProps && deleteValue
+        isProcessingProps: column.editable && !!column.preProcessEditCellProps && deleteValue
       };
       return acc;
     }, {});
@@ -340,8 +341,8 @@ export const useGridRowEditing = (apiRef, props) => {
     if (fieldToFocus) {
       apiRef.current.setCellFocus(id, fieldToFocus);
     }
-    columnFields.filter(field => !!apiRef.current.getColumn(field).preProcessEditCellProps && deleteValue).forEach(field => {
-      const column = apiRef.current.getColumn(field);
+    columns.filter(column => column.editable && !!column.preProcessEditCellProps && deleteValue).forEach(column => {
+      const field = column.field;
       const value = apiRef.current.getCellValue(id, field);
       const newValue = deleteValue ? getDefaultCellValue(column) : initialValue ?? value;
       Promise.resolve(column.preProcessEditCellProps({

package/hooks/features/rowSelection/utils.js

@@ -97,7 +97,7 @@ const getFilteredRowNodeSiblings = (tree, filteredRows, id) => {
 export const findRowsToSelect = (apiRef, tree, selectedRow, autoSelectDescendants, autoSelectParents, addRow, selectedIds = new Set(gridRowSelectionStateSelector(apiRef.current.state))) => {
   const filteredRows = gridFilteredRowsLookupSelector(apiRef);
   const selectedDescendants = new Set([]);
-  if (!autoSelectDescendants && !autoSelectParents) {
+  if (!autoSelectDescendants && !autoSelectParents || filteredRows[selectedRow] === false) {
     return;
   }
   if (autoSelectDescendants) {

package/index.js

@@ -1,5 +1,5 @@
 /**
- * @mui/x-data-grid v7.29.2
+ * @mui/x-data-grid v7.29.3
  *
  * @license MIT
  * This source code is licensed under the MIT license found in the

package/modern/hooks/features/editing/useGridRowEditing.js

@@ -11,7 +11,7 @@ import { GridEditModes, GridRowModes } from "../../../models/gridEditRowModel.js
 import { useGridApiMethod } from "../../utils/useGridApiMethod.js";
 import { gridEditRowsStateSelector, gridRowIsEditingSelector } from "./gridEditingSelectors.js";
 import { isPrintableKey, isPasteShortcut } from "../../../utils/keyboardUtils.js";
-import { gridColumnFieldsSelector, gridVisibleColumnFieldsSelector } from "../columns/gridColumnsSelector.js";
+import { gridColumnDefinitionsSelector, gridVisibleColumnFieldsSelector } from "../columns/gridColumnsSelector.js";
 import { gridRowsLookupSelector } from "../rows/gridRowsSelector.js";
 import { deepClone } from "../../../utils/utils.js";
 import { GridRowEditStopReasons, GridRowEditStartReasons } from "../../../models/params/gridRowParams.js";
@@ -313,8 +313,9 @@ export const useGridRowEditing = (apiRef, props) => {
       initialValue
     } = params;
     const row = apiRef.current.getRow(id);
-    const columnFields = gridColumnFieldsSelector(apiRef);
-    const newProps = columnFields.reduce((acc, field) => {
+    const columns = gridColumnDefinitionsSelector(apiRef);
+    const newProps = columns.reduce((acc, col) => {
+      const field = col.field;
       const cellParams = apiRef.current.getCellParams(id, field);
       if (!cellParams.isEditable) {
         return acc;
@@ -331,7 +332,7 @@ export const useGridRowEditing = (apiRef, props) => {
       acc[field] = {
         value: newValue,
         error: false,
-        isProcessingProps: !!column.preProcessEditCellProps && deleteValue
+        isProcessingProps: column.editable && !!column.preProcessEditCellProps && deleteValue
       };
       return acc;
     }, {});
@@ -340,8 +341,8 @@ export const useGridRowEditing = (apiRef, props) => {
     if (fieldToFocus) {
       apiRef.current.setCellFocus(id, fieldToFocus);
     }
-    columnFields.filter(field => !!apiRef.current.getColumn(field).preProcessEditCellProps && deleteValue).forEach(field => {
-      const column = apiRef.current.getColumn(field);
+    columns.filter(column => column.editable && !!column.preProcessEditCellProps && deleteValue).forEach(column => {
+      const field = column.field;
       const value = apiRef.current.getCellValue(id, field);
       const newValue = deleteValue ? getDefaultCellValue(column) : initialValue ?? value;
       Promise.resolve(column.preProcessEditCellProps({

package/modern/hooks/features/rowSelection/utils.js

@@ -97,7 +97,7 @@ const getFilteredRowNodeSiblings = (tree, filteredRows, id) => {
 export const findRowsToSelect = (apiRef, tree, selectedRow, autoSelectDescendants, autoSelectParents, addRow, selectedIds = new Set(gridRowSelectionStateSelector(apiRef.current.state))) => {
   const filteredRows = gridFilteredRowsLookupSelector(apiRef);
   const selectedDescendants = new Set([]);
-  if (!autoSelectDescendants && !autoSelectParents) {
+  if (!autoSelectDescendants && !autoSelectParents || filteredRows[selectedRow] === false) {
     return;
   }
   if (autoSelectDescendants) {

package/modern/index.js

@@ -1,5 +1,5 @@
 /**
- * @mui/x-data-grid v7.29.2
+ * @mui/x-data-grid v7.29.3
  *
  * @license MIT
  * This source code is licensed under the MIT license found in the

package/node/hooks/features/editing/useGridRowEditing.js

@@ -321,8 +321,9 @@ const useGridRowEditing = (apiRef, props) => {
       initialValue
     } = params;
     const row = apiRef.current.getRow(id);
-    const columnFields = (0, _gridColumnsSelector.gridColumnFieldsSelector)(apiRef);
-    const newProps = columnFields.reduce((acc, field) => {
+    const columns = (0, _gridColumnsSelector.gridColumnDefinitionsSelector)(apiRef);
+    const newProps = columns.reduce((acc, col) => {
+      const field = col.field;
       const cellParams = apiRef.current.getCellParams(id, field);
       if (!cellParams.isEditable) {
         return acc;
@@ -339,7 +340,7 @@ const useGridRowEditing = (apiRef, props) => {
       acc[field] = {
         value: newValue,
         error: false,
-        isProcessingProps: !!column.preProcessEditCellProps && deleteValue
+        isProcessingProps: column.editable && !!column.preProcessEditCellProps && deleteValue
       };
       return acc;
     }, {});
@@ -348,8 +349,8 @@ const useGridRowEditing = (apiRef, props) => {
     if (fieldToFocus) {
       apiRef.current.setCellFocus(id, fieldToFocus);
     }
-    columnFields.filter(field => !!apiRef.current.getColumn(field).preProcessEditCellProps && deleteValue).forEach(field => {
-      const column = apiRef.current.getColumn(field);
+    columns.filter(column => column.editable && !!column.preProcessEditCellProps && deleteValue).forEach(column => {
+      const field = column.field;
       const value = apiRef.current.getCellValue(id, field);
       const newValue = deleteValue ? (0, _utils3.getDefaultCellValue)(column) : initialValue ?? value;
       Promise.resolve(column.preProcessEditCellProps({

package/node/hooks/features/rowSelection/utils.js

@@ -105,7 +105,7 @@ const getFilteredRowNodeSiblings = (tree, filteredRows, id) => {
 const findRowsToSelect = (apiRef, tree, selectedRow, autoSelectDescendants, autoSelectParents, addRow, selectedIds = new Set((0, _gridRowSelectionSelector.gridRowSelectionStateSelector)(apiRef.current.state))) => {
   const filteredRows = (0, _gridFilterSelector.gridFilteredRowsLookupSelector)(apiRef);
   const selectedDescendants = new Set([]);
-  if (!autoSelectDescendants && !autoSelectParents) {
+  if (!autoSelectDescendants && !autoSelectParents || filteredRows[selectedRow] === false) {
     return;
   }
   if (autoSelectDescendants) {

package/node/index.js

@@ -1,5 +1,5 @@
 /**
- * @mui/x-data-grid v7.29.2
+ * @mui/x-data-grid v7.29.3
  *
  * @license MIT
  * This source code is licensed under the MIT license found in the

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mui/x-data-grid",
-  "version": "7.29.2",
+  "version": "7.29.3",
   "description": "The Community plan edition of the Data Grid components (MUI X).",
   "author": "MUI Team",
   "main": "./node/index.js",