-
Notifications
You must be signed in to change notification settings - Fork 13
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
Font choice #28
Comments
Inter is the wrong typeface to use here. Safari has a built-in typeface keyword which can be used to invoke the apple system typeface: |
The actual code already uses From body {
…
font-family: -apple-system, BlinkMacSystemFont, Segoe UI, Roboto, Oxygen,
Ubuntu, Cantarell, Fira Sans, Droid Sans, Helvetica Neue, sans-serif;
} |
I'm not familiar with the javascript at work here but the typeface being used is pretty clearly not the system typeface. I have no clue why though as reading the style sheet tells me exactly what you've posted. The safari inspector however, tells us the real typeface is in fact SF Pro followed by Helvetica Neue. |
Hello everyone,
I stumbled upon the CodeEdit project a while ago and now saw that you made a landing page for it. I really like how you're mimicking the apple website. However, what throws me off just very slightly is the fallback to the Helvetica font. This even applies to most apple devices as they somehow do not recognize
SF Pro Display
in Safari.To complete the apple-appearance I suggest using the Inter font. It comes very close to SF Pro and is a longterm favorite of mine.
If you like, I'd be happily implementing it into the site 😃
Regards,
Adrian
The text was updated successfully, but these errors were encountered: