-
Notifications
You must be signed in to change notification settings - Fork 311
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
Incorrect spacing between f and i in Fira Mono #170
Comments
That's not (incorrect) spacing, it's an "fi" ligature. Which isn't really appropriate in a monospaced font, and I believe it has been removed from the current version. That site appears to be using the Google Fonts service, and I guess they're still deploying an old release. (See also issue #14.) |
I don't know about that… I am pretty sure latest has that; but it is discretionary.
|
But the page indicated above doesn't enable discretionary ligatures, so the fact that it's appearing shows that they're using a font where it is present in the on-by-default 'liga' feature. That shouldn't be the case in the current release, afaik. |
@jfkthame How were you able to determine if the page above was using discretionary ligatures? Is it configured via a CSS property or perhaps via a querystring parameter when calling the Google Fonts API? |
In Firefox, you can use the Element Inspector to tinker with CSS properties; if you set
on the title element where "Sofia" has the unwanted ligature, you'll see that it decomposes; which indicates that this ligature comes from a Common Ligatures feature in the font. (Equivalently, setting
would do the same thing, using the low-level OpenType feature tag.) |
@jfkthame Learn something new everyday! Thanks :) |
As shown in this article, the 'fi' in Sofia, has incorrect spacing between the letters. Even increasing the letter-spacing CSS property through Chrome Inspector fails to add the correct spacing.
http://abduzeedo.com/sofia-luxury-residence-branding
The text was updated successfully, but these errors were encountered: