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

docs: fix fontweight #20

Open
zoepage opened this issue Aug 20, 2015 · 3 comments
Open

docs: fix fontweight #20

zoepage opened this issue Aug 20, 2015 · 3 comments

Comments

@zoepage
Copy link
Contributor

zoepage commented Aug 20, 2015

http://hoodiehq.github.io/hoodie-css/colour-fonts.html

we use 300, 400 && 500. please fix this is the documentation :)

@verpixelt
Copy link
Member

I'd like to put the font-weight: 400 up for discussion. We're using it just in a few places where imho either 300 || 500 would be a fit. This could save us a few kb in the font.css file.

@tamacodechi
Copy link
Member

I'd like to weigh in ;)
I think it's fine to take out 400. Poked around changing it up for 300 and 500 and nothing drastic happened. It would save a nice relative amount of kb.
(Maybe add to the docs how to add 400 if someone using the hoodie-css for their own thing wants to - maybe even on a guide/docs for the basic template so there's not too much clutter?)

@NickColley
Copy link

👍 for removing 400, fonts are so huge :(

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

4 participants