-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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.
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
[docs] Make cell editable in demos #1817
Conversation
packages/grid/_modules_/grid/hooks/features/keyboard/useGridKeyboard.ts
Outdated
Show resolved
Hide resolved
packages/grid/x-grid-data-generator/src/renderer/renderEditProgress.tsx
Outdated
Show resolved
Hide resolved
eba40ca
to
3741047
Compare
7f50193
to
254654c
Compare
packages/grid/_modules_/grid/hooks/features/rows/useGridEditRows.ts
Outdated
Show resolved
Hide resolved
170d2b2
to
a24035d
Compare
// See https://github.com/mui-org/material-ui/issues/10534 | ||
if ( | ||
(event.target as any).nodeType === 1 && | ||
!event.currentTarget.contains(event.target as Element) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
BTW I thought I put a comment about this one, or is it on another PR and we are repeating it again?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a0da475
to
6973927
Compare
6973927
to
993314a
Compare
I've updated this PR to also make the cells editable in the employee dataset. |
Closes #1625
To be merged after #1772
Previews: