-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Inter font downloaded but not applied #11507
Comments
Hi @amoser67! Thanks for reporting this issue. How are you specifying the |
Hi Sam, thanks for the response. I think I may have figured out my issue. I saw that the Inter font was being downloaded, but I think that was just from the Shopify admin, rather than the app iframe. So I added
to my If this sounds correct to you, then I think we can close this issue. |
Yes, that sounds correct to me! Thanks for verifying, @amoser67 👍 I think the browser will load it from cache even in the iframe since the source is the same. Conditions will vary depending on the user's browser and settings, but that seems like the best solution for now 😄 |
@amoser67 Double confirming this is the correct fix and noting we have since updated the Remix template 🚀 |
Issue summary
It seems that the "Inter" font is being downloaded as expected (https://cdn.shopify.com/static/fonts/inter/v4/InterVariable.woff2?v=4.0), but when I adjust the Polaris sans-serif font family variable from "Inter, -apple-system, ..., sans-serif" to "Inter", all text on the page defaults to a serif font.
I first noticed this because only normal and bold font weights were working, even when using the "medium" or "semibold" values for the fontWeight property of the Text component, making me suspect the Inter font wasn't being applied as expected.
My conclusion is that the Inter font is not being used, so I wanted to reach out and see if this was a known issue, and/or if others were able to replicate the issue.
For reference, I have been using Polaris to build out an app using the Remix app template for the past several months, and everything else seems to be working as expected.
Specifications
The text was updated successfully, but these errors were encountered: