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

FontAwesome icons are not displayed in IE8 #361

Closed
tombu opened this issue Sep 2, 2012 · 1 comment
Closed

FontAwesome icons are not displayed in IE8 #361

tombu opened this issue Sep 2, 2012 · 1 comment

Comments

@tombu
Copy link

tombu commented Sep 2, 2012

There seems to be an issue with FontAwesome in IE8 (maybe also in IE7, I haven´t tried that yet). For me, using FontAwesome, the icons were just displayed as rectangles or vertical lines in IE8. After comparing the @font-face declaration of FontAwesome (https://github.com/FortAwesome/Font-Awesome/blob/master/less/font-awesome.less) and of this gem (https://github.com/seyhunak/twitter-bootstrap-rails/blob/master/vendor/toolkit/fontawesome.less) I recognized the missing format(...) declaration.

I discovered that this was removed in PR #312 but I couldn´t detect why the removal was necessary. Can someone please enlighten me?

At the moment I patched that problem by overriding the fontawesome.less or rather requiring a fontawesome_override.css.less (copied from the FontAwesome repo) which contains the format(...) declaration. Further, I recognized that this workaround also fixes my DeadClient error that I used to have with phantomjs (teampoltergeist/poltergeist#44).

It would help me a lot if anyone could explain why the format(...) declaration was removed in PR #312. Depending on your opinion(s) I will open a PR.

@toadkicker
Copy link
Contributor

Its caused over confusion of using single quotes or double quotes. We should always use double quotes when using @font-face in CSS.

toadkicker@363ca64

This issue was 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

No branches or pull requests

2 participants