Skip to content
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

Conversation

danilo-leal
Copy link
Contributor

@danilo-leal danilo-leal commented Aug 23, 2023

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/

@danilo-leal danilo-leal added the website Pages that are not documentation-related, marketing-focused. label Aug 23, 2023
@danilo-leal danilo-leal self-assigned this Aug 23, 2023
@mui-bot
Copy link

mui-bot commented Aug 23, 2023

Netlify deploy preview

https://deploy-preview-38617--material-ui.netlify.app/

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against eb556e2

@zanivan
Copy link
Contributor

zanivan commented Aug 23, 2023

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 mui.com—I wonder if doing this it wouldn't harm our number of access to both the templates and design kits page.

Maybe we could think of a redundant way to do this, by leaving exclusively the Material UI templates on mui.com/material-ui, and Joy UI templates on mui.com/joy-ui, and so on, but keeping an overall view on mui.com directing users to the mui.com/templates —IMHO we could also polish out this page a bit to add both Joy UI and Base UI templates in the future

@danilo-leal
Copy link
Contributor Author

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 /templates and /design-kits, with a copywriting that's a bit broader, covering them for all libraries (even if we don't have, for example, a Figma kit for Joy UI ready to go yet).

docs/src/components/productMaterial/MaterialDesignKits.tsx Outdated Show resolved Hide resolved
[DEMOS[2]]: <ShoppingBag fontSize="small" />,
};

const TEMPLATES = {
Copy link
Member

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.

Copy link
Contributor Author

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?

docs/src/components/productMaterial/MaterialTemplates.tsx Outdated Show resolved Hide resolved
@oliviertassinari
Copy link
Member

oliviertassinari commented Aug 23, 2023

Nice 👍


Product / Design ideas:

  • To consider if we want to have a horizontal scrollbar (new)
Screenshot 2023-08-23 at 23 43 41
  • The section feels quite large on my end. I wouldn't be against having it a bit (10%?) smaller.
Screenshot 2023-08-24 at 00 12 34

One thing, though, is about the removal of those items from mui.com—I wonder if doing this it wouldn't harm our number of access to both the templates and design kits page.

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 😁:

1 Screenshot 2023-08-23 at 23 46 03
2 Screenshot 2023-08-23 at 23 46 01
3 Screenshot 2023-08-23 at 23 46 08
  • I don't think we should load the images with background color, this prevents the browser to lazily loading them based on the viewport. These are now loaded
  • It looks like we could compress a bit more the images, ImageOptim found 10% win on the largest image.

@siriwatknp
Copy link
Member

It'd be great if the height of the carousel is smaller. Currently, I cannot focus on it with my 13" Mac pro

image

@siriwatknp
Copy link
Member

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

@danilo-leal
Copy link
Contributor Author

@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.

@danilo-leal
Copy link
Contributor Author

@siriwatknp + @zanivan + @gerdadesign can y'all check this out again when there's time?

Copy link
Contributor

@zanivan zanivan left a 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! 🚀

@danilo-leal danilo-leal added the package: material-ui Specific to @mui/material label Aug 31, 2023
@danilo-leal
Copy link
Contributor Author

@siriwatknp + @bharatkashyap Hey y'all, I'd appreciate it if you could chime in here and drop a dev-focused review! :)

Copy link
Member

@bharatkashyap bharatkashyap left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@danilo-leal
Copy link
Contributor Author

@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.

@oliviertassinari
Copy link
Member

oliviertassinari commented Sep 11, 2023

PageInsights, though many of them seem not directly related to the changes done on this PR.

@danilo-leal in terms of engineering content:

Anything that you want to point out

On mobile, the arrows are maybe too large

@danilo-leal
Copy link
Contributor Author

@oliviertassinari thank you! Couldn't find the lost h6, though 😅

@oliviertassinari
Copy link
Member

oliviertassinari commented Sep 11, 2023

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

@danilo-leal danilo-leal merged commit 1915190 into mui:master Sep 12, 2023
18 checks passed
@danilo-leal danilo-leal deleted the add-templates-designkits-tothe-materialui-page branch September 12, 2023 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
package: material-ui Specific to @mui/material website Pages that are not documentation-related, marketing-focused.
Projects
Status: Recently completed
Development

Successfully merging this pull request may close these issues.

6 participants