adobe-fonts / adobe-fonts/source-han-sans

Repeated contours in glyphs

Open
#463 2 comments 0 reactions 0 assignees View on GitHub
fixed for next version
Dominant language
Python
Stars
17.2k
Forks
1.4k
PR merge metrics
No merged PRs in 30d

Description

In this issue “repeated contours” refers to at least two contours that are completely overlapped. I believe they serve no real purpose and are inadvertently duplicated. After all, static fonts will have these overlapped contours removed during build. While they don’t cause usage problems and are certainly not high priority, it may still be worth fixing them someday so that the problem won’t propagate to new glyphs added in the future due to copy and paste.

![image](https://github.com/adobe-fonts/source-han-sans/assets/959433/00fb2761-4282-4ddd-a25e-208f27bb9d89)

The following list shows repeated contours as groups. For instance, [1,3] means contours 1 and 3 are repeated (entirely overlapped), and either one of them can be removed.

I first spotted 2 pairs of repeated contours in 肧 (uni80A7-HK), and decided to write a program to find glyphs with similar issue. So the list should be complete.

| Glyph | Repeated contour groups
| ----------------- | -----------------
| uni2600 | (The whole glyph)
| uni3229-JP | [1,3];[2,4]
| uni3345 | [4,6];[5,7]
| uni465B-CN | [4,16]
| uni5065uE0101-JP | [11,12]
| uni56A5-TW | [1,14]
| uni5964-JP | [1,12]
| uni80A7-HK | [1,3];[2,4]
| uni98BC-HK | [5,19]
| uniAEF4 | [1,8]
| uniB1F5 | [1,7]
| uniB367 | [5,7];[6,8]
| uniC254 | [3,10];[4,11]
| uniC3A4 | [1,10];[2,11]
| uniCAC5 | [3,9];[4,10]
| uniCAC6 | [3,9];[4,10]
| uniCAC7 | [3,9];[4,10]
| uniCAC8 | [3,9];[4,10]
| uniCAC9 | [3,9];[4,10]
| uniCACA | [3,9];[4,10]
| uniCACB | [3,9];[4,10]
| uniCACE | [3,9];[4,10]
| uniCAD5 | [3,9];[4,10]
| uniCAD7 | [3,9];[4,10]
| uniCB34 | [3,9];[4,10]
| uniCB35 | [1,7];[2,8]
| uniCB36 | [3,9];[4,10]
| uniCB37 | [1,7];[2,8]
| uniCB38 | [1,7];[2,8]
| uniCB39 | [1,7];[2,8]
| uniCB3A | [1,7];[2,8]
| uniCB3B | [1,7];[2,8]
| uniCB3E | [1,7];[2,8]
| uniCB43 | [1,7];[2,8]
| uniCB45 | [2,8];[3,9]
| uniCB47 | [1,7];[2,8]
| uniCB51 | [2,8];[3,9]
| uniCB52 | [2,8];[3,9]
| uniCB53 | [1,7];[2,8]
| uniCB54 | [2,8];[3,9]
| uniCB55 | [2,8];[3,9]
| uniCB56 | [2,8];[3,9]
| uniCB57 | [2,8];[3,9]
| uniCB5A | [2,8];[3,9]
| uniCB5F | [2,8];[3,9]
| uniCB61 | [2,8];[3,9]
| uniCB63 | [2,8];[3,9]
| uniCBC0 | [4,10];[5,11]
| uniCBC1 | [2,8];[3,9]
| uniCBC2 | [2,8];[3,9]
| uniCBC3 | [1,7];[2,8]
| uniCBC4 | [2,8];[3,9]
| uniCBC5 | [2,8];[3,9]
| uniCBC6 | [2,8];[3,9]
| uniCBCA | [2,8];[3,9]
| uniCBCF | [2,8];[3,9]
| uniCBD1 | [2,8];[3,9]
| uniCBD3 | [2,8];[3,9]
| uniCBDC | [4,10];[5,11]
| uniCBDD | [2,8];[3,9]
| uniCBDE | [2,8];[3,9]
| uniCBDF | [1,7];[2,8]
| uniCBE0 | [2,8];[3,9]
| uniCBE1 | [2,8];[3,9]
| uniCBE2 | [2,8];[3,9]
| uniCBE3 | [2,8];[3,9]
| uniCBE6 | [2,8];[3,9]
| uniCBEB | [2,8];[3,9]
| uniCBED | [2,8];[3,9]
| uniCBEF | [2,8];[3,9]

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.