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

Update to FiraCode 2.0 broke some ligatures #82

Closed
hohmannr opened this issue Mar 13, 2020 · 8 comments
Closed

Update to FiraCode 2.0 broke some ligatures #82

hohmannr opened this issue Mar 13, 2020 · 8 comments
Labels

Comments

@hohmannr
Copy link

hohmannr commented Mar 13, 2020

Hello, thanks for your work!
I saw you have just updated the code to use Fira Code 2.0, I guess this is why some of the ligatures are broken when I patch my font.

Following Ligatures are broken for me (other ones work fine; probably not a full list):

  • !=
  • ^=
  • ~@
  • ~=
  • **
  • *>
  • |=
  • :=
  • =:=
  • ==
  • ===
  • ==>
  • =!=
  • =>
  • =>>
  • =<<
  • =/=
  • !==

Thanks for maintaining! Raph

@hohmannr hohmannr changed the title Update to FiraCode 2.0 bgit clone --recurse-submodules -j8roke some ligatures Update to FiraCode 2.0 broke some ligatures Mar 13, 2020
@ToxicFrog
Copy link
Owner

ToxicFrog commented Mar 13, 2020

Which font are you testing? They work for me in Liga Cousine, Liga Hack, and a few others I've tried -- I haven't comprehensively gone through every font, and won't until I'm cutting a release (which won't be until I've merged a few more patches, hopefully sometime this weekend).

Note that if you're currently using the font, you will probably need to restart the program you're testing in before it will probably load all changes to the font file, and depending on your system you might need to restart the entire desktop environment. And yes, this can result in ligatures not properly working until you restart. It's quite frustrating when testing!

@hohmannr
Copy link
Author

hohmannr commented Mar 14, 2020

Note that if you're currently using the font, you will probably need to restart the program you're testing in before it will probably load all changes to the font file, and depending on your system you might need to restart the entire desktop environment.

I'm using kitty version 0.16.0 as Terminal Emulator and i3-gaps as WM. I updated the font-cache and restarted i3 as well rebooted the PC. No change unfortunately.

Which font are you testing?

I'm using a nerd-fonts patched font called Sauce Code Pro (comes from Source Code Pro), patching the same font last week (before Fira 2.0 update) on a different machine worked fine.

They work for me in Liga Cousine, Liga Hack, and a few others I've tried.

Tested Hack, works for me. Tested also the provided Source Code Pro Font, also worked. Probably a bug with Nerd-Fonts?

@hohmannr
Copy link
Author

hohmannr commented Mar 14, 2020

I just patched ligatures for a different font, SF-Mono. Ligatures work fine, but the naming seems buggy. In the patched file names there is the Liga prefix, but patching this font does not create a new name for the font family, so the font Familiy is still "SF Mono" instead of "Liga SF Mono".

When I have more time, I will try to patch this newly ligaturized font with nerd-fonts, if it works and ligatures are still present after the nerd-fonts patch, then the workaround is simple: 1st patch ligatures, then patch nerd-font-icons.

@ToxicFrog
Copy link
Owner

SF Mono naming being wrong is #15 and unrelated to this issue, I think. It's something specific to SF Mono that I still haven't figured out.

@hohmannr
Copy link
Author

Tested it and a workaround is:

  1. patch ligatures
  2. then patch nerd-font-icons

Other way around does not seem to work for me!

Thanks for responding so fast. I have tested this workaround for SF Mono and Source Code Pro. Works both times. Have a great Weekend!

@jonnydee
Copy link

On my Windows 10 system "Liga Hack" doesn't support Fira Code's <==> ligature at least. Didn't try other fonts.

@ToxicFrog
Copy link
Owner

Does the four character ligature <!-- work, or are four-character ligatures in general broken?

<==> and <!-- both work for me in Liga Hack in Linux, FWIW.

@jonnydee
Copy link

jonnydee commented Mar 15, 2020

Does the four character ligature <!-- work, or are four-character ligatures in general broken?

<==> and <!-- both work for me in Liga Hack in Linux, FWIW.

<!-- does work, so it seems like there is not a problem with four-character ligatures in general.

@hohmannr hohmannr removed their assignment Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants