Closed (fixed)
Project:
@font-your-face
Version:
7.x-2.x-dev
Component:
Code (general)
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
21 Feb 2012 at 10:13 UTC
Updated:
11 Jun 2012 at 21:21 UTC
I love the module but think it could use some android support.
according to this site: http://www.brianhadaway.com/font-face-declarations-on-android-devices/ it can be done with the updated syntax.
Is this something that you would be looking to update or would it conflict with your bullet proof implementation?
Comments
Comment #1
sreynen commentedWe're only generating the @font-face syntax for local fonts, so that's the most we could change. The hosted font providers do their own thing there. I didn't realize the current syntax doesn't work on Android. I'll have to read more on alternative syntaxes to make it work on Android without breaking anything else. If we do update the syntax, we'll do it in 7.x-2.x first, then backport it.
Comment #2
btmash commentedI don't have font-your-face enabled on my site so I haven't checked exactly what the syntax looks like. But the syntax we use on our sites for custom fonts that I was able to get working (seems to work on the various browsers/devices) was the following:
What I see fontsquirrel now using is the following:
So the only different seems to be the format for the iefix. I'll try and test things out on my end to see how well it all works.
Comment #3
sreynen commentedI committed the syntax BTMash suggested, which is also what Fontspring recommends. Can anyone confirm this works on Android now?
Comment #4
florisg commentedConfirmed patch from: http://drupalcode.org/project/fontyourface.git/blobdiff/c9afc518b20a7209...
works against @font-your-face 7.x-2.2
Comment #5
sreynen commentedSounds like this has been reviewed.
Comment #6
sreynen commentedThis seems to be fixed. If anyone is still having this problem with latest dev or the 7.x-2.3 release I'm making right now, please re-open this.