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

Add Serbian, Macedonian and related characters #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Strahinja
Copy link

@Strahinja Strahinja commented May 3, 2024

Continuing the discussion from fcambus#38, I updated all the .bdf files from your fork with the characters specific to Serbian and Macedonian, as well as added some more characters:

0462 CYRILLIC CAPITAL LETTER YAT
0463 CYRILLIC SMALL LETTER YAT
2013 EN DASH
2014 EM DASH
201E DOUBLE LOW-9 QUOTATION MARK

the existing punctuation characters for quotes were not consistently present in all font files, so I added them where needed.

The second commit in this PR contains a patch file patch/serbian-cyrillic.patch and adds the target sr-cyr to the Makefile. This patch file changes the look of several Cyrillic characters so they better fit Serbian typographic standards.

The way the patch would work is this: after cloning the git repository, user would issue the commands:

$ make sr-cyr
$ make

to generate the font files with the Cyrillic characters adjusted to follow Serbian typography. This requires the program patch(1). If the characters following Russian typography are desired, the above commands would of course not be needed.

If this sounds too complicated, or if you would like to instead use just the font with the Cyrillic characters following Serbian typography, I can also supply that case as separate PR.

@Strahinja
Copy link
Author

Strahinja commented May 29, 2024

If there is anything that you think should be added or changed in this PR, feel free to comment. I've seen that upstream has a PR which overlaps with this PR, so if there are any potential conflicts, I'm open to working on resolving them.

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.

1 participant