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

Glyphs of U+11ED ᇭ, U+D7F5 ퟵ, U+D7F6 ퟶ #6

Closed
acuteaccent opened this issue Apr 4, 2017 · 17 comments
Closed

Glyphs of U+11ED ᇭ, U+D7F5 ퟵ, U+D7F6 ퟶ #6

acuteaccent opened this issue Apr 4, 2017 · 17 comments
Assignees

Comments

@acuteaccent
Copy link

Yesieung (ㆁ), not ieung (ㅇ). This includes .tjmo0[1-4] glyphs of those three hangul jamo characters as well.

@kenlunde kenlunde self-assigned this Apr 4, 2017
@kenlunde
Copy link
Contributor

kenlunde commented Apr 4, 2017

Thank you. I will report this issue to Sandoll's designer for consideration.

@kenlunde kenlunde added bug and removed investigating labels Apr 5, 2017
@kenlunde
Copy link
Contributor

kenlunde commented Apr 5, 2017

Sandoll's designer confirmed this bug, which affects the following 15 glyphs: uni11ED, uni11ED.tjmo01 through uni11ED.tjmo04 (4), uniD7F5, uniD7F5.tjmo01 through uniD7F5.tjmo04 (4), uniD7F6, and uniD7F6.tjmo01 through uniD7F6.tjmo04 (4). The first update will reflect fixed versions of these glyphs.

@acuteaccent
Copy link
Author

acuteaccent commented Apr 6, 2017

BTW, under which condition is tjmo01, tjmo02, tjmo03, or tjmo04 displayed? I want to know what a trailing (final) consonant needs to be preceded by in each condition in order to have its glyph replaced with one of tjmo0[1-4].
If I know this, it will be much easier when testing the font.

@kenlunde
Copy link
Contributor

kenlunde commented Apr 6, 2017

[uni1161.vjmo01 uni1163.vjmo01 uni116A.vjmo01 uni1184.vjmo01 uni1189.vjmo01 uni118E.vjmo01 uni11A6.vjmo01 uniD7B2.vjmo01 uniD7B9.vjmo01 uniD7C5.vjmo01] + any T = tjmo01

[uni1165.vjmo01 uni1167.vjmo01 uni116C.vjmo01 uni116F.vjmo01 uni1171.vjmo01 uni1174.vjmo01 uni1175.vjmo01 uni117F.vjmo01 uni1186.vjmo01 uni1188.vjmo01 uni118F.vjmo01 uni1191.vjmo01 uni1194.vjmo01 uni119D.vjmo01 uni119F.vjmo01 uni11A1.vjmo01 uni11A5.vjmo01 uniD7B0.vjmo01 uniD7B4.vjmo01 uniD7B5.vjmo01 uniD7BA.vjmo01] + any T = tjmo02

[uni1162.vjmo01 uni1164.vjmo01 uni1166.vjmo01 uni1168.vjmo01 uni116B.vjmo01 uni1170.vjmo01 uni1180.vjmo01 uni1181.vjmo01 uni1185.vjmo01 uni118A.vjmo01 uni118C.vjmo01 uni1190.vjmo01 uni1192.vjmo01 uni1198.vjmo01 uni1199.vjmo01 uni11A7.vjmo01 uniD7B3.vjmo01 uniD7B6.vjmo01 uniD7B7.vjmo01 uniD7BB.vjmo01 uniD7BE.vjmo01 uniD7BF.vjmo01 uniD7C0.vjmo01 uniD7C4.vjmo01 uniD7C6.vjmo01] + any T = tjmo03

[uni1160.vjmo01 uni1169.vjmo01 uni116D.vjmo01 uni116E.vjmo01 uni1172.vjmo01 uni1173.vjmo01 uni1176.vjmo01 through uni117E.vjmo01 uni1182.vjmo01 uni1183.vjmo01 uni1187.vjmo01 uni118B.vjmo01 uni118D.vjmo01 uni1193.vjmo01 uni1195.vjmo01 through uni1197.vjmo01 uni119A.vjmo01 through uni119C.vjmo01 uni119E.vjmo01 uni11A0.vjmo01 uni11A2.vjmo01 through uni11A4.vjmo01 uniD7B1.vjmo01 uniD7B8.vjmo01 uniD7BC.vjmo01 uniD7BD.vjmo01 uniD7C1.vjmo01 through uniD7C3.vjmo01] + any T = tjmo04

(Note that the fourth set, for the tjmo04 glyphs, includes ranges.)

@kenlunde
Copy link
Contributor

kenlunde commented Apr 6, 2017

To follow up on this, in addition to fixing the 15 glyphs that were originally reported, Sandoll's designer decided to adjust the following four additional glyphs: uni118C.vjmo01, uni1190.vjmo01, uni1192.vjmo01, and uni1112uni119Euni11D9. The last glyph is one of the 500 high-frequency archaic hangul syllables.

@acuteaccent
Copy link
Author

acuteaccent commented Apr 6, 2017

tjmo01: when preceded by ᅡ, ᅣ, ᅪ, ᆄ, ᆉ, ᆎ, ᆦ, ힲ, ힹ, ퟅ
tjmo02: when preceded by ᅥ, ᅧ, ᅬ, ᅯ, ᅱ, ᅴ, ᅵ, ᅿ, ᆆ, ᆈ, ᆏ, ᆑ, ᆔ, ᆝ, ᆟ, ᆡ, ᆥ, ힰ, ힴ, ힵ, ힺ
tjmo03: when preceded by ᅢ, ᅤ, ᅦ, ᅨ, ᅫ, ᅰ, ᆀ, ᆁ, ᆅ, ᆊ, ᆌ, ᆐ, ᆒ, ᆘ, ᆙ, ᆧ, ힳ, ힶ, ힷ, ힻ, ힾ, ힿ, ퟀ, ퟄ, ퟆ
tjmo04: when preceded by ᅠ (jungseong filler), ᅩ, ᅭ, ᅮ, ᅲ, ᅳ, ᅶ, ᅷ, ᅸ, ᅹ, ᅺ, ᅻ, ᅼ, ᅽ, ᅾ, ᆂ, ᆃ, ᆇ, ᆋ, ᆍ, ᆓ, ᆕ, ᆖ, ᆗ, ᆚ, ᆛ, ᆜ, ᆞ, ᆠ, ᆢ, ᆣ, ᆤ, ힱ, ힸ, ힼ, ힽ, ퟁ, ퟂ, ퟃ

Thank you for listing those conditions. Do those apply to Source Han Sans as well, without any differences?

@kenlunde
Copy link
Contributor

kenlunde commented Apr 6, 2017

Yes, Source Han Sans is the same.

@acuteaccent
Copy link
Author

Just in case, can you please list the conditions for ljmo0[1-6] as well? I just want to check whether what I thought is the same as the ones defined in the font.

@acuteaccent
Copy link
Author

acuteaccent commented Apr 6, 2017

The one about uni1112uni119Euni11D9 is interesting. I wonder how ᄒᆞᇙ became 흐ᇙ. Perhaps you can keep the current glyph (흐ᇙ) and rename that uni1112uni1173uni11D9, and add a new glyph for uni1112uni119Euni11D9. Rather than getting rid of the well-designed 흐ᇙ glyph (rather than the effort used to design the 흐ᇙ glyph getting wasted), I suggest keeping it while adding the correct glyph for ᄒᆞᇙ.

@kenlunde
Copy link
Contributor

kenlunde commented Apr 7, 2017

I received the 19 updated glyphs from Sandoll's designer and processed them this morning. The PDF synopsis below shows the ExtraLight, Medium, and Heavy weights, and the combining glyphs are shown before I shifted them to the left and zero-out their horizontal advances (so that they are easier to see here). Also, the CIDs don't correspond to Source Han Serif CIDs, but are for the back-end source data.

fixed-shserif-hangul-jamo

@kenlunde
Copy link
Contributor

kenlunde commented Apr 7, 2017

Here are the CID to working glyph name correspondences:

238 uni11ED
12745 uni11ED.tjmo01
12882 uni11ED.tjmo02
13019 uni11ED.tjmo03
13156 uni11ED.tjmo04
506 uniD7F5
12806 uniD7F5.tjmo01
12943 uniD7F5.tjmo02
13080 uniD7F5.tjmo03
13217 uniD7F5.tjmo04
507 uniD7F6
12807 uniD7F6.tjmo01
12944 uniD7F6.tjmo02
13081 uniD7F6.tjmo03
13218 uniD7F6.tjmo04
12530 uni118C.vjmo01
12534 uni1190.vjmo01
12536 uni1192.vjmo01
13487 uni1112uni119Euni11D9

@kenlunde
Copy link
Contributor

kenlunde commented Apr 7, 2017

Any L + [uni1161 through uni1168 uni1175 uni1198 uni1199 uni119D uni11A5 uniD7BE through uniD7C0 uniD7C4] + any T = ljmo01
Any L + [uni1160 uni1169 uni116D uni116E uni1172 uni1173 uni1182 uni1183 uni1187 uni118D uni1193 uni1195 uni1196 uni119E uni11A0 uni11A2 uniD7B1 uniD7B8 uniD7BC] + any T = ljmo02
Any L + [uni116A through uni116C uni116F through uni1171 uni1174 uni1176 through uni1181 uni1184 through uni1186 uni1188 through uni118C uni118E through uni1192 uni1194 uni1197 uni119A through uni119C uni119F uni11A1 uni11A3 uni11A4 uni11A6 uni11A7 uniD7B0 uniD7B2 through uniD7B7 uniD7B9 through uniD7BB uniD7BD uniD7C1 through uniD7C3 uniD7C5 uniD7C6] + any T = ljmo03

lmjo04 through ljmo06 are identical to lmjo01 through ljmo03, respectively, except that the "any T" portion is removed from their context.

@acuteaccent
Copy link
Author

acuteaccent commented Apr 8, 2017

About yesieung- jamo: Great.

For those three vjmo01 glyphs: The new design is better, as they are followed by a T. Under the old design, it is hard to see the short horizontal strokes.

Conditions for ljmo: Thank you. They are the same as I thought.

@acuteaccent
Copy link
Author

Note to self (and to others who want to test archaic hangul jamo in Source Han Sans and Source Han Serif)

ljmo01: when followed by [ᅡ, ᅢ, ᅣ, ᅤ, ᅥ, ᅦ, ᅧ, ᅨ, ᅵ / ᆘ, ᆙ, ᆝ, ᆥ, ힾ, ힿ, ퟀ, ퟄ] + any T
ljmo02: when followed by [ᅩ, ᅭ, ᅮ, ᅲ, ᅳ / ᅠ (jungseong filler), ᆂ, ᆃ, ᆇ, ᆍ, ᆓ, ᆕ, ᆖ, ᆞ, ᆠ, ᆢ, ힱ, ힸ, ힼ] + any T
ljmo03: when followed by [ᅪ, ᅫ, ᅬ, ᅯ, ᅰ, ᅱ, ᅴ / ᅶ, ᅷ, ᅸ, ᅹ, ᅺ, ᅻ, ᅼ, ᅽ, ᅾ, ᅿ, ᆀ, ᆁ, ᆄ, ᆅ, ᆆ, ᆈ, ᆉ, ᆊ, ᆋ, ᆌ, ᆎ, ᆏ, ᆐ, ᆑ, ᆒ, ᆔ, ᆗ, ᆚ, ᆛ, ᆜ, ᆟ, ᆡ, ᆣ, ᆤ, ᆦ, ᆧ, ힰ, ힲ, ힳ, ힴ, ힵ, ힶ, ힷ, ힹ, ힺ, ힻ, ힽ, ퟁ, ퟂ, ퟃ, ퟅ, ퟆ] + any T
ljmo04: same as ljmo01, but without T
ljmo05: same as ljmo02, but without T
ljmo06: same as ljmo03, but without T

vjmo01: when followed by a T
vjmo02: when not followed by a T

tjmo01: when preceded by [ᅡ, ᅣ, ᅪ / ᆄ, ᆉ, ᆎ, ᆦ, ힲ, ힹ, ퟅ]
tjmo02: when preceded by [ᅥ, ᅧ, ᅬ, ᅯ, ᅱ, ᅴ, ᅵ / ᅿ, ᆆ, ᆈ, ᆏ, ᆑ, ᆔ, ᆝ, ᆟ, ᆡ, ᆥ, ힰ, ힴ, ힵ, ힺ]
tjmo03: when preceded by [ᅢ, ᅤ, ᅦ, ᅨ, ᅫ, ᅰ / ᆀ, ᆁ, ᆅ, ᆊ, ᆌ, ᆐ, ᆒ, ᆘ, ᆙ, ᆧ, ힳ, ힶ, ힷ, ힻ, ힾ, ힿ, ퟀ, ퟄ, ퟆ]
tjmo04: when preceded by [ᅩ, ᅭ, ᅮ, ᅲ, ᅳ / ᅠ (jungseong filler), ᅶ, ᅷ, ᅸ, ᅹ, ᅺ, ᅻ, ᅼ, ᅽ, ᅾ, ᆂ, ᆃ, ᆇ, ᆋ, ᆍ, ᆓ, ᆕ, ᆖ, ᆗ, ᆚ, ᆛ, ᆜ, ᆞ, ᆠ, ᆢ, ᆣ, ᆤ, ힱ, ힸ, ힼ, ힽ, ퟁ, ퟂ, ퟃ]

@acuteaccent
Copy link
Author

acuteaccent commented Apr 8, 2017

Examples for testing
(not using any jamo appearing in the high-frequency archaic 500 hangul syllables)

ljmo01: Lᆘᇫ
ljmo02: Lᆢᇫ
ljmo03: Lᅷᇫ
ljmo04: Lᆘ
ljmo05: Lᆢ
ljmo06: Lᅷ

vjmo01: ᄓVᇫ
vjmo02: ᄓV

tjmo01: ᄓᆄT
tjmo02: ᄓᅿT
tjmo03: ᄓᆀT
tjmo04: ᄓᅷT

Do NOT use these when testing:
ᄀ, ᄁ, ᄂ, ᄃ, ᄄ, ᄅ, ᄆ, ᄇ, ᄈ, ᄉ, ᄊ, ᄋ, ᄌ, ᄍ, ᄎ, ᄏ, ᄐ, ᄑ, ᄒ / ᄠ, ᄡ, ᄢ, ᄧ, ᄩ, ᄫ, ᄭ, ᄯ, ᄲ, ᄶ, ᅀ, ᅌ, ᅘ, ᅙ
ᅡ, ᅢ, ᅣ, ᅥ, ᅦ, ᅧ, ᅨ, ᅩ, ᅪ, ᅬ, ᅭ, ᅮ, ᅯ, ᅱ, ᅲ, ᅳ, ᅴ, ᅵ / ᅶ, ᆈ, ᆑ, ᆒ, ᆔ, ᆞ, ᆡ
ᆨ, ᆫ, ᆮ, ᆯ, ᆰ, ᆲ, ᆳ, ᆷ, ᆸ, ᆺ, ᆼ / ᇇ, ᇙ, ᇝ, ᇢ, ᇰ

@kenlunde
Copy link
Contributor

kenlunde commented Apr 8, 2017

Thank you for confirming the fixes.

@kenlunde
Copy link
Contributor

kenlunde commented Apr 11, 2017

Consolidated with Issue #39.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants