-
Notifications
You must be signed in to change notification settings - Fork 21
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
Opsz bug in Firefox #208
Comments
I can reproduce this in Firefox stable, but not in Firefox Nightly: I am curious why this is happening, though. I guess a careful comparison of an opsz font that does not do this using a tool like https://github.com/googlefonts/fontdiffenator or https://github.com/source-foundry/fdiff or OTMaster could uncover what is happening... |
This seems to happen only in Fraunces, so I would like this to be debugged and fixed by Viviana or someone else :) |
Hi! I have a possible solution that seems to fix this issue but I didn‘t tested if it cause something unwanted in other areas. I can make a PR later if no one is already working on this. |
PR would be great, or just an explanation :) |
I changed the default value for opsz to match the min value and rebuilt the fonts. |
I'm curious what the TTX diff is for the fvar table, eg
|
That was exactly what I’ve done to know why the minisite version was working fine.
|
Thanks, Guido. It was solved here |
Hi!
I’ve found an odd issue when setting Fraunces to the max value for opsz and a font size greater than 256px. When these conditions are matched It doesn‘t renders the correct font variation.
In the screenshot below you can see two lines.
Both uses the same font variation settings but different font files. The top line uses the current font files from the "fonts" folder of this repo and the bottom line, that is working as expected, uses a font file from the Fraunces Minisite.
From Left to Right: Chrome, Safari, Firefox.
Probably it‘s a FF bug but I couldn’t reproduce this error with other fonts that has opsz too.
Here I attach the test html so you can see this by yourselves.
FrauncesTestCase.zip
Best.
The text was updated successfully, but these errors were encountered: