search

devicons / devicon

Set of icons representing programming languages, designing & development tools
https://devicon.dev
MIT License
9.97k stars 2.32k forks source link

update icon: llvm: Simplify plain icon #2264

Open Finii opened 2 months ago

Finii commented 2 months ago

[why] While the plain icon is plainer than the original icon, it is still quite complex with very thin strokes.

[how] Just take the outermost outlines of the plain icon and crate a solid icon from it.

Note

This can be controversial. For me the plain icon is not really plain, it has lots and lots of detail and consists of 2600 nodes:

image

The simple plain icon proposed here drops all the details and uses just the outline (260 nodes)

image

If one needs an icon with all the glory details the -original should be a good fit, the plain should be a simple(r) version, I believe. But anyhow, this is very opinionated and I do not want to mess with your ideas here. For a small text icon the current plain icon is far too complex and all the small detail hairlines can not be seen anyhow.

Double check these details before you open a PR

Features

This PR closes NONE

Notes

@Snailedlt as per https://github.com/ryanoasis/nerd-fonts/pull/1691#issuecomment-2313090871

Finii commented 2 months ago

Hmm, hand-remove the gaps in the tail, that is an obvious artifact of the fill related broken outline in the original svg. Force push.

image

Finii commented 2 months ago

hmmm, I don't agree with this one

:laughing:

As I said, controversial (see image below).

For me, as "in a font" the current plain one is just too intricate. Have a look at it sized down:

image

But that is just for ME and font usage, which might or might not be important for you. Just wanted to share this icon version with you.

image

Snailedlt commented 2 months ago

I agree it looks better in the font, but in larger sizes it looks significantly worse. Perhaps we could find some middle ground? One way would be to roughly inverse the current plain version, and remove some of the detailing lines.

I'm thinking something like this (just with transparent lines instead of white ones): image

Snailedlt commented 2 months ago

@Finii Here's a suggestion. Though it should be optimized more, because it's very large (80Kb). I optimized it using SVGOMG, but it should probably be optimized more by reducing the amount of nodes in the path. llvm-original

Snailedlt commented 2 months ago

Sidenote: The old plain version can still be used as a line version instead :)