Skip to content
This repository was archived by the owner on Apr 24, 2020. It is now read-only.

Name of nerd-font icon set is confusing #458

Closed
dritter opened this issue Mar 24, 2017 · 5 comments
Closed

Name of nerd-font icon set is confusing #458

dritter opened this issue Mar 24, 2017 · 5 comments
Assignees

Comments

@dritter
Copy link
Member

dritter commented Mar 24, 2017

The name of the nerd-font icon array is called "nerdfont-fontconfig". This is a bit misleading, because the complete versions of nerd-font (the ones we support) are already pre patched fonts, right? We could use them in a fontconfig setup, but that is not necessary.

If I understood @ryanoasis correctly, the code points are stable in the complete fonts. So there should be no difference in using nerdfont in a fontconfig setup and as directly installed.

I suggest the name "nerdfont-complete". For BC reasons, I would keep the old name (but only as a fallback in functions/icons.zsh. Everywhere else the new name should be referenced.

What do you think?

Documentation:
In #351 the nerdfont support was merged.

@onaforeignshore
Copy link
Contributor

Shouldn't this be "nerdfont-patched" to match "awesome-patched"?

@dritter
Copy link
Member Author

dritter commented Mar 27, 2017

@onaforeignshore I thought about "nerdfont-patched" too, but came to the conclusion that "nerdfont-complete" reflects the situation better. The NerdFont project is a set of patched fonts. On the opposite, awesome-terminal-fonts offer explicitly a fontconfig strategy and a patched strategy. Even in the fontconfig variant the codepoints get shifted, so they are different from the ones in the source font. That is something that does not exist in NerdFonts. Of course, you could use the source fonts in a fontconfig setup as well, but would get collisions eventually.

@ryanoasis
Copy link
Member

@dritter Yes they should be stable (at the very least for any major version). I am all for using 'complete' as an identifier that makes sense to me given the conversation here

@bhilburn
Copy link
Member

bhilburn commented Apr 4, 2017

Excellent point, @dritter. @onaforeignshore raises a fair concern, though, in that as the number of different configuration keywords grows we risk even further confusing users over an already confusing topic, hah. I'm in favor of staying technically accurate, though, and it sounds like -fontconfig isn't the best option. Since the nerdfont maintainer, @ryanoasis, agrees with -complete, let's roll with that.

Have a look at #466!

@bhilburn bhilburn self-assigned this Apr 4, 2017
@onaforeignshore onaforeignshore mentioned this issue Apr 7, 2017
20 tasks
bhilburn added a commit that referenced this issue Apr 7, 2017
Changing name of nerdfont option per #458
@onaforeignshore onaforeignshore mentioned this issue Apr 8, 2017
23 tasks
@bhilburn
Copy link
Member

Merged in #466!

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

4 participants