-
Notifications
You must be signed in to change notification settings - Fork 2.4k
new icon: jiraalign (original,plain,original-wordmark,plain-wordmark) #1468
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
Conversation
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.
Those don't look like the official logos 🙊
Please change to the official ones (I linked to them in issue #789)
I'll grouping them to see if that fixes the issue. |
I had to use the ones you provided on the issue itself as the grouping didn't work. Thanks for that! |
Hello there!
And add the To Wish for the best! |
Hey @lunatic-fox! I think I have covered everything you asked me for but if you see there is something missing just let me know. Best. |
The icons look great now @yeyeto2788! 🤩 However there are some issues yet:
Keep rocking! 🚀 |
Hey @lunatic-fox! I hope this time is correct. So for uniting the path I followed these steps:
I guess that could be added into the documentation. I really helped me a lot the example with the box creation and this might as well help others. |
Hello @yeyeto2788!
But Checking layers
Hope everything works fine! 👍🏼 |
Hopefully this one is the last one. Once again sorry for the inconveniences. Best. |
No worries! There is no inconvenience at all. 🙂 Suggestions of tags "tags": [
"development",
"integration",
"software"
] The icons are great now! 🎉 Like I said before you can also contact us on Discord server. I also loved your idea of adding some instructions on using Inkscape! |
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.
Almost there! 🚀
Just need to add the name with whitespaces as an altname and minimize all of the icons. You can minimize the icons like I explained here: https://github.com/devicons/devicon/pull/1465/files#r1008766362
Yes, this could be added to the wiki as a part of this issue: #1306 |
I'll be adding this later Instructions to resize to square and Checking layers. |
@lunatic-fox Nice! Please add links to those as comments with the same format I used in the issue I created: #1306 |
Co-authored-by: Jørgen Kalsnes Hagen <[email protected]>
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, |
@yeyeto2788 Please resize the viewbox as explained above. I recommend using iloveimg to make it fast and easy :) |
Hi there, I'm Devicons' Peek Bot and I just peeked at the icons that you wanted to add using icomoon.io. Here are the SVGs as intepreted by Icomoon when we upload the files: Here are the zoomed-in screenshots of the added icons as SVGs: Here are the icons that will be generated by Icomoon: Here are the zoomed-in screenshots of the added icons as icons: Here are the colored versions: The maintainers will now check for:
In case of font issues, it might be caused by Icomoon not accepting strokes in the SVGs. Check this doc for more details and fix the issues as instructed by Icomoon and update this PR once you are done. Thank you for contributing to Devicon! I hope that your icons are accepted into the repository. Note: If the images don't show up, it has been autodeleted by Imgur after 6 months due to our API choice. Cheers, |
Hi there, I'm Devicons' Peek Bot and I just peeked at the icons that you wanted to add using icomoon.io. Here are the SVGs as intepreted by Icomoon when we upload the files: Here are the zoomed-in screenshots of the added icons as SVGs: Here are the icons that will be generated by Icomoon: Here are the zoomed-in screenshots of the added icons as icons: Here are the colored versions: The maintainers will now check for:
In case of font issues, it might be caused by Icomoon not accepting strokes in the SVGs. Check this doc for more details and fix the issues as instructed by Icomoon and update this PR once you are done. Thank you for contributing to Devicon! I hope that your icons are accepted into the repository. Note: If the images don't show up, it has been autodeleted by Imgur after 6 months due to our API choice. Cheers, |
env.MESSAGE |
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.
Looks good now, but the icons are still quite big. Could you please optimize them using this method? #1306 (comment)
I did that step before my last commit but I have done again hoping it does not screw anything as the last time. Best. |
Hi there, I'm Devicons' Peek Bot and I just peeked at the icons that you wanted to add using icomoon.io. Here are the SVGs as intepreted by Icomoon when we upload the files: Here are the zoomed-in screenshots of the added icons as SVGs: Here are the icons that will be generated by Icomoon: Here are the zoomed-in screenshots of the added icons as icons: Here are the colored versions: The maintainers will now check for:
In case of font issues, it might be caused by Icomoon not accepting strokes in the SVGs. Check this doc for more details and fix the issues as instructed by Icomoon and update this PR once you are done. Thank you for contributing to Devicon! I hope that your icons are accepted into the repository. Note: If the images don't show up, it has been autodeleted by Imgur after 6 months due to our API choice. Cheers, |
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.
Looking good! ✔️
Thank you for your contribution! 💯
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.
Approved! ✔
Great contribution! 🚀
…devicons#1468) * Add Jira Align icons * Fix logos * Fix minor issues discovered in PR. * Fix color for the icon and unite paths * Fix paths and ungroup objects. * Add altnames and tags to icon * Update devicon.json Co-authored-by: Jørgen Kalsnes Hagen <[email protected]> * Minimize icons * Resize icon * Pass icons through SVGOMG Co-authored-by: Jørgen Kalsnes Hagen <[email protected]>
Double check these details before you open a PR
develop
branch.devicon.json
file as seen here**This PR closes #789 **
Link to prove your SVG is correct and up-to-date.