mui / material-ui

Material UI: Comprehensive React component library that implements Google's Material Design. Free forever.
https://mui.com/material-ui/
MIT License
93.76k stars 32.24k forks source link

[joy-ui] Missing Inter font #38306

Open oliviertassinari opened 1 year ago

oliviertassinari commented 1 year ago

Steps to reproduce 🕹

You can reproduce this pretty much everywhere:

Current behavior 😯

Load Inter

Expected behavior 🤔

The correct font is loaded

Context 🔦

I didn't know we could sort the font by the popularity in Google Font: https://fonts.google.com/?preview.size=24&sort=popularity. With Inter and Roboto, we effectively cover some of the most popular fonts.

Your environment 🌎

npx @mui/envinfo ``` Don't forget to mention which browser you used. Output from `npx @mui/envinfo` goes here. ```
siriwatknp commented 1 year ago

https://mui.com/joy-ui/react-button/ loads "Inter":

image

I will fix the examples.

oliviertassinari commented 1 year ago

@siriwatknp Perfect, I updated the issue linking the fix.

oliviertassinari commented 1 year ago

38462 doesn't solve the issue, it addresses point 2 but not 3, nor 4.

Actually in 3, it breaks the example, we should apply the style

Screenshot 2023-08-18 at 02 26 16
siriwatknp commented 1 year ago

@oliviertassinari looks like (3) is outdated, I don't see joy-next-app-router-ts anymore in the latest master.

38540 will fix the above error.

oliviertassinari commented 1 year ago

@siriwatknp Looks like #38540 solve point 3 👍

danilo-leal commented 1 year ago

@siriwatknp + @oliviertassinari Just checking if we can close this issue ⎯ I see we have point 4 left there still open. CodeSanbox is apparently loading Inter correctly, at least to me. Are we good or are we missing something?