-
-
Notifications
You must be signed in to change notification settings - Fork 32.8k
[website] Add Case studies to the homepage #46416
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
Conversation
Netlify deploy previewhttps://deploy-preview-46416--material-ui.netlify.app/ Bundle size report
|
|
@prakhargupta1, don't miss these additional comments: #46416 (review) 😉 |
Replaced framework -> library.
Removed the quotation mark.
MUI X Data Grid Premium -> Data Grid. Removed MUI X and Premium/Pro from all occurrences. Also, at some places Data
Added it in a quote and removed 'strongly'.
Replaced user experience -> UX wherever it made sense. |
|
Thanks @prakhargupta1 Let me know when the rest of the feedback has been addressed, and I'll take another look |
@alelthomas, I'm assuming you're looking into the rest of the styling related feedback but let me know if there is something that I should check. |
|
Made a bunch of little adjustments to tackle Matt and Nora's feedback too:
Let me know if I missed anything. @prakhargupta1 |
|
Some things that I noticed:
Just my 2 cents on the design:
Everything else looks good to me 🙌 |
|
RE testimonials section, this was the approved design months ago designed by Nora and Victor:
I personally like the original option better, but the implementation I went for seemed like a good compromise - but I shall open the discussion again with the rest of the team to avoid bias. I've opened a new PR to address the rest: #47555 |






Preview: https://deploy-preview-46416--material-ui.netlify.app/customers/
To do:
Continuation of #45675