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

[examples] Fix Joy UI Next.js App Router font loading #38095

Merged

Conversation

IgnacioUtrilla
Copy link
Contributor

@IgnacioUtrilla IgnacioUtrilla commented Jul 21, 2023

This PR aims to align all the used font-family styles of MUI components when next/font is used.
Similar to this PR: #38026

Before

Screenshot 2023-07-21 at 17 19 49

After

Screenshot 2023-07-21 at 17 19 20

Close #38083

@mui-bot
Copy link

mui-bot commented Jul 21, 2023

Netlify deploy preview

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

Bundle size report

No bundle size changes (Toolpad)
No bundle size changes

Generated by 🚫 dangerJS against 80e50e6

@oliviertassinari oliviertassinari added examples Relating to /examples bug 🐛 Something doesn't work labels Jul 23, 2023
@oliviertassinari oliviertassinari changed the title fix: joy-next-app-router Font usage with next/font [examples] Fix Joy UI Next.js App Router font loading Jul 23, 2023
@oliviertassinari
Copy link
Member

I have pushed a commit to normalize all the Next.js font loading.

Off-topic. The content of the pages for the Joy UI and Material UI examples don't really make sense for me, those are starters, so I think they should point to the templates page but not include any.

@mj12albert
Copy link
Member

I have pushed a commit to normalize all the Next.js font loading.

Thanks ~ looks neater to do this in the theme!

Copy link
Member

@mj12albert mj12albert left a comment

Choose a reason for hiding this comment

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

@mj12albert mj12albert merged commit 42fa316 into mui:master Jul 24, 2023
<FormLabel>Email</FormLabel>
<Input name="email" type="email" placeholder="[email protected]" />
</FormControl>
<FormControl>
<FormControl id="password">
Copy link
Member

Choose a reason for hiding this comment

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

Fix related to this problem #38103.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something doesn't work examples Relating to /examples
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[examples][Joy] Fix font loading in the Joy UI + Next.js App Router example
5 participants