1001044 - [tarako] error messages related to loading Fira Sans, which should already be available via FxOS

Status: RESOLVED FIXED

(Marketplace Graveyard :: Consumer Pages, defect, P2)

Description

[David Durst [:ddurst]]

Probably related to bug 993716, bug 991419 (and probably not bug 993717), and certainly bug 996066, the tarako packaged app is showing warnings and downloads for what should be a local font.

If we need to remove the font-face declarations, so be it.

Comment 1

[David Durst [:ddurst]]

Note: 991419 has commits, but the messages are still being seen. Probably where to start.

Comment 2

[Jonathan Kew [:jfkthame]]

It's not clear to me where to look for the current code of the app, or exactly what Gecko version(s) you're looking at. 

**Provided** you have Gecko bug 769194 fixed in the B2G version concerned, it should be possible to avoid any Fira downloads or warnings. But the CSS may need some fixup to achieve this, because bug 754215 is not yet fixed.

For example, at

 https://github.com/mozilla/fireplace/blob/master/hearth/media/css/lib.styl#L75

I see the code

 $open-stack = "Fira Sans OT", "Fira Sans", FiraSansWeb, sans-serif;

which I think is problematic: because of bug 754215, using this value for font-family will always attempt to instantiate FiraSansWeb, even though "Fira Sans OT" is available.

Then *if you don't have the fix for bug 769194* the src:local descriptors will fail, and the src:url descriptors will be downloaded.

It sounds to me like you may want bug 769194 to be uplifted to 1.3T, so that the src:local descriptors will work, if this is important for the product.

Alternatively, if the app in question is specifically designed for Tarako (rather than a website targeting all browsers on all systems), there's no real need for it to have @font-face declarations for Fira at all; just use font-family: "Fira Sans OT" and you'll get the desired font. Or indeed font-family:sans-serif, given that Fira is the default sans-serif font on the device.

Comment 3

[Davor Spasovski [:spasovski]]

https://github.com/mozilla/yogafire/commit/89fb4c6