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

Update to Nerd Fonts 3.0 #34

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

vinnydiehl
Copy link

@vinnydiehl vinnydiehl commented Jun 9, 2023

Most of the Material icons were broken with 3.0, and many have been added. Not only were the keycodes broken, many have been renamed or removed altogether.

This PR fixes all of these issues and adds over 5,000 icons. As far as I can find, this is the complete set. The name formatting is now consistent with the official names of the icons.

This list was taken from this repository and transposed with this script.

Most of the Material icons were broken, and many have been added.
@doctorfree
Copy link

This repository seems to have fallen out of maintenance, the last commit about a year and a half ago. It's an excellent plugin and I want to continue to use it. My fork at https://github.com/doctorfree/cheatsheet.nvim includes several fixes and improvements including this PR.

When and if the original maintainer comes back online I will submit PRs to sync but until then please feel free to use and/or contribute to my fork.

@vinnydiehl
Copy link
Author

vinnydiehl commented Jun 10, 2023

@doctorfree Thank you! I didn't know about your fork, I've switched my config to use yours and will leave this issue open so that anyone searching for this compatibility fix may find it. I have also updated the Telescope wiki to point to your fork, of course with a reference to this repository as the original plugin.

@duraki
Copy link

duraki commented Aug 6, 2023

@doctorfree: thanks a lot for the fork and your updates! Gotta check this out. You should enable Issues in your repository fork (@doctorfree).

@vinnydiehl: thanks a lot for the PR. I'll fetch the yours, upstream repo.

@doctorfree
Copy link

@duraki thanks for the heads up. I enabled issues in my fork.

@duraki
Copy link

duraki commented Aug 7, 2023

Awesome man. Thankz, that was fast. I won't spam on this PR anymore, just wanted to note I will credit your fork as much. Bless u 🙏

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

Successfully merging this pull request may close these issues.

3 participants