-
Notifications
You must be signed in to change notification settings - Fork 12
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 #13
Comments
i don't think we should use helvetica for the text, too. Serifs are nice. Try garamond. |
Serifs are nice, but Garamond is not a good web typeface because of its On Mon, Mar 11, 2013 at 2:50 PM, Vivek Bhagwat notifications@github.comwrote:
|
+1 |
georgia pushed! yay |
feel free to reopen if you're not a fan of how it looks |
I think we should use Helvetica for the headers still. |
howzat? On Mon, Mar 11, 2013 at 5:50 PM, Vivek Bhagwat notifications@github.comwrote:
|
that looks better, although there are still a few issues with the typography overall. My suggested changes:
|
I don't know about Georgia. But it might also be a spacing issue of the text on the page. Idk. Actually, yeah, I just don't like serifs. D: ANYWAY. Adrian said do fonts last, so I guess we can leave this issue open. |
I like Baskerville fwiw. it's a serif font. |
sexy 10/10 +1 On Tue, Mar 12, 2013 at 10:50 AM, Daria notifications@github.com wrote:
|
fixed by #17 |
I vote we use Open Sans. it's the new hotness and looks better than Helvetica. |
Not opposed. http://hellohappy.org/beautiful-web-type/ is worth checking out. (Open Sans on the third part) Daria, you should make a pull request. |
^ Yep. I'm using http://www.google.com/fonts/specimen/Playfair+Display and Open Sans for my personal site. |
Idk what looks good. I just updated with better helvetica but i think it might be a bit too thin. but it's really pretty. lemme know if you guys have font suggestions.
The text was updated successfully, but these errors were encountered: