-
-
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
[docs] Smooth performance animation #8986
[docs] Smooth performance animation #8986
Conversation
Netlify deploy previewNetlify deploy preview: https://deploy-preview-8986--material-ui-x.netlify.app/ Updated pagesNo updates. These are the results for the performance tests:
|
e228ae5
to
d3deb48
Compare
This section of the docs would go away if we go through with the row memoization by default, I wouldn't spend too much time on this. |
Agree, sorry I should have clarified what I had in mind with the "(overkill PR)" in the PR description 😁
@noraleonte Yeah, depending on what we want to prioritize, I feel that we could indeed improve these demo progress bars, it's not very welcoming, e.g. on https://mui.com/x/ ![]() The custom theme demo is already better: ![]() |
@oliviertassinari I agree. The smooth corners, bolder font and slightly smoother colors make a huge difference. It's a cool improvement opportunity I think 🎉 |
I'm merging for now, we could continue to improve it, but since we are working on fixing the root problem of these rerenders, this demo might soon be removed from the docs. Thanks for the input. Two points I think that can make as general rules:
|
Since #7846, we have a great performance demo, however, I noticed that we could improve the design of the animation, so I did for fun (overkill PR):
Screen.Recording.2023-05-14.at.19.19.12.mov
https://mui.com/x/react-data-grid/performance/#memoize-inner-components-with-react-memo. It smoothly appears (feels a bit laggy) and then instantly disappears (feels a bit brutal). I think that it should be reversed like Chrome whose repaints feel great:
Screen.Recording.2023-05-14.at.18.12.27.mov
For the colors, we could consider picking the ones of React Dev Tools' UX, but it feels a bit cheap:
Screen.Recording.2023-05-14.at.18.10.37.mov
Here is how it looks now: https://deploy-preview-8986--material-ui-x.netlify.app/x/react-data-grid/performance/#memoize-inner-components-with-react-memo
Screen.Recording.2023-05-14.at.19.26.13.mov