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

Prefer nerd font to correctly render icons #20

Open
pnodet opened this issue Nov 6, 2023 · 4 comments
Open

Prefer nerd font to correctly render icons #20

pnodet opened this issue Nov 6, 2023 · 4 comments

Comments

@pnodet
Copy link

pnodet commented Nov 6, 2023

The google font version of Fira Code doesn't handle icons existing in the nerd font version.

Screenshot 2023-11-06 at 07 32 19

Screenshot 2023-11-06 at 07 33 43

One thing to consider is that it's not a variable font, maybe worth taking into consideration.

@ekzhang
Copy link
Owner

ekzhang commented Nov 14, 2023

Thanks for the suggestion! It's interesting to hear how other people use terminal fonts.

I'm actually using the official Fira Code package, not the Google Fonts version. Perhaps I can add a font switcher to the UI so people can choose their preferred setting!

@pnodet
Copy link
Author

pnodet commented Nov 14, 2023

Sure, would love this!
From the reactions to my issue above I think many would like it as well.

One "technical" challenge is that I'm not sure the nerd fonts are hosted by unpkg like the Fira Code, I don't know how the font switcher should handle this, maybe downloading and hosting some fonts in the repo would be a solution, and allowing users to choose between provided fonts.

@ekzhang
Copy link
Owner

ekzhang commented Nov 14, 2023

We're not hosting them on unpkg; the fonts are served by our server.

@silopolis
Copy link

Yes please!
My lsd is on a bad trip without it ;)

Of course, a font selector would be freaking cool...

Thanks for this fantastic gem 🙏
Take care
J

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

3 participants