Skip to content
This repository has been archived by the owner on Sep 27, 2019. It is now read-only.

double letterspaced ligatures #106

Closed
schlcht opened this issue Jul 8, 2013 · 4 comments
Closed

double letterspaced ligatures #106

schlcht opened this issue Jul 8, 2013 · 4 comments
Assignees
Labels
Milestone

Comments

@schlcht
Copy link

schlcht commented Jul 8, 2013

Using the letterspace font feature, letterspacing within ligatures will be doubled (this does not happen with the typesetters function):

\input luaotfload.sty
\tracingoutput 1
\showboxdepth  10
\showboxbreadth100
\font\x = "EB Garamond:mode=node;+liga;+dlig" at 10pt
{\x\directlua{typesetters.kerns.set(0.2)}
Thefoofi}

\font\x = "EB Garamond:mode=node;-liga;letterspace=0.2" at 10pt
\x Thefoofi

\font\x = "EB Garamond:mode=node;+liga;letterspace=0.2" at 10pt
\x Thefoofi

\font\x = "EB Garamond:mode=node;+liga;+dlig;letterspace=0.2" at 10pt
\x Thefoofi
\bye

(BTW: Is there a possibility to selectively disable the breaking of ligatures? I reckon it could somehow be done with the keeptogether and/or keepligatures functions, but I don't really understand their intended use.)

@ghost ghost assigned phi-gamma Jul 9, 2013
@phi-gamma
Copy link
Member

Thanks for reporting, I’ll look into it. Regarding your other question:

(BTW: Is there a possibility to selectively disable the breaking of ligatures? I reckon it could somehow be done with the keeptogether and/or keepligatures functions, but I don't really understand their intended use.)

keeptogether() and keepligatures() are predicates: they receive a pair of glyph nodes or a ligature node respectively and depending on whether they return true or false the node processor will insert a kern or leave them untouched. I’ve put together a demo:

It’s very underoptimized to keep it simple (unnecessary string building etc.), but it shows one approach.

@schlcht
Copy link
Author

schlcht commented Jul 14, 2013

Thank you very much for the demo, this should get me started.

@phi-gamma
Copy link
Member

As of today (2013-08-31) this should no longer occur.
@schlcht please verify.

@phi-gamma
Copy link
Member

Closing this. @schlcht please reopen if you disagree.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

2 participants