Skip to content

Expand font glyph coverage for Latin scripts - #192

Open
Yesanith wants to merge 1 commit into
Infiziert90:mainfrom
Yesanith:main
Open

Expand font glyph coverage for Latin scripts#192
Yesanith wants to merge 1 commit into
Infiziert90:mainfrom
Yesanith:main

Conversation

@Yesanith

Copy link
Copy Markdown

Replace hardcoded French/Romanian glyph additions with a full Latin Extended-A range import (U+0100–U+017F), then explicitly add Romanian comma-below letters (ȘșȚț) from Latin Extended-B. This broadens support for commonly typed accented characters like Turkish, Polish, Czech, Hungarian, and Baltic letters.

Replace hardcoded French/Romanian glyph additions with a full Latin Extended-A range import (U+0100–U+017F), then explicitly add Romanian comma-below letters (ȘșȚț) from Latin Extended-B. This broadens support for commonly typed accented characters like Turkish, Polish, Czech, Hungarian, and Baltic letters.
Copilot AI lite review requested due to automatic review settings August 21, 2026 07:49

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@Yesanith

Copy link
Copy Markdown
Author

Worth mentioning this helps beyond Chat 2 itself. it also unblocks a fix in
ChatTranslated: kelvin124124/ChatTranslated#104

That plugin prints translations into the game's chat log, so translating into Turkish or
Polish comes out as =unu g=rd=m, since the game font has no glyph for ş. Its fallback
is to swap those letters for the closest match (şs) readable, but the spelling is
gone. Chat 2 draws the log with a configurable font instead, and the default
(Noto Sans CJK) has these letters, so once this PR is in, the plugin can send them through
untouched and those users see the real spelling.

So merging this gets correct characters in translated messages for Chat 2 users on the
default font setup. Nothing needed on your side beyond this PR.

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

Successfully merging this pull request may close these issues.

2 participants