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

Use of Outdated Glyphs #45

Open
manuth opened this issue May 5, 2023 · 13 comments
Open

Use of Outdated Glyphs #45

manuth opened this issue May 5, 2023 · 13 comments

Comments

@manuth
Copy link

manuth commented May 5, 2023

logo-ls currently uses outdated nf-mdi-* glyphs which have, according to the nerdfont website, been removed.

These need to be migrated to be used with up-to-date nerd fonts

@D3vil0p3r
Copy link

I'm getting this behavior but I'm not sure if it is related to the issue above:
image

PS: is still this repository maintained?

@pi9x
Copy link

pi9x commented Jun 26, 2023

Not sure if this is the reason but I'm getting this even though I have changed many nerd fonts. This issue happens to logo-ls only. The icons display properly when I use lsd.

image

@D3vil0p3r
Copy link

Not sure if this is the reason but I'm getting this even though I have changed many nerd fonts. This issue happens to logo-ls only. The icons display properly when I use lsd.

image

It occurs because you installed google-fonts and it can produce that wrong glyphs.

@pi9x
Copy link

pi9x commented Jun 27, 2023

Not sure if this is the reason but I'm getting this even though I have changed many nerd fonts. This issue happens to logo-ls only. The icons display properly when I use lsd.
image

It occurs because you installed google-fonts and it can produce that wrong glyphs.

No I use nerd fonts from https://www.nerdfonts.com/

@orhnk
Copy link

orhnk commented Jul 12, 2023

Have a look at here

@manuth
Copy link
Author

manuth commented Jul 12, 2023

Downgrading Nerd Fonts is not a fix, tho 😅
As a fix, I'd rather recommend you to locally merge this Pull Request and build your own logo-ls

@manuth
Copy link
Author

manuth commented Jul 12, 2023

@orhnk
Copy link

orhnk commented Jul 12, 2023

image

:/

@manuth
Copy link
Author

manuth commented Jul 12, 2023

Ouch - now that's strange.
Did you merge PR #46?
Also - what version is your Nerd Font?

@orhnk
Copy link

orhnk commented Jul 12, 2023

Did you merge PR #46?

I cloned your fork and compiled it as following:

go mod tidy
go build

Also - what version is your Nerd Font?

I just reinstalled JetBrains Nerd Font from the official site

@manuth
Copy link
Author

manuth commented Jul 12, 2023

It does work for me - even with the JetBrains Nerd Font enabled.
Be sure to check out the branch nerd-font-3 of my repository.

@orhnk
Copy link

orhnk commented Jul 12, 2023

Somehow I did clone the original repo?!?
Nvm thanks for the fast response and sorry.

Here:

image

@manuth
Copy link
Author

manuth commented Jul 12, 2023

no worries, you're fine 😄
I'm glad I could help!

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

No branches or pull requests

4 participants