Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inconsistent uni309B-V and uni309C-V glyph position between Serif and Sans #157

Closed
3 tasks done
trueroad opened this issue Jan 29, 2022 · 1 comment
Closed
3 tasks done

Comments

@trueroad
Copy link

Prerequisites

  • If you are reporting an issue that affects glyphs for characters for a particular region or regions, did you verify that the characters are within the supported scope of the region or regions? This generally means GB 18030 or Tōngyòng Guīfàn Hànzìbiǎo (通用规范汉字表) for China, Big Five or CNS 11643 Planes 1 & 2 for Taiwan, HKSCS-2016 for Hong Kong, the JIS standards for Japan, and KS X 1001 and KS X 1002 for Korea.
  • Did you thoroughly search the open and closed issues to avoid reporting a duplicate issue?
  • Did you go through the official font readme file to better understand the scope of the project, to include the Known Issues section at the very end?

Description

In SourceHanSerif 2.001, the vertical glyphs for U+309B and U+309C (i.e. uni309B-V and uni309C-V) have inconsistent positions with SourceHanSans 2.004.

In SourceHanSans, the vertical glyphs are placed in the right bottom corner and the horizontal glyphs are placed in the left top corner.
However, in SourceHanSerif, both vertical and horizontal glyphs are same placed in the left top corner.

inconsistent

@punchcutter
Copy link
Member

Source Han Serif 2.001, Source Han Serif 2.002, Source Han Sans 2.004

Horizontal (no change)
309B-309C-horizontal

Vertical (Serif position fixed and matches Sans)
309B-309C-vertical

trueroad added a commit to trueroad/HaranoAjiFonts-generator that referenced this issue Oct 9, 2023
SourceHanSerif 2.002 fixes
the inconsistent uni309B-V and uni309C-V glyph positions.
adobe-fonts/source-han-serif#157

So this commit reverts its workaround
a46765c
.
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

No branches or pull requests

2 participants