Wrong spacing before punctuation characters in Nerd Font version
IgorKrupenja opened this issue · 17 comments
- Install Monego Nerd Font on a Mac.
- Enable the font in an app (reproducible at least in iTerm and VSCode).
- Try typing something containing some punctuation, e.g.
"This is a test:'.
Notice how : and ' look as if there is a space in front of them even though there isn't.
Hey @krupenja. The issue seems to stem from Ligaturizer. Does this version without ligatures work for you?
No, sorry, the problem persists even with the version without ligatures from your comment.
Reporting in with the same issues.
Liga Monego/Monego Nerd Font:
Monego:
Notice the different kerning in erv, . and ;.
Otherwise I really dig your font. Have you been able to figure a fix for this yet @cseelus?
@prashaantt Currently I'm not using Monego, so I can't test, but PRs are welcome.
Did anyone ever find a fix for this because I just downloaded them and I am having the same issue
Any movement on this?
I've just recreated the Nerd Font version with different ligature settings, as the kerning issues are caused by Ligaturizer (see my comment above). Still not ideal, but unless the Ligaturizer kerning issue is resolved, there is not much I can do about it.
Hi I am going to try look into this issue myself, but I need to ask a few questions to kind of know where to start. I see in the linked comment here that the issue seems to be from using the ligatures. Am I correct in thinking that ligatures are the things where a syntax such as => gets turned into one arrow symbol? If so, why does the nerd font that doesn't have the ligatures have the issue? Or does the nerd font automatically create the ligatures?
Hi @matteematt, the Nerd Font version is based on the version with the ligatures and yes, those are special characters that replace => and others.
Did you test the new version?
@cseelus yeah unfortunately it still seems like an issue. It seems to affect hardly any of the symbols but unfortunately with it affecting the . symbol which is used loads in programming it makes it look very strange
I don't know anything about fonts, ligatures, and kerning. So going to have to learn about it and see if I can come up with anything. Will be unlikely but I really like the font so I think it is worth a go
I'm on macOS and the dot seems to be looking good now:
Here are some other programming fonts, for comparision:
Menlo (macOS monospace font that followed after Monaco)
PT Mono
Still, the letter-spacing/kerning isn't optimal, so if you manage to improve upon the current state with Ligaturizer and Nerd Fonts, I'd be more than happy to merge it.
I haven't checked MacOS (can check tomorrow once I use my work laptop). This is the issue I am seeing on my PC though (Arch Linux + Alacritty)
You can really notice the issue with the placeholder symbols I am using to represent spaces in the yaml files, but you can also see in the Haskell file that the dot on line 29 is very obviously mismatched (same with the # and some other symbols).
I have made some progress and I have tested it on machine and it looks good to me. Here are the same two examples.
I was going to make a draft PR so you could check them on your machine (or so I could check them on my Mac) but it seems I don't have rights to push to this repo. Is that something you need to allow, or do I need to make a PR another way? Sorry I haven't really contributed to any public repos before
Another issue I would like to fix too is the font underlining. Here is a font with underlining that you can see goes underneath the text
But the underlining seems to intersect with the text in this font slightly
Fixed that too I think
@matteematt, seems you made great progress improving the font. I’ll check it out when I’m back in my computer.
To make a PR in a public Github repo, you have to follow these steps: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork
@matteematt I‘ve added you as a collaborator, so you should be able to open a PR here directly.
@cseelus I have created a PR with the fixed nerd font. Feel free to test it out and let me know any issues. I have now tested on MacOS too and it seems to work fine. However, I have noticed the ligature symbols themselves are still not fixed - more info on the PR. I personally don't use ligatures though