Skip to content

Conversation

@jmcwilliams403
Copy link
Contributor

This saves an import in letter/latin/v.ptl and letter/latin/w.ptl.

Also further code cleanup in the surrounding files.

Effectively nothing is changed otherwise, although there might be some incidental improvements of floating point rounding by simplifying some of the math with some of the additional code cleanup in letter/latin/v.ptl, particularly for Lower V with Curl ().

@be5invis
Copy link
Owner

be5invis commented Nov 6, 2025

I think isolating DiagTail-specific parameters into separate files is good for organizing things.

Actually I somehow want to move some of the parameters out from meta/aesthetics.ptl, and put them into the corresponded functions.

@jmcwilliams403 jmcwilliams403 changed the title Move certain metrics surrounding [DiagTail] from letter/shared.ptl to meta/aesthetics.ptl. Move [DiagTail.[DefaultInnerRadius]] from letter/shared.ptl to meta/aesthetics.ptl. Nov 6, 2025
@jmcwilliams403
Copy link
Contributor Author

Compromise: How about we only move [DefaultInnerRadius] out? The cursive w/v shapes have little to do with a literal diagonal tail but still use this specific metric.

@be5invis
Copy link
Owner

be5invis commented Nov 6, 2025

You can rename it to DefaltTightBendInnerRadius

@be5invis be5invis merged commit d540038 into be5invis:dev Nov 8, 2025
5 checks passed
@jmcwilliams403 jmcwilliams403 deleted the DiagTail branch November 8, 2025 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants