Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[data grid] "Enter" key doesn't exit cell when quickly typing in a column that has preProcessEditCellProps #15348

Open
sd-danhill opened this issue Nov 8, 2024 · 2 comments
Open

[data grid] "Enter" key doesn't exit cell when quickly typing in a column that has preProcessEditCellProps #15348

sd-danhill opened this issue Nov 8, 2024 · 2 comments
Labels
bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! feature: Editing Related to the data grid Editing feature support: premium standard Support request from a Premium standard plan user. https://mui.com/legal/technical-support-sla/

Comments

@sd-danhill
Copy link

sd-danhill commented Nov 8, 2024 •
edited by github-actions bot
Loading

Steps to reproduce

Link to live example: https://stackblitz.com/edit/github-dwb4re?file=src%2Fdemo.tsx

  1. Have a column that has preProcessEditCellProps defined

Current behavior

  1. Single click on a cell
  2. Press Enter key to go into edit mode
  3. Quickly make changes (e.g. remove all content, add new name) and then press Enter key

(waiting a while after you've finished typing and pressing Enter works fine)

Expected behavior

Pressing Enter should leave the cell

Context

Cell editing with keyboard to work reliably and consistently

Your environment

npx @mui/envinfo

Mac Sequoia 15.0.1
Chrome Version 130.0.6723.116

  Don't forget to mention which browser you used.
  Output from `npx @mui/envinfo` goes here.

Search keywords: cell not exiting preProcessEditCellProps
Order ID: 78997
@sd-danhill sd-danhill added bug 🐛 Something doesn't work status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Nov 8, 2024
@github-actions github-actions bot added component: data grid This is the name of the generic UI component, not the React module! support: premium standard Support request from a Premium standard plan user. https://mui.com/legal/technical-support-sla/ labels Nov 8, 2024
@sd-danhill
Copy link
Author

fyi also tested this before #15058 was merged. Issue still happens pre + post change

@michelengelen michelengelen changed the title "Enter" key doesn't exit cell when quickly typing in a column that has preProcessEditCellProps [data grid] "Enter" key doesn't exit cell when quickly typing in a column that has preProcessEditCellProps Nov 8, 2024
@michelengelen michelengelen added feature: Editing Related to the data grid Editing feature and removed status: waiting for maintainer These issues haven't been looked at yet by a maintainer labels Nov 8, 2024
@michelengelen
Copy link
Member

Thanks for opening it. I could confirm the bug and added it to the board! 👍🏼

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug 🐛 Something doesn't work component: data grid This is the name of the generic UI component, not the React module! feature: Editing Related to the data grid Editing feature support: premium standard Support request from a Premium standard plan user. https://mui.com/legal/technical-support-sla/
Projects
MUI X Data Grid
Status: 🆕 Needs refinement
Milestone
No milestone
Development

No branches or pull requests
2 participants
@michelengelen @sd-danhill