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

feat(neotree): replace all-the-icons by nerd-icons #816

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

Conversation

forty
Copy link

@forty forty commented Jan 2, 2024

I have issues when missing all-the-icons and nerd-icons. It seems that nerd-icons is the new thing, so I updated neotree extension to use it, and it seems to work fine here (both in simple and normal mode)

Related #263


  • I searched the issue tracker and this hasn't been PRed before.
  • My changes are not on the do-not-PR list for this project.
  • My commits conform to the git conventions.
  • My changes are visual; I've included before and after screenshots.
  • I am blindly checking these off.
  • Any relevant issues or PRs have been linked to.
  • This a draft PR; I need more time to finish it.

image

@forty
Copy link
Author

forty commented Jan 2, 2024

Note that I also have similar update for the treemacs extension, but I don't use treemacs and haven't tested at all, so I'm not very confortable doing the MR, unless someone wants to try it out (also, I don't really understand everything I'm doing ;) )

@forty forty force-pushed the forty/neotree-nerd-fonts branch 2 times, most recently from 462df22 to 47462a0 Compare August 21, 2024 08:17
@forty forty force-pushed the forty/neotree-nerd-fonts branch 2 times, most recently from 9157d3a to ed64143 Compare August 23, 2024 20:14
Main caveat is that there is no "nf-oct-file_text" in nerd-icons so
"file-text" octicon have been replaced by "nf-oct-file" (so they will
appear as generic files rather than text files)
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.

1 participant