Closed ofseed closed 5 months ago
Thank you for finding all these inconsistencies in the various script icons.
The test icons were added #215 based on vscode and are not "official" javascript icons. It appears that they are the material icons rather than the seti icons.
We appear to be in a loop here, moving between icon sets.
We are using seti and material icons however we are inconsistent.
nvim-web-devicons current:
VSCode default seti icons:
VSCode material icons:
Idea: audit the icons noting the class and hex.
This could be done during generate_colors.lua and comments added to the icon. Lua 5.3+ utf8 support would provide this.
Maybe it's time to introduce icon sets? Or pick one style and support only that one. Consistency is a good thing.
Maybe it's time to introduce icon sets? Or pick one style and support only that one. Consistency is a good thing.
Yes. We seem to mainly use seti, octicons, devicons and material.
Let's see how an audit goes.
Some analysis, mapping the icons to their nerd-font classes:
https://gist.github.com/alex-courtis/26271d581e390fe26058bf0743539cc3
TL;DR: we are using mostly seti and dev with some material design and font awesome.
Maybe it's time to introduce icon sets? Or pick one style and support only that one. Consistency is a good thing.
I'm happy to pick seti and use that for the defaults.
Users can add different icon sets via setup and I think there is a material configuration out there somewhere. A contributor could add icon sets with plumbing to nvim-web-devicons if they are motivated.
It might be time to reopen #192 which will be simpler now that we have the generation scripts.
Many thanks @ofseed and all for the great discussion.
I've updated #192 to move to css class names.
I'll close this and #377 - we will resolve all the js/ts icons in one change.
The original ts and js icons keeps the same style:
But https://github.com/nvim-tree/nvim-web-devicons/commit/808627b8d412b2a6b6fc6eed816fec3557198b01 and https://github.com/nvim-tree/nvim-web-devicons/commit/33e27b859f549873a826d8aaa348fd31195e439f break this:
So I recommend to update the ts icon as well:
And use the original ts icon for
tsconfig.json
, because they are very similar.But what I recommend most is to revert these two PRs directly. There is no reason to do this update, and this also affects the uniform style of other icons, for test.js
This new icon is a completely different style:
Note: I'm not swapping icons for tsconfig and ts, These two icons look the same but they have very subtle differences. These are two icons with different code points. Compare to ts:
e69d(nf-seti-tsconfig):![image](https://github.com/nvim-tree/nvim-web-devicons/assets/61115159/6797ecff-f1c3-4ac8-a06c-7ee80a013df1)
f06e6(nf-md-language_typescript):![image](https://github.com/nvim-tree/nvim-web-devicons/assets/61115159/3c3f7f6b-c926-4677-943e-4b7b38ffa3c1)