source-serif icon indicating copy to clipboard operation
source-serif copied to clipboard

Is the contour direction of `theta` incorrect/reversed?

Open tamcy opened this issue 1 year ago • 1 comments

Two similarily looking glyphs, theta and fita, are created with a different contour order:

This results different contour order when converting the master .ufo to .pfa with the tx command:

Take this command for instance:

tx -t1 ./source-serif-main/Roman/Masters/text/master_0/SourceSerif_0.ufo /tmp/test.pfa

Here's the result (Left: theta, Right: fita): image

If a font is produced with this source, fita will render correctly, but theta could be incorrect at certain size, probably depending on the rendering engine:

image (Same font but in different size; Word 365 on Windows 11)

Note that fonts released here seems fine. The problem arises when a custom build from the masters is needed.

I checked several glyphs that are similar in composition (Oslash, oslash, theta, Fita, fita, Obarcyr, obarcyr, Fita.sc, Obarcyr.sc), with theta's contour direction being the only exception, thus I tend to think this is something to do with the source, not the tx script, so I am reporting it here. Thanks in advance!

tamcy avatar Aug 19 '23 07:08 tamcy

안녕하세요! Safari와 Chrome에서 가변 다이나믹 서브셋으로 제공하는 Pretendard를 썼을 때 이상 증상을 발견하지 못했습니다. 자세한 사용 환경을 알 수 있을까요?

  • macOS, 브라우저 버전
  • 웹폰트 종류(가변 다이나믹 서브셋 등)
  • 증상 재현 가능한 예제 파일 또는 CSS 속성

제가 테스트해본 환경은 다음과 같습니다:

  • arm64, macOS 14.2.1, Safari 17.2.1, Chrome 120.0.6099.234, 가변 다이나믹 서브셋 사용

orioncactus avatar Jan 24 '24 12:01 orioncactus

@orioncactus 스크린샷을 보니 Flutter화면으로 보이고, 저에게도 같은 증상이 있었습니다.

ksiom avatar Jan 30 '24 09:01 ksiom

저도 Flutter에서 재현됩니다.

kms0219kms avatar Mar 29 '24 10:03 kms0219kms

저도 플러터로 개발중이고 pretendardVaribale.ttf 파일 사용중입니다. ios에서는 이런 증상이 없는데 android에서 깨지는 현상이 발생합니다. 굵은글자(700w)에서 발생하고 있어요.

eunseoDev avatar Apr 30 '24 08:04 eunseoDev

모두 의견을 남겨주셔서 감사합니다! 이번주에 한번 확인해보고 알려드리겠습니다.

orioncactus avatar May 02 '24 15:05 orioncactus

@orioncactus flutter에서 적용해 사용하고 있습니다. TextStyle 적용할 때 fontWeight 말고

fontVariations: [ FontVariation('wght', 700), ],

이런 식으로 적용하면 해결되긴 합니다.

kjw3676 avatar May 13 '24 08:05 kjw3676

같은 이슈를 겪고 있습니다.

image

RealYoungk avatar Aug 02 '24 02:08 RealYoungk

약 3개월이 지났는데요, 현재도 같은 문제가 발생하고 있습니다.

혹시 해당 이슈와 관련하여 수정 진행중인 부분이 있는지 확인 요청드려도 될까요?

emotionalboySY avatar Aug 10 '24 13:08 emotionalboySY