Closed Bug 1457412 Opened 2 years ago Closed 2 years ago

On https://v-fonts.com/ the Jabin and Decovar fonts do not respond to variations

Categories

(Core :: Layout: Text and Fonts, enhancement)

Product:
Core
Component:
Layout: Text and Fonts
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla61
Tracking Flags:
Tracking Status
firefox61 --- fixed

People

(Reporter: jfkthame, Assigned: jfkthame)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

Update OTS to upstream tip (c903692702), to pick up recent fixes
11.78 KB, patch
jrmuizel
: review+
Details | Diff | Splinter Review 
This happens because their variation tables are dropped by the font sanitizer, due to minor spec deviations in the MVAR table.

Analysis shows the issues to be harmless, and the sanitizer can be relaxed to allow these fonts to load as expected.

Submitted https://github.com/khaledhosny/ots/pull/169 upstream; when this is merged, we can pull an OTS update and the issue here will be resolved.
In particular, this fixes the validation of a couple of example variation fonts seen in the wild.
Attachment #8971652 - Flags: review?(jmuizelaar)
Assignee: nobody → jfkthame
Status: NEW → ASSIGNED
Attachment #8971652 - Flags: review?(jmuizelaar) → review+
Pushed by jkew@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c9a94881c7b4
Update OTS to upstream tip (c903692702), to pick up recent fixes. r=jrmuizel
https://hg.mozilla.org/mozilla-central/rev/c9a94881c7b4
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox61: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla61
You need to log in before you can comment on or make changes to this bug.