search

StefanPeev / Common-Serif

The Common Serif project is based on Libertinus Serif.
SIL Open Font License 1.1
55 stars 3 forks source link

Kerning of lowercase p is a little off #45

Closed waldyrious closed 10 months ago

waldyrious commented 1 year ago

Note how the "p" is too close to the "i" in the word "episode" from this paragraph of a random Wikipedia article:

Libertinus Serif:

Screen Shot 2023-06-05 at 11 39 55

Common Serif:

Screen Shot 2023-06-05 at 11 39 44

Detailed comparison:

image

StefanPeev commented 1 year ago

@waldyrious Thank you for the note, I'll check the issue.

StefanPeev commented 1 year ago

Fixed. Need to be checked.

waldyrious commented 1 year ago

Hmm... the kerning after the "p" seems better, but the kerning before it continues to be wider than Libertinus Serif, and IMO the latter works better.

Common Serif before (v.1.027):

Common Serif now (v.1.028): common-serif

Libertinus Serif:

(The difference in the images is clearer in 100% zoom.)

StefanPeev commented 11 months ago

@waldyrious In version 1.031 I made the horizontal metrics in "episode" to be the same as in Libertinus Serif.

StefanPeev commented 10 months ago

@waldyrious I close this issue because the problems are solved in v.1.033.