search

rsms / inter

The Inter font family
https://rsms.me/inter/
SIL Open Font License 1.1
17.72k stars 398 forks source link

cv06 (stylized "r") too tall for the Display inter font #485

Closed michealp-coder closed 2 years ago

michealp-coder commented 2 years ago

Describe the bug The stylized "r" (cv06) is too tall.

To Reproduce Steps to reproduce the behavior:

  1. https://rsms.me/inter/lab/?family=display
  2. Type the word "Guarn"
  3. Increase the Size (font size on the panel) to "70" or larger
  4. Toggle cv06 (on the side panel) to on. Note how the letter "r" is much much taller than the letter "u", "a" or "n".

Expected behavior The stylized letter "r" (cv06) to be at the same height as non-stylized letters.

Environment

kenmcd commented 2 years ago

This does not appear to be fixed in the most current beta. Which you can get here: https://github.com/rsms/inter/discussions/463

@rsms The cv06 version of the r is still too tall in the latest beta when the opsz is set to the max, and the weight is set to 400. So it appears the Display master may be the problem.

rsms commented 2 years ago

It's a TODO for Display. You can see what glyphs are still work in progress for display by filtering the Glyphs document on the tag "TODO-opsz".

https://github.com/rsms/inter/blob/d4b714e2c2779d7226d6dab89ae923f4e2eb5cfe/src/Inter.glyphspackage/glyphs/r.1.glyph#L701

rsms commented 2 years ago

/r.1 will be removed in the next release

michealp-coder commented 1 year ago

Hi @rsms

I don’t believe this has been fixed.

How to reproduce:

  1. Go to https://rsms.me/inter/lab/
  2. Set font to Display
  3. Set weight to 300+ (no bug <300 weights)
  4. Set font size to 70+ (large enough so you can easily see letter height)
  5. Set cv06 to on
  6. Type the word “Guarn”
  7. You’ll see how “r” is too tall

Thanks in advance for all you do. Truly appreciate this great typeface you’ve made.

kenmcd commented 1 year ago

This has been fixed in the v4 dev version (v4 beta 7 is fine). What is on the demo page above is the old v3.19.

Current development is focused on v4. So it is not likely v3.19 will be updated. And it is probably too early to put v4 on the demo page.