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

Correct diacritic rotation for Czech and Slovak symbols #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

lilmayu
Copy link

@lilmayu lilmayu commented Apr 24, 2022

These letters were fixed: Čč Ď Ěě Ňň Řř Šš Ť Žž and also U+02C7 was rotated as well.

The only characters which have incorrect diacritics are ď and ť - could not fix those since I am no expert on fonts and I could not get it working and I've spent already too much time on fixing Comic sans-like font.

This PR partially closes issue #16

An example case of č:
Old:
obrazek
New:
obrazek

If you want to use this fix before it would be merged into master (I suspect that it won't be merged any time soon), you can download .ttf here or .otf here

@AndreaCatania
Copy link

Hey @shannpersand. Are you still maintaining this font? There are few interested folks here, it would be nice if you could review and merge these changes, in any case thanks a lot for this font, it's really cool!!

@jesusmgg
Copy link

jesusmgg commented Feb 16, 2023

Hello, I'm maintaining an updated fork of this while we wait to hear news from the original version.
If you can you can submit your PRs there, I'll gladly accept them: https://github.com/jesusmgg/comic-shanns-mono

@Finii
Copy link

Finii commented Mar 18, 2024

@jesusmgg There is a PR similar to this waiting in your repo ;-)

@lilmayu very good fix does not attend the bugs in dcaron and tcaron, the caron should be expressed as apostrophe like thing. That has been addressed here.

image

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.

None yet

4 participants