-
Notifications
You must be signed in to change notification settings - Fork 2.3k
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
new icon: remix (original, original-wordmark, line, line-wordmark) #2150
Conversation
- plain - plain wordmark - line - line wordmark
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hi @AnshSinghSonkhia, thx for your contribution.
You need to check your SVG files according to https://github.com/devicons/devicon/blob/develop/CONTRIBUTING.md#svgStandards
- The plain and line versions (with or without wordmark) need to stay as simple as possible. They must have only one color and the paths are united. The color will be removed when being turned into icons so the .svg can have any color.
- The SVG should be on a single line, with a following empty line
- Check the Line version, to be a path, not strokes
I made some changes @weh. |
flipped original & plain for base & alias
Changed color to #121212 Co-authored-by: Alex Canales <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent work 👍 Almost there.
Don't forget to that all the SVGs need to have 1 path only and must be optimized.
You can follow the tutorial I provided before on how to unify the paths and optimize the SVGs. The section is called Unify the Paths and create Plain Icons (Inkscape)
. You don't need to create plain icons, but make sure that the original svgs and the line svgs only have 1 path, 1 fill color, and the paths are not grouped.
Dont forget to optimize all the SVGs too using SVGOMG. You can also find this step in the tutorial.
I successfully Unified the Paths according to the tutorial Is it necessary to optimizing the SVG for removing inkscape's attributes? |
Hey @AnshSinghSonkhia, yes. the SVGs have to be optimized. Also, taking a look at the SVGs, there is more than 1 path for all SVGs,. Original: 2 Paths and its still grouped. It's the same for the line SVGs. If possible, could you post the SVG code after the optimizing it in SVGOMG. |
Optimizing is a 1 Click task with SVGVIEWERI found it more easier to do with it.
BOOM !!! The SVG is optimized, without ruining the viewbox. |
SVG Viewer is an awesome alternative, a previous contributor even recommended it, but notice how the SVGs still have the height, width and even stroke attributes after optimizing them in svg viewer.
And this is how it looks when I copied the original svg code from above into SVGOMG. The viewbox is still
|
Okay @canaleal, so what do you want me to do now, for the best results? |
With SVGOMG<svg xmlns="http://www.w3.org/2000/svg" fill="none" viewBox="0 0 128 128"><path fill="#121212" d="M8.389 0v26.09h55.445c14.656 0 21.986 6.29 21.986 16.67 0 11.804-7.33 16.263-21.986 16.263H8.389v26.663h53.818c11.603 0 17.912 3.255 18.93 17.91.77 10.01.674 14.965.582 19.718-.03 1.53-.06 3.038-.06 4.686h34.218c0-7.567 0-14.5-1.15-29.29-1.425-14.857-9.161-22.386-22.393-25.032 15.674-2.036 27.277-14.248 27.277-32.973C119.611 15.671 103.122 0 69.941 0zm0 108.125V128h43.535v-12.72c0-2.674-1.312-7.155-7.356-7.155z"/></svg> I have not committed these changes, @canaleal It is showing 150px height & width, Will it work? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi! I'm the
Check our CONTRIBUTING guide for more details regarding these errors. Please address these issues. When you update this PR, I will check your SVGs again. Thanks for your help, |
Still, this pr has one pending review @canaleal |
Yup, thats pretty much the process. At least 2 reviewers are need to ensure that the changes are correct. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Hi! I'm the
Check our CONTRIBUTING guide for more details regarding these errors. Please address these issues. When you update this PR, I will check your SVGs again. Thanks for your help, |
new icon: remix (original, original-wordmark, line, line-wordmark)
Double check these details before you open a PR
This PR closes #2127
Link to prove your SVG is correct and up-to-date.
credits
Thanks to @weh