-
-
Notifications
You must be signed in to change notification settings - Fork 32.1k
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
[website] Add a templates & design kits section to the Material UI page #38617
[website] Add a templates & design kits section to the Material UI page #38617
Conversation
Netlify deploy previewhttps://deploy-preview-38617--material-ui.netlify.app/ Bundle size report |
I like this a lot! It's a great way to showcase the available templates and what's achievable when customizing Material UI. One thing, though, is about the removal of those items from Maybe we could think of a redundant way to do this, by leaving exclusively the Material UI templates on |
Yeah, the potential page views drop as an effect of removing the Templates & Design kits of the Products section is an important thing to acknowledge. It'd be great to set up a funnel view on GA to see where people are coming from when accessing those routes. Overall though, doing so doesn't mean that the homepage shouldn't talk about them. I'm looking forward to revamping the homepage in a general way this year still but, until that's done, we could add a section that would point to |
[DEMOS[2]]: <ShoppingBag fontSize="small" />, | ||
}; | ||
|
||
const TEMPLATES = { |
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.
Could we share these with https://mui.com/templates/? So we can easily keep it up to date.
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.
Any suggestions on how to do that? Is creating a JSON file with it a correct way to do it?
Nice 👍 Product / Design ideas:
I doubt this is a frequent user flow. Most are likely coming from the side nav link, and the template pages in the docs. Google Analytics flow charts could tell us thoughts but I don't have the time to check. Overall, I think changes like this should be reviewed by an engineer (not me). I did a quick pass while I was at it. What I can spot, mostly focused on page lighthouse score (e.g. performance). So to add a developer in the review 😁:
|
Not sure if some changes in this PR affect this section or not, but this looks like a bug to me. Screen.Recording.2566-08-25.at.15.37.01.mov |
@siriwatknp Uhm... I'm afraid not. 😕 It's like this in the current prod version, too, and I figure it's because of the addition of the resizing handle there. |
@siriwatknp + @zanivan + @gerdadesign can y'all check this out again when there's time? |
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.
Design-wise, looks great! 🚀
@siriwatknp + @bharatkashyap Hey y'all, I'd appreciate it if you could chime in here and drop a dev-focused review! :) |
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.
🎉
@oliviertassinari Anything that you want to point out about this one before merging it? 😬 Want to make sure we've got most of it covered as I still see some stuff on PageInsights, though many of them seem not directly related to the changes done on this PR. |
@danilo-leal in terms of engineering content:
On mobile, the arrows are maybe too large |
@oliviertassinari thank you! Couldn't find the lost h6, though 😅 |
Ah right, it's gone. All good on my end then. I pushed a commit to fix https://www.webpagetest.org/vitals.php?test=230911_BiDcC2_BK6&run=3#cls, this one has been annoying me for some time. Screen.Recording.2023-09-12.at.01.48.17.mov |
This PR comes in preparation for removing the Templates and Design kits as items within the home page switcher. This removal will allow the introduction of Toolpad there (#38604) in a better way: not only having five products there is a bit overwhelming, but they don't sit, at least conceptually, on the same level as MUI Core, MUI X, and Toolpad. Templates and Design kits are more subproducts of Material UI (and Joy UI, in the future). With that said, their pages will keep existing, and we may add more Joy UI-related content soon as they're getting ironed out.
https://deploy-preview-38617--material-ui.netlify.app/material-ui/