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

Added all the Proxima Nova font files and enabled them in the CSS #10

Merged
merged 1 commit into from
Jan 25, 2012

Conversation

andreaforni
Copy link
Contributor

Hi,
I added all the Proxima Nova font files found into http://foundation.zurb.com/fonts/ to "assets/fonts" and I have re-enabled the @font-face section into the CSS.

I have added all the fonts I have found in the Foundation's website but there are two groups of fonts which are not referenced into the CSS: "black" and "regularitalic". The first is bolder then the "bold" font, the second is the same of "regitalic"; both are not present in the CSS so I think they will not be loaded by the browser. I was in doubt if remove them or not and, at the end, I leave them. If you think are needless, fell free to delete them. The files are:

  • proximanova-black-webfont.eot
  • proximanova-black-webfont.svg
  • proximanova-black-webfont.ttf
  • proximanova-black-webfont.woff
  • proximanova-regularitalic-webfont.eot
  • proximanova-regularitalic-webfont.svg
  • proximanova-regularitalic-webfont.ttf
  • proximanova-regularitalic-webfont.woff

I tried to load the page into Firefox, Chrome and Safari on my Mac and the font seems to be loaded correctly. I also tried the website under IE8 but I didn't see changes, so I tried to understand why but I didn't found a reason. I've noticed that if I change the "browser mode" from IE8 to IE7 then the font seems to be loaded.... bah!

Andrea

andreareginato added a commit that referenced this pull request Jan 25, 2012
Added all the Proxima Nova font files and enabled them in the CSS
@andreareginato andreareginato merged commit 48037a4 into andreareginato:master Jan 25, 2012
@pigats pigats mentioned this pull request May 22, 2012
Closed
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

Successfully merging this pull request may close these issues.

2 participants