search

octaviopardo / EBGaramond12

SIL Open Font License 1.1
223 stars 25 forks source link

fi ligature broken? #5

Closed ArchangeGabriel closed 6 years ago

ArchangeGabriel commented 7 years ago

So I’ve compiled a .tex doc of mine, and comparing it to the previous version (with @georgd releases), there seems to have been some kerning/spacing changes (but maybe that’s a XeLaTeX thing), but most important is that the fi ligature is broken (and I guess others are too, can test later).

georgd commented 7 years ago

Kerning/Spacing was changed by @octaviopardo and AFAIKS it’s better now. Ligatures fall in the category features to be revised.

ArchangeGabriel commented 7 years ago

Kerning/Spacing was changed by @octaviopardo and AFAIKS it’s better now.

Yes it looks better in most cases indeed. Will try to do a close comparison/check when I have time.

Ligatures fall in the category features to be revised.

I guessed as much after your previous post. ;)

RGB-es commented 7 years ago

The f ligatures seems to be implemented as discretionary ligatures instead of "standard ligatures". Also, the "long tail capital Q" is now implemented as swash instead of standard ligature as before and this is a problem, specially with italics where there are many swashes. I really like the Qu ligature, but I'm not a fan of swashes ;)

georgd commented 7 years ago

It won't stay this way, the original behaviour will be more or less restored.

RGB-es notifications@github.com schrieb am Fr., 7. Apr. 2017, 22:12:

The f ligatures seems to be implemented as discretionary ligatures instead of "standard ligatures". Also, the "long tail capital Q" is now implemented as swash instead of standard ligature as before and this is a problem, specially with italics where there are many swashes. I really like the Qu ligature, but I'm not a fan of swashes ;)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/octaviopardo/EBGaramond12/issues/5#issuecomment-292640129, or mute the thread https://github.com/notifications/unsubscribe-auth/AAND8rNcXhrdA2rMaKDIsU02DlRhEpEwks5rtpirgaJpZM4MsP6R .

ya-d commented 7 years ago

Thanks for all your work and the update. Unfortunately ligatures are still broken for me in LibreOffice v5.4.0.3 (with v0.8 of your fonts). Can anybody confirm this? Because strangly it seems to be fine in OpenOffice 4.1.3. Any other font with ligatures shows them by default – including the original version of EB Garamond. (And even italics of v.0.8)

Plus I'm all for restoring the original grouping of ligatures.

standard ligatures missing in lo

ArchangeGabriel commented 7 years ago

From your captures it seems that italic ligatures are working, but not regular ones. And I’m not sure about fh, fk and fb (+ ff versions) behaviours.

ArchangeGabriel commented 7 years ago

The fi ligature is working for me in XeLaTeX. Maybe that’s an LO issue then?

ArchangeGabriel commented 7 years ago

On a side note, spacing/kerning seems much better now. The 1 have been fixed too apparently, so for my use case this version is now excellent. :)