-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Remove Noto Serif and use system fonts #2782
Comments
Sounds good. For the block content we'd naturally use the same font as in the front end. Haven't looked yet is that easy or not. |
Worth noting that core does not use system fonts for the editor. For that it uses Georgia. Would be great to stick to a serif font. :) |
SInce ideally this is usually overridden by a theme's stylesheet, I think that avoiding loading an external resource that won't be used is a good idea. |
Agree with @iseulde, would be great to keep a serif stack. If it's system fonts (georgia, times, etc) it shouldn't add any weight. Either way, also agree with @aaronjorbin - the themes should be overwriting this anyway :D Let me know if help is needed there @karmatosed |
Giving the editor a little separation from the rest of the admin definitely feels quite natural, so not sure if using system fonts would improve things from a UI perspective. But making it super-easy to use front-end fonts is essential (IMO). |
@hugobaeta I'd be interested in Georgia, what do you think? |
This ticket was mentioned in Slack in #core-editor by jeffpaul. View the logs. |
Discussion in today's Gutenberg bug scrub was to close this keeping Noto Serif for now and we can revisit if necessary. |
I know this one was closed, but still... my 2 cents: WP-Core was for a long time using Open-Sans for the dashboard and then in font-natively we reverted to system fonts. |
We are using Noto Serif for the editor, if we remove this and go for system fonts we can make some speed gains. It will remove some personality, but the gains would be good to have. WordPress itself uses system fonts, so there is a precedent.
The text was updated successfully, but these errors were encountered: