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

Remove Noto Serif and use system fonts #2782

Closed
karmatosed opened this issue Sep 25, 2017 · 9 comments
Closed

Remove Noto Serif and use system fonts #2782

karmatosed opened this issue Sep 25, 2017 · 9 comments

Comments

@karmatosed
Copy link
Member

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.

@samikeijonen
Copy link
Contributor

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.

@ellatrix
Copy link
Member

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. :)

@aaronjorbin
Copy link
Member

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.

@hugobaeta
Copy link

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

@NewJenk
Copy link

NewJenk commented Sep 26, 2017

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).

@karmatosed
Copy link
Member Author

@hugobaeta I'd be interested in Georgia, what do you think?

@jeffpaul
Copy link
Member

This ticket was mentioned in Slack in #core-editor by jeffpaul. View the logs.

@jeffpaul
Copy link
Member

Discussion in today's Gutenberg bug scrub was to close this keeping Noto Serif for now and we can revisit if necessary.

@aristath
Copy link
Member

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.
I agree that Noto is prettier, but at the same time there's precedence and I wouldn't want us to use Noto now only to remove it next year.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

8 participants