-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
[test] Add tests for the custom slots of TreeItem2
#13314
Conversation
Deploy preview: https://deploy-preview-13314--material-ui-x.netlify.app/ |
skip: ['reactTestRenderer', 'componentProp', 'componentsProp', 'themeVariants'], | ||
})); | ||
|
||
describeSlotsConformance({ |
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.
This works great for super basic slots that are always rendered, but describeSlotsConformance
will need a lot more work to support more complex slots like dragAndDropOverlay
or components like DatePicker
.
But it's a 1st draft and for now I think it does the job.
This pull request has conflicts, please resolve those before we can evaluate the pull request. |
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.
Very nice addition 🎉 It's a great first step 💙 - it will be interesting to adapt for the label editing 🤔
Part of #12434