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

add nerd-font support #390

Closed
wants to merge 6 commits into from
Closed

add nerd-font support #390

wants to merge 6 commits into from

Conversation

lcorsini
Copy link
Contributor

This is my proposal to support nerd-font using fontconfig fallback support.
I tested it under linux and iTerm2 on OSX, and seems to work very well, you can see a screen here:
#382
instead of adding custom codepoints to fontawesome config I create a new config mode and added all nerdfont codepoints (for the complete font variant) there

@bhilburn
Copy link
Member

Adding additional font config options is cheap for us, and makes the font for accessible for other users. I'm not a nerdfont user myself, so I need to ask for your help, @lcorsini to make sure we stay up-to-date with the current nerd-font codepoints.

I really appreciate you making this PR and upstreaming this feature, @lcorsini. Nice work =)

@bhilburn
Copy link
Member

I want to merge this in on next, but the changeset is based on master. If you get a chance to rebase this branch today, @lcorsini, that would be great. Otherwise I'll get to it tomorrow =)

@lcorsini
Copy link
Contributor Author

Sure I'll try to keep it up to date (maybe implement also the patched version of the font if differs) and improve it as nerd font development continues. As to rebase it, unfortunately I can't do it until monday so I can't help today, I'm sorry

@Arzte
Copy link

Arzte commented Feb 21, 2017

@lcorsini What's the status of rebasing what branch your merging on?

@lcorsini
Copy link
Contributor Author

lcorsini commented Feb 21, 2017 via email

@lcorsini lcorsini changed the base branch from master to next February 21, 2017 16:57
@Arzte
Copy link

Arzte commented Feb 23, 2017

@bhilburn Looks like its good to merge!

@bhilburn
Copy link
Member

I just merged #351, which added nerdfont support. I diff'd against this branch to try and get an idea of the changes, but it was impossible for me to really tell as I don't have nerdfonts working on my system. I'm going to open a new issue to discuss this.

@bhilburn bhilburn closed this Feb 23, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants