Skip to content

Commit e56e2b3

Browse files
1024th1024th
authored
Treat diagonal-tailed ij variants like hooky (not flat-tailed) (#2947)
* Treat diagonal-tailed `ij` variants like hooky (not flat-tailed) Remove outdated `dfHook` in `Body.DiagonalTailed` of lower-j * Adjust the spacing of diagonal-tailed `j`
1 parent 3ab78df commit e56e2b3

File tree

2 files changed

+10
-9
lines changed

2 files changed

+10
-9
lines changed

‎packages/font-glyphs/src/letter/latin/lower-il.ptl‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -168,18 +168,18 @@ glyph-block Letter-Latin-Lower-I : begin
168168
'serifedAsymmetric' { Body.Serifed Serifs.SerifedShort Marks.Serifed [XMiddle mode].Serifed para.advanceScaleI Stroke }
169169
'tailed' { Body.Tailed Serifs.None Marks.Serifless [XMiddle mode].Tailed para.advanceScaleI Stroke }
170170
'tailedSerifed' { Body.Tailed Serifs.Hooky Marks.Serifed [XMiddle mode].TailedSerifed para.advanceScaleI Stroke }
171+
'diagonalTailed' { Body.DiagTailed Serifs.None Marks.Serifless [XMiddle mode].Tailed para.advanceScaleI Stroke }
172+
'serifedDiagonalTailed' { Body.DiagTailed Serifs.Hooky Marks.Serifed [XMiddle mode].TailedSerifed para.advanceScaleI Stroke }
171173
'flatTailed' { Body.FlatTailed Serifs.None Marks.Serifless [XMiddle mode].FlatTailed para.advanceScaleI Stroke }
172174
'serifedFlatTailed' { Body.FlatTailed Serifs.Hooky Marks.Serifed [XMiddle mode].SerifedFlatTailed para.advanceScaleI Stroke }
173-
'diagonalTailed' { Body.DiagTailed Serifs.None Marks.Serifless [XMiddle mode].FlatTailed para.advanceScaleI Stroke }
174-
'serifedDiagonalTailed' { Body.DiagTailed Serifs.Hooky Marks.Serifed [XMiddle mode].SerifedFlatTailed para.advanceScaleI Stroke }
175175
'semiTailed' { Body.SemiTailed Serifs.None Marks.Serifless [XMiddle mode].FlatTailed para.advanceScaleI Stroke }
176176
'serifedSemiTailed' { Body.SemiTailed Serifs.Hooky Marks.Serifed [XMiddle mode].SerifedFlatTailed para.advanceScaleI Stroke }
177177
# Decompressed
178178
'hookyDec' { Body.Serifless Serifs.Hooky Marks.Serifed [XMiddle mode].Hooky para.advanceScaleI 0 }
179179
'seriflessDec' { Body.Serifless Serifs.None Marks.Serifless [XMiddle mode].Center para.advanceScaleI 0 }
180180
'tailedDec' { Body.Tailed Serifs.None Marks.Serifless [XMiddle mode].Tailed para.advanceScaleI Stroke }
181+
'diagonalTailedDec' { Body.DiagTailed Serifs.None Marks.Serifless [XMiddle mode].Tailed para.advanceScaleI Stroke }
181182
'flatTailedDec' { Body.FlatTailed Serifs.None Marks.Serifless [XMiddle mode].FlatTailed para.advanceScaleI Stroke }
182-
'diagonalTailedDec' { Body.DiagTailed Serifs.None Marks.Serifless [XMiddle mode].FlatTailed para.advanceScaleI Stroke }
183183
'semiTailedDec' { Body.SemiTailed Serifs.None Marks.Serifless [XMiddle mode].FlatTailed para.advanceScaleI Stroke }
184184
'hookyBottomDec' { Body.HookyBottom Serifs.None Marks.Serifless [XMiddle mode].HookyBottom para.advanceScaleI Stroke }
185185
# R Tail
@@ -194,8 +194,8 @@ glyph-block Letter-Latin-Lower-I : begin
194194
# Special variants for Tau (which is built using dotlessi)
195195
'tau/tailless' { Body.Serifless Serifs.None Marks.Serifless [XMiddle mode].Center 1 0 }
196196
'tau/tailed' { Body.Tailed Serifs.None Marks.Serifless [XMiddle mode].Tailed 1 Stroke }
197+
'tau/diagonalTailed' { Body.DiagTailed Serifs.None Marks.Serifless [XMiddle mode].Tailed 1 Stroke }
197198
'tau/flatTailed' { Body.FlatTailed Serifs.None Marks.Serifless [XMiddle mode].FlatTailed 1 Stroke }
198-
'tau/diagonalTailed' { Body.DiagTailed Serifs.None Marks.Serifless [XMiddle mode].FlatTailed 1 Stroke }
199199
'tau/semiTailed' { Body.SemiTailed Serifs.None Marks.Serifless [XMiddle mode].FlatTailed 1 Stroke }
200200
'tau/shortTailed' { Body.ShortTailed Serifs.None Marks.Serifless [XMiddle mode].Center 1 Stroke }
201201

‎packages/font-glyphs/src/letter/latin/lower-j.ptl‎

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@ glyph-block Letter-Latin-Lower-J : begin
3838
export : define [StraightSerifless df] : begin df.middle
3939
export : define [StraightSerifed df] : df.middle + [IBalance2 df]
4040
export : define [BentHook df] : df.middle + JBalance
41+
export : define [DiagonalTailed df] : df.middle + [IBalance2 df]
4142
export : define [FlatHookSerifless df] : df.middle + 0.25 * JBalance * df.adws * [mix 1 df.adws 2]
4243
export : define [FlatHookSerifed df] : df.middle + JBalance * df.adws
4344

@@ -66,11 +67,10 @@ glyph-block Letter-Latin-Lower-J : begin
6667
serif -- false
6768

6869
export : define [DiagonalTailed df top xMiddle] : glyph-proc
69-
local dfHook : DivFrame para.advanceScaleI
7070
include : dispiro
7171
widths.center
7272
flat xMiddle top [heading Downward]
73-
DiagTail.L xMiddle Descender [DiagTail.StdDepth dfHook Stroke] Stroke
73+
DiagTail.L xMiddle Descender [DiagTail.StdDepth df Stroke] Stroke
7474
set-base-anchor "trailing" (xMiddle - [HSwToV HalfStroke]) Descender
7575

7676
define Serifs : namespace
@@ -89,6 +89,7 @@ glyph-block Letter-Latin-Lower-J : begin
8989

9090
define Div : namespace
9191
export : define BentHook 1
92+
export : define DiagonalTailed para.advanceScaleI
9293
export : define StraightSerifless para.advanceScaleII
9394
export : define StraightSerifed para.advanceScaleI
9495
export : define FlatHookSerifless para.advanceScaleII
@@ -98,15 +99,15 @@ glyph-block Letter-Latin-Lower-J : begin
9899
'bentHookSerifless' { "BentHook" Serifs.None XMiddle.BentHook Marks.Serifless Div.BentHook }
99100
'bentHookShortSerifed' { "BentHook" Serifs.Short XMiddle.BentHook Marks.Serifless Div.BentHook }
100101
'bentHookSerifed' { "BentHook" Serifs.Long XMiddle.BentHook Marks.Serifed Div.BentHook }
102+
'diagonalTailedSerifless' { "DiagonalTailed" Serifs.None XMiddle.DiagonalTailed Marks.Serifless Div.DiagonalTailed }
103+
'diagonalTailedShortSerifed' { "DiagonalTailed" Serifs.Short XMiddle.DiagonalTailed Marks.Serifless Div.DiagonalTailed }
104+
'diagonalTailedSerifed' { "DiagonalTailed" Serifs.Long XMiddle.DiagonalTailed Marks.Serifed Div.DiagonalTailed }
101105
'straightSerifless' { "Straight" Serifs.None XMiddle.StraightSerifless Marks.Serifless Div.StraightSerifless }
102106
'straightShortSerifed' { "Straight" Serifs.Short XMiddle.StraightSerifless Marks.Serifless Div.StraightSerifless }
103107
'straightSerifed' { "Straight" Serifs.Long XMiddle.StraightSerifed Marks.Serifed Div.StraightSerifed }
104108
'flatHookSerifless' { "FlatHook" Serifs.None XMiddle.FlatHookSerifless Marks.Serifless Div.FlatHookSerifless }
105109
'flatHookShortSerifed' { "FlatHook" Serifs.Short XMiddle.FlatHookSerifless Marks.Serifless Div.FlatHookSerifless }
106110
'flatHookSerifed' { "FlatHook" Serifs.Long XMiddle.FlatHookSerifed Marks.Serifed Div.FlatHookSerifed }
107-
'diagonalTailedSerifless' { "DiagonalTailed" Serifs.None XMiddle.FlatHookSerifless Marks.Serifless Div.FlatHookSerifless }
108-
'diagonalTailedShortSerifed' { "DiagonalTailed" Serifs.Short XMiddle.FlatHookSerifless Marks.Serifless Div.FlatHookSerifless }
109-
'diagonalTailedSerifed' { "DiagonalTailed" Serifs.Long XMiddle.FlatHookSerifed Marks.Serifed Div.FlatHookSerifed }
110111

111112
foreach { suffix { shapeId Serif xMiddleT Marks adws } } [Object.entries JConfig] : do
112113
local df : DivFrame adws

0 commit comments

Comments
 (0)