Skip to content

[DataGrid] Preserve key input during row edit when using rowModesModel - #20759

Merged
michelengelen merged 5 commits into
mui:masterfrom
michelengelen:bugfix/11980
Feb 24, 2026
Merged

[DataGrid] Preserve key input during row edit when using rowModesModel#20759
michelengelen merged 5 commits into
mui:masterfrom
michelengelen:bugfix/11980

Conversation

@michelengelen

@michelengelen michelengelen commented Dec 23, 2025

Copy link
Copy Markdown
Member

This now preserves the first printable key when starting row edit mode through controlled row modes.

Fixes #11980

Before: https://stackblitz.com/edit/ltbl9cra?file=src%2FDemo.tsx
After: https://stackblitz.com/edit/ltbl9cra-cze2bltz?file=src%2FDemo.tsx

…sModel`

Signed-off-by: michel <jsnerdic@gmail.com>
@michelengelen michelengelen self-assigned this Dec 23, 2025
@michelengelen michelengelen added scope: data grid Changes related to the data grid. type: regression A bug, but worse, it used to behave as expected. feature: Editing Related to the data grid Editing feature labels Dec 23, 2025
@mui-bot

mui-bot commented Dec 23, 2025

Copy link
Copy Markdown

Deploy preview: https://deploy-preview-20759--material-ui-x.netlify.app/

Bundle size report

ℹ️ Using snapshot from parent commit 0d23231 (fallback from merge base 2cb92ab).

Bundle Parsed size Gzip size
@mui/x-data-grid 🔺+88B(+0.02%) 🔺+42B(+0.03%)
@mui/x-data-grid-pro 🔺+88B(+0.02%) 🔺+33B(+0.02%)
@mui/x-data-grid-premium 🔺+88B(+0.01%) 🔺+30B(+0.01%)
@mui/x-charts 🔺+726B(+0.20%) 🔺+179B(+0.16%)
@mui/x-charts-pro 🔺+730B(+0.15%) 🔺+193B(+0.13%)
@mui/x-charts-premium 🔺+732B(+0.15%) 🔺+224B(+0.15%)
@mui/x-date-pickers 0B(0.00%) 0B(0.00%)
@mui/x-date-pickers-pro 0B(0.00%) 0B(0.00%)
@mui/x-tree-view 0B(0.00%) 0B(0.00%)
@mui/x-tree-view-pro 0B(0.00%) 0B(0.00%)

Details of bundle changes

Generated by 🚫 dangerJS against a00244b

@cherniavskii cherniavskii left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested with https://stackblitz.com/edit/ltbl9cra-xqtgg2ph?file=package.json
Seems to work fine 👍🏻

@cherniavskii cherniavskii added needs cherry-pick The PR should be cherry-picked to master after merge. v8.x labels Feb 19, 2026
@michelengelen
michelengelen merged commit 330535b into mui:master Feb 24, 2026
28 checks passed
@michelengelen
michelengelen deleted the bugfix/11980 branch February 24, 2026 09:07
@github-actions

Copy link
Copy Markdown
Contributor

Cherry-pick PRs will be created targeting branches: v8.x

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature: Editing Related to the data grid Editing feature needs cherry-pick The PR should be cherry-picked to master after merge. scope: data grid Changes related to the data grid. type: regression A bug, but worse, it used to behave as expected. v8.x

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[data grid] Different edit behaviour with/without controlled rowModesModel

3 participants