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

Fix ligature issue #10

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

Conversation

ZoeyWoohoo
Copy link

According to this issue, remove the ligature fi and fl with FontForge manually.

MonacoLigatureFix

@Karmenzind
Copy link
Owner

Thanks. I need to check if this happens on other systems.

@laris
Copy link

laris commented Oct 8, 2023

I install the fonts but got same issue in Windows 11 Terminal.
image

@laris
Copy link

laris commented Oct 8, 2023

I patch font with latest script but still got same issue in Windows 11, macOS show fine.

https://github.com/ryanoasis/nerd-fonts/releases/tag/v3.0.2
https://github.com/ryanoasis/nerd-fonts#option-9-patch-your-own-font
Original font come from macOS Sonoma 14.0, /System/Library/Fonts/Monaco.ttf, version: 17.0d1e5

Looks like in 3.0.1 have Ubuntu issue:

@Karmenzind
Copy link
Owner

I don't have a Windows to test it. Sorry.
Maybe you can try https://github.com/thep0y/monaco-nerd-font to see if it fixed your problem.

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.

3 participants