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.

Sign up for GitHub

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

Jump to bottom

Make refresh* functions public #298

Closed
simonmandlik opened this issue Aug 28, 2023 · 2 comments
Closed

Make refresh* functions public #298

simonmandlik opened this issue Aug 28, 2023 · 2 comments

Comments

@simonmandlik
Copy link
Contributor

simonmandlik commented Aug 28, 2023 •
edited
Loading

It would be great to be able to call require("nvim-web-devicons").refresh* functions programatically on demand, not only on autocommand. This would help immensely especially during startup, when autocommands may not (yet) be available.

If you agree, I can make a PR for that
@simonmandlik simonmandlik changed the title Make refresh_icons public Make refresh* functions public Aug 28, 2023
@alex-courtis
Copy link
Member

That sounds reasonable. We should call set_up_highlights as well.

Yes please, do create a PR: https://github.com/nvim-tree/nvim-web-devicons/blob/master/CONTRIBUTING.md

simonmandlik added a commit to simonmandlik/nvim-web-devicons that referenced this issue Sep 2, 2023
@simonmandlik
refresh function exposed, fixes nvim-tree#298
e01ca51
@simonmandlik
Copy link
Contributor Author

Sure, here it is #301

@simonmandlik simonmandlik mentioned this issue Jan 15, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@alex-courtis @simonmandlik