-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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] Update Data Grid demo to v6-beta #7863
[docs] Update Data Grid demo to v6-beta #7863
Conversation
These are the results for the performance tests:
|
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.
Nice improvement 🎉
A very minor nitpick for some future PR, as the list grows, it might be a little bit difficult to scroll to the desired feature (if a user decides not to use search), maybe it will make dx
slightly better by sorting them alphabetically (also maintaining the package order).
@@ -169,6 +168,25 @@ export const featuresSet = [ | |||
detailPage: '/tree-data/', | |||
demo: <TreeDataFullExample />, | |||
}, | |||
{ |
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.
Should we also add it to the v5 doc with a link to v6 and a small note ?
Co-authored-by: Matheus Wichman <matheushw@outlook.com> Signed-off-by: José Rodolfo Freitas <joserodolfo.freitas@gmail.com>
Co-authored-by: Matheus Wichman <matheushw@outlook.com> Signed-off-by: José Rodolfo Freitas <joserodolfo.freitas@gmail.com>
Summary
Preview
https://deploy-preview-7863--material-ui-x.netlify.app/x/react-data-grid/demo/