Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Don't include Roboto fonts by default #2247

Closed
jgoux opened this issue Apr 10, 2015 · 3 comments
Closed

Don't include Roboto fonts by default #2247

jgoux opened this issue Apr 10, 2015 · 3 comments
Assignees
Milestone

Comments

@jgoux
Copy link

jgoux commented Apr 10, 2015

Hi,
would it be possible to remove the @import url(https://fonts.googleapis.com/css?family=Roboto:300,400,500,700,400italic); from the build by default ?
I don't want to rely on external resources and this forces the user to do so.
I prefer to use something like https://www.npmjs.com/package/webfont-dl to have them locally :
webfont-dl \"https://fonts.googleapis.com/css?family=Roboto:300,400,500,700,400italic\" -o ./src/styles/vendors/_fonts.scss --font-out=./src/assets/fonts --css-rel=assets/fonts

@grizzm0
Copy link

grizzm0 commented Apr 10, 2015

This was changed in 2d2d960 via #1009. And I agree with you. I don't think the lib should force me to load assets from a remote source.

Even tho I use google fonts myself I don't think we should force it. Also it's a soft dependency as nothing will break without it.

Also, what if I'm creating an offline application?

Ping @marcysutton

@marcysutton
Copy link
Contributor

Great points. We have reopened #1009 as we have a discussion happening with the Material Design UX team about the best way to handle this. We will keep you posted!

@ThomasBurleson
Copy link
Contributor

FYI -
screen shot 2015-04-11 at 8 11 07 am

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

No branches or pull requests

4 participants