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

Materializecss: Fix issues in paths #156

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

Conversation

Finii
Copy link

@Finii Finii commented Mar 18, 2024

[why]
The glyph is extremely complex with very thin open spaces and bridges. Obviously a conversion from some colored icon, without QC afterwards.

[how]
Remove all the small crevices, smoothen the surface carefully and make too thin strokes a bit wider.

We still keep the Devicons goal "very crisp icons" so that no rounding of the paths and reduction of points is used.

This is just the svg, no other assets are generated (fonts, images).

Related, the Devicons used by Nerd Fonts have this fix applied (a fixed font can be downloaded there as well):

Zoom into the icon

image

Before and after

image

[why]
The glyph is extremely complex with very thin open spaces and bridges.
Obviously a conversion from some colored icon, without QC afterwards.

[how]
Remove all the small crevices, smoothen the surface carefully and make
too thin strokes a bit wider.

We still keep the Devicons goal "very crisp icons" so that no rounding
of the paths and reduction of points is used.

Related:
ryanoasis/nerd-fonts#1548

Signed-off-by: Fini Jastrow <[email protected]>
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