1340298 - Headers on postmodernjukebox.com use the wrong font in Nightly/DevEdition, with "downloadable font: Layout: DFLT script doesn't satisfy the spec. LangSysCount is not zero: 1 " and more errors

Status: VERIFIED WORKSFORME

(Web Compatibility :: Site Reports, defect, P5)

Description

[Daniel Holbert [:dholbert]]

STR:
 1. visit http://postmodernjukebox.com/ , in a fullscreen browser (to be sure you don't get a modified responsive smaller-screen version of the site).

 2. Look at the header links ("Home", "Buy Tickets", ..., "Sign Up").

EXPECTED RESULTS:
The header links should be using a custom font which has a distinctive "small-caps" look. And nothing should be overlapping.

ACTUAL RESULTS:
The header links use a normal font -- the downloadable font gets rejected for some reason.  This results in larger text than the page is designed for, and at some browser sizes, some text gets covered up by the logo at the center of this header section.

Comment 1

[Daniel Holbert [:dholbert]]

Mozregression says this regressed quite a while ago:
 1:42.26 INFO: Last good revision: 2387ada864282880d3a498d643abe3d8b887ee59 (2015-10-14)
 1:42.26 INFO: First bad revision: e193b4da0a8c1025aa76a403c64663ff1cd41709 (2015-10-15)
 1:42.26 INFO: Pushlog:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=2387ada864282880d3a498d643abe3d8b887ee59&tochange=e193b4da0a8c1025aa76a403c64663ff1cd41709


BUT, current Firefox Release 51.0.1 gives "expected results".  So I'm not sure what's going on, regression-wise. Maybe there's a feature involved that's Nightly-only?

Comment 2

[Daniel Holbert [:dholbert]]

Attachment: errors from browser console

Here are all the errors from my browser console when I load the page (& see the problem) in current Nightly, 54.0a1 (2017-02-16) (64-bit)

The font in question for these header-links is called "look".  The first few errors we post with that font name are here:
==========
downloadable font: Layout: DFLT script doesn't satisfy the spec. LangSysCount is not zero: 1 (font-family: "look" style:normal weight:200 stretch:normal src index:1) source: http://postmodernjukebox.com/wp-content/themes/jukebox/fonts/LookSans-Lig.woff  style.css:1:684
downloadable font: Layout: Failed to parse script table 0 (font-family: "look" style:normal weight:200 stretch:normal src index:1) source: http://postmodernjukebox.com/wp-content/themes/jukebox/fonts/LookSans-Lig.woff  style.css:1:684
downloadable font: GSUB: Failed to parse script list table (font-family: "look" style:normal weight:200 stretch:normal src index:1) source: http://postmodernjukebox.com/wp-content/themes/jukebox/fonts/LookSans-Lig.woff  style.css:1:684
downloadable font: rejected by sanitizer (font-family: "look" style:normal weight:200 stretch:normal src index:1) source: http://postmodernjukebox.com/wp-content/themes/jukebox/fonts/LookSans-Lig.woff  style.css:1:684

Comment 3

[Daniel Holbert [:dholbert]]

Based on the Nightly regression range (and ignoring the fact that this somehow works in current Firefox release 51.0.1), I'm guessing this was a regression from bug 1193050.

jfkthame, could you take a look? Do you know what might be going on here?

Comment 4

[Daniel Holbert [:dholbert]]

Attachment: screenshot showing broken rendering (top) vs expected rendering in Chrome (bottom)

Comment 5

[Jonathan Kew [:jfkthame]]

Known issue, see bug 1244693. In Nightly and Aurora, we apply OTS validation to OpenType layout tables, and reject the font if they fail (so there's some chance of out-of-spec fonts being noticed by or reported to people who are in a position to get them fixed); but in Beta & Release, we allow them to bypass the validator, to minimize the impact on the majority of users.

(In bug 1331737, I just landed a patch to make this behavior controllable via a pref, so it's possible to work around it, but the correct response to seeing a case like that is to report it to the site: their font does not fully conform to the OpenType spec.)

Comment 6

[Daniel Holbert [:dholbert]]

Thanks! I mostly wanted to be sure we didn't have a regression that was riding the trains to imminent release.  I'll reclassify this as Tech Evang, then.

Comment 7

[Firefox Bug Husbandry Bot]

See bug 1547409. Moving webcompat whiteboard tags to keywords.

Comment 8

[Oana Arbuzov [:oanaarbuzov]]

The layout has changed and the buttons are visible on the footer of the page, and are similar to Chrome.
https://prnt.sc/jEGNCjVGUPph
https://prnt.sc/hBfcbJSog47E

Tested with:
Browser / Version: Firefox Nightly 102.0a1 (2022-05-18)
Operating System: Windows 10 Pro

Comment 9

[Daniel Holbert [:dholbert]]

Yup, the site seems to have changed and this isn't an issue anymore. Thanks!

(Note, "INVALID" isn't quite right for this situation; that implies the bug never existed or was premised on a misunderstanding. WORKSFORME encompasses this "huh, something changed and it's no longer reproducible" scenario.)