Open vv-monsalve opened 2 years ago
Hi @RobLeuschke, We've created a separate repo for Luxurious Roman.
Please use the latest file Luxurious-Roman.glyphs that I've just uploaded in DropBox to check the design issues checklist for it.
There are some issues already found.
M
height does not match the uppercase height.
second
glyph is missing and minute
needs to be adjusted.
c
)
Also, would you please provide information for this font Description file? Thanks
@vv-monsalve I have completed the modifications of the forms of this font and created a preliminary exercise of the spacing and kerning. Nodes have been simplified in most (if not all) characters. Please review this font before moving on to production.
Hi @RobLeuschke. I'll check the current file in DropBox and let you know.
@crystaltype After merging the latest PR with the Mastering job, the anchor position for n
was not ok in Luxurious.
Please remember what we've talked about the counter and the anchor position relationship.
L1 Mastering Font checklist
Diacritics
ogonek
cedilla
to avoid huge marksGlyphs & Outlines
Resolve all metrics
to update the metrics in the entire font.Update Glyph Info
command to avoid bad names issuesConvert to cubic
command to avoid qcurves@
¿?
¡!
””
«»
position, spacing is goodOT Features
.fina
glyphs are actually ending shapes or ss are not repeated under different names.fina
andinit
casesSmallCaps
feature instead and change it.update
&compile
buttons to check everything is working well.Testing changes
gen-html diff
if needed (for update font cases)gen-html proof
to see if it's working well on web. Use the--imgs
flag to produce images.