Comic Shanns is using circumflex instead of caron/wedge

[freed00m]

🗹 Requirements

• I have searched the issues for my issue and found nothing related and/or helpful
• I have searched the FAQ for help
• I have searched the Wiki for help

🎯 Subject of the issue

Experienced behavior:

Czech languages use carons for diacritics charaters.

The following sentence contains all of Czech diacritics
"Příliš žluťoučký kůň úpěl ďábelské ódy"
Shows like this

Expected behavior:
řšžčěňŘŠŽČĚŤĎŇto have caron
ťď is rarity to have caron modified to apostrophe-like stroke instead of caron

Example symbols:

🔧 Your Setup

ComicShannsMonoNerdFontMono-Regular.otf
from Arch linux repository

Using foot terminal with sway environment running linux.

| Installed package info:
| Name : otf-comicshanns-nerd
| Version : 3.1.1-1

[Finii]

Uups that looks indeed strange.

Concentrating on one glyph, ž, codepoint 17e.

Comparing here with Cascadia Code, the error is already in the source font:

And they even drew the glyph completely instead of just combining a regular z with a regular caron, like Cascadia does:

I guess this at least fixes the non-combining chars, which would make it far easier to fix your problem:

• Replace duplicated glyphs with References and Fix carons jesusmgg/comic-shanns-mono#7

I will work a bit on this, maybe we can incorporate the fix upstream.

[Finii]

Even the stand alone caron is not a caron.
The Breve is ok, though.

Wikipedia explaining how caron should look like

Caron next to Circumflex in Comic Shanns

---

Then I found this here [1]:

which is a Nerd Font patch of Comic Mono, with the correct Caron, BUT the original Comic Mono [2] does not have it?!
There is no documentation on that patch, and to be honest the diacritics look rather out of place.

[1] Patched Comic Mono with diacritics: https://github.com/mavonarx/ComicMonoFonts
[2] Original Comic Mono: https://github.com/dtinth/comic-mono-font

[freed00m]

Is there a way for you to patch it on your end?

It has been reported upstream in 2021, maybe font abandoned.

[Finii]

Is the list given in the issue description with the char list (expected behavior) complete? Sent from Nine

…

________________________________ From: Antonín Dach ***@***.***> Sent: Sunday, March 17, 2024 22:26 To: ryanoasis/nerd-fonts Cc: Fini; Comment Subject: Re: [ryanoasis/nerd-fonts] Comic Shanns is using circumflex instead of caron/wedge (Issue #1492) Is there a way for you to patch it on your end? It has been reported upstream in 2021, maybe font abandoned. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you commented.Message ID: ***@***.***>

[freed00m]

Yes,

but to be sure, here you have all Czech ones

áéěíýóúůžščřďťň
ÁÉĚÍÝÓÚŮŽŠČŘĎŤŇ

I also checked Slovak diacritics and there is no extra letters Czechs don't use that would be broken.

[Finii]

Note to self

U+00E1 U+00E9 U+011B U+00ED áéěí
U+00FD U+00F3 U+00FA U+016F ýóúů
U+017E U+0161 U+010D U+0159 žščř
U+010F U+0165 U+0148 U+00C1 ďťňÁ
U+00C9 U+011A U+00CD U+00DD ÉĚÍÝ
U+00D3 U+00DA U+016E U+017D ÓÚŮŽ
U+0160 U+010C U+0158 U+010E ŠČŘĎ
U+0164 U+0147 ŤŇ

[Finii]

Comic Shanns repos

• https://github.com/shannpersand/comic-shanns (the source)
• https://github.com/dtinth/comic-mono-font (comic mono fork, based on shannpersand's v1)
• https://github.com/jesusmgg/comic-shanns-mono (our upstream, based on shannpersand's v2)

[Finii]

ťď is rarity to have caron modified to apostrophe-like stroke instead of caron

U+010F U+0165

Seems we should remove the 'real' caron and add an apostrophe thing?

Noto does this for example. But on the other hand this violates 'monospaced'.

[Finii]

Iosevka Term...

OK, we will do the same.

[Finii]

@freed00m These look about right?

[freed00m]

@Finii I think the ď acute like stroke should not be touching, it looks weird like a completely new character.
I think going a bit out of monospace should be acceptable.

I tried to look up fonts and can't find any that would have acute like stroke touching the letter.

[Finii]

I think going a bit out of monospace should be acceptable.

Ok, fixing

[Finii]

ok?

[freed00m]

Yes,

Though I am not sure if the view is not cut off top and bottom for the uppercase characters, but if it's there and just violating top bottom line space then it's looking good.

[Finii]

That is just an artifact from how fontforge determines what to show in this preview.

Thanks, I will merge.

[Finii]

Though I am not sure if the view is not cut off top and bottom for the uppercase characters, but if it's there and just violating top bottom line space then it's looking good.

Ah well, that depends on platform and we fix it before we patch:

# WIN line metrics obviously wrong, as Nerd Fonts sometimes prefers that, fix it already
font.os2_winascent = font.os2_typoascent
font.os2_windescent = -font.os2_typodescent
font.os2_use_typo_metrics = 1

font.generate('ComicShannsMono-Regular.otf')

Excerpt of https://github.com/ryanoasis/nerd-fonts/blob/master/src/unpatched-fonts/ComicShannsMono/make_bold

and later

DEBUG: Naming mode 1
DEBUG: Monospace check: Panose is invalid ([0, 0, 0, 0, 0, 0, 0, 0, 0, 0]); glyph-width-mono True
INFO: Setting Panose 'Family Kind' to 'Latin Text and Display' (was 'Any')
INFO: Setting Panose 'Proportion' to 'Monospaced' (was 'Any')
INFO: Redistributing line gap of 200 (100 top and 100 bottom)
DEBUG: Extended glyphs wider bounding box than basic glyphs
DEBUG: Font has negative right side bearing in extended glyphs
DEBUG: Final font cell dimensions 550 w x 1200 h

[github-actions]

This issue has been automatically locked since there has not been any recent activity (i.e. last half year) after it was closed. It helps our maintainers focus on the active issues. If you have found a problem that seems similar, please open a new issue, complete the issue template with all the details necessary to reproduce, and mention this issue as reference.