mui-x
mui-x copied to clipboard
[MUIX-11171] - onRowEditStop not called when blurring the row from a non-editable cell
Description
Update implementation to ensure that onRowEditStop is triggered when user clicks on an uneditable cell after editing a cell on same row.
https://www.loom.com/share/911cef0d17f5442794ee8e71d80b650e
Fixes #11171
- [x] I have followed (at least) the PR section of the contributing guide.
This code was written and reviewed by GitStart Community. Growing great engineers, one PR at a time.
@Janpot @michelengelen this PR is ready for review
Deploy preview: https://deploy-preview-11377--material-ui-x.netlify.app/
Generated by :no_entry_sign: dangerJS against aa28267958de67f36de518782bb7e5936c6be65a
Thanks @gitstart ... could you first make sure that the CI checks are all green? Thanks again!! 🙇🏼
Hi @gitstart I think you might have obscured the gitlog with 83b07a5. Could you try a hard reset and force push to 6d13c03.
on your current branch do this.
git reset --hard 6d13c038f2c9316fcca22d603f0a85858b1697bc;
git push --force;
This will reset the branch to the commit that had all the valid changes. After that try merging upstream/next and push that.
If you need more help please feel free to tag me.
To avoid this from regressing again in the future, do you see an opportunity for adding an automated test that covers this behavior?
This pull request has conflicts, please resolve those before we can evaluate the pull request.
This pull request has conflicts, please resolve those before we can evaluate the pull request.
To avoid this from regressing again in the future, do you see an opportunity for adding an automated test that covers this behavior?
@gitstart gentle ping! ^^
Hi @michelengelen, apologies for the delay in implementing this feedback, this is due to some internal resource shuffling on our end. However, we will attend to this as soon as we are able, thanks.
This pull request has conflicts, please resolve those before we can evaluate the pull request.