Skip to content

Some fonts displays double as wide as before #895

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

Closed
Finii opened this issue Aug 26, 2022 · 21 comments
Closed

Some fonts displays double as wide as before #895

Finii opened this issue Aug 26, 2022 · 21 comments

Comments

@Finii
Copy link
Collaborator

Finii commented Aug 26, 2022

@Finii My Nerd Fonts are managed by a Scoop bucket and it just pushed out an update for many of the NF fonts, Mononoki included, so I assumed it had been: matthewjberger/scoop-nerd-fonts@dfded9f. After installing the font and restarting the system, at least something has changed because the font in VS Code/Codium now displays double as wide as before:

VSCodium

Not sure why, but other programs seem to be unaffected by this bug, and it can be essentially mitigated in Code/Codium by using editor.letterSpacing": -5.

Also, the other NF fonts just updated display the same double-wide behaviour: Switching to Consolas renders with the correct letter spacing. Maybe Code/Codium has a preference that I've altered which is affecting the NF fonts' rendering, but why not Consolas for example? Hmm.

Originally posted by @redactedscribe in #575 (comment)

@Finii
Copy link
Collaborator Author

Finii commented Aug 26, 2022

image
Image shows Mono font

I can not see an issue with the non-mono font.

Examining reason.

@redactedscribe
Copy link

I can not see an issue with the non-mono font.

Am I using the non-mono font? I didn't change VS Code's config between restarts and the font's worked fine for months so I presumed I was using the mono variant.

How do I specify the Mono variant in VS Code if you happen to know?

@Finii Finii closed this as completed in b71a9e4 Aug 26, 2022
Finii added a commit that referenced this issue Aug 26, 2022
This is a hotfix for #895, which broke all `Mono` fonts.

Sorry :-(

Signed-off-by: Fini Jastrow <[email protected]>
@Finii
Copy link
Collaborator Author

Finii commented Aug 26, 2022

Released anew ... @redactedscribe https://github.com/ryanoasis/nerd-fonts/releases/tag/v2.2.1

@redactedscribe
Copy link

Thanks for the quick fix. I'm hoping the Scoop bucket will push out the 2.2.1 update as that'll be the least messy way for me to test it. Once it does, I can confirm the fix.

@adelarsq
Copy link

@Finii Good job! I was thinking that was some miss config on my side. 😄
Thanks for the quick update. 🥇

@redactedscribe
Copy link

Once it does, I can confirm the fix.

Fixed. Thanks!

@blacktop
Copy link

I think Hack Mono has the same issue?

@Finii
Copy link
Collaborator Author

Finii commented Aug 28, 2022

Hack Mono v2.2.1?

Looks good at first glance:

image

@blacktop Please check which version you have.

@blacktop
Copy link

hack

Looks like I'm still on 2.2.0, I'll try and update, but I think brew hasn't updated it yet?

@Finii
Copy link
Collaborator Author

Finii commented Aug 29, 2022

Looks like I'm still on 2.2.0, I'll try and update, but I think brew hasn't updated it yet?

The update is here Homebrew/homebrew-cask-fonts@7530545

@adastx
Copy link

adastx commented Aug 30, 2022

Jet Brains Mono doesn't look right after the update.

@Finii
Copy link
Collaborator Author

Finii commented Aug 30, 2022

@adastx can you provide a screenshot?
Shure you are on v2.2.1 and not v2.2.0?
Reboot or similar after update?

@adastx
Copy link

adastx commented Aug 31, 2022

Here are some screenshots. The issue is more subtle than what people were having before with mono fonts.

It's most obvious by looking at the arrow symbol.

2.1.0

2 1 0

2.2.1

2 2 1

Reboot doesn't fix the issue and I can switch back and forth between versions without rebooting. I only need to restart alacritty to see the effect.

Also yes, I am sure about the versions.

@Finii
Copy link
Collaborator Author

Finii commented Aug 31, 2022

Ah, you mean vertically?

image

Or do you mean the green arrow? Are you using Windows Terminal?

@adastx
Copy link

adastx commented Aug 31, 2022

Im using alacritty on arch linux. vertically and the green arrow...

@Finii
Copy link
Collaborator Author

Finii commented Aug 31, 2022

Windows Terminal

unlikely if you are on Arch. Hmm.

Sure you do not use the Nerd Font Mono variant?

@adastx
Copy link

adastx commented Aug 31, 2022

2022-08-31-082817_398x94_scrot

from my alacritty.yml

@Finii
Copy link
Collaborator Author

Finii commented Aug 31, 2022

The two images should be the same on both versions. Agree?

Not necessarily. Maybe the source changed or bugs got fixed.
The 2.2.1 version should look like JetBrains 2.251 (except added symbols).
The 2.1.0 version is based on JetBrains 1.0.2.

@Finii
Copy link
Collaborator Author

Finii commented Aug 31, 2022

The vertical stuff did change in the source (i.e. JetBrains):

image

And I guess the green-arrow thing is something about #900

@adastx
Copy link

adastx commented Aug 31, 2022

Ok, thanks for the info. I guess I'll just be sticking to 2.1.0 for now.

@github-actions
Copy link
Contributor

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 12, 2023
LNKLEO pushed a commit to LNKLEO/Nerd that referenced this issue Nov 24, 2023
[why]
With commit
  821ac68  Create symbols only font directly from sfd template

we have a default width for source fonts without a width (i.e. empty
fonts).

Unfortunately the detection for 'empty font' is wrong.

[how]
Reorder the commands such that we have a meaningful calculation.

Maybe this has been broken by too many (manual) rebases.

Fixes: ryanoasis#895

Reported-by: redactedscribe
Signed-off-by: Fini Jastrow <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants