Skip to content

Non colored file icons #1172

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

Closed
Diaoul opened this issue Apr 16, 2022 · 3 comments · Fixed by #1175
Closed

Non colored file icons #1172

Diaoul opened this issue Apr 16, 2022 · 3 comments · Fixed by #1175

Comments

@Diaoul
Copy link

Diaoul commented Apr 16, 2022

Is your feature request related to a problem? Please describe.
I find colored icons too distracting in nvim-tree.
image

Describe the solution you'd like
The ability to show the icon in the same color as the text with an option or the ability to customize it entirely (like the folder icons)

feline.nvim has a colored_icon option that allows to get the icon without the color.
image

Here is an example with exa that seems to use non-colored icons by default as well (and probably another set of icons):
exa -al --icons

image

Describe alternatives you've considered
Using NvimTreeFileIcon highlight group does not work.

Additional context
🤷

@alex-courtis
Copy link
Member

alex-courtis commented Apr 17, 2022

Looks like we forgot to add the note that the HL group doesn't apply to web-devicons: #1131

Changing web-devicons colours can be done: take a look at the conversation in the PR and you should be able to put something together. PRs are always welcome :)

Edit: both resolved in #1175

@Diaoul
Copy link
Author

Diaoul commented Apr 17, 2022

Wow that was fast and looks like a clean fix in the PR. Thanks a lot 👍

@alex-courtis
Copy link
Member

I would be grateful if you could test this branch:

#1172-non-coloured-webdev-file-icons

PR is still open, so best to catch issues now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants