-
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: rollup (original, original-wordmark, plain, plain-wordmark, line, line-wordmark) #1497
Conversation
Hello there @PantheraRed! What do you think?
|
Hello @lunatic-fox, |
What do you think about this suggestion @Snailedlt, @Panquesito7?
|
@lunatic-fox Love it! |
Any of you wanna try and make a line icon too? "How do I do that?" I hear you ask. |
@PantheraRed can you add those two icons to the PR and let me know when it's there please? :) |
Done @Snailedlt |
@PantheraRed just found the wordmark logo too on their website. You wanna add original, plain and line verdions of that too? I'm on the phone so can't share a previw or direct link to it, but the svg is top left corner on this page: https://rollupjs.org/guide/en/ |
I am sure not sure if they have an actual wordmark logo. It's not in their repository. But if want me to add them sure. @Snailedlt |
Isn't the one you screenshot there their wordmark logo? |
I don't think so. It defers from platform to platform. It's not on their repository's readme either. @Snailedlt |
Ahh, okey :) |
Sure if you're able to @Snailedlt. |
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 for the most part. Added some suggestions for devicon.json, the icons themselves look perfect though 💯
Sidenote: I reached out to rollup devs about the wordmark icons in this issue: rollup/rollupjs.org#171 . Hopefully we can get a response from them soon :)
Hi there, I'm Devicons' Peek Bot and it seems we've ran into a problem.
Make sure that:
I will retry once everything is fixed. If I still fail or there are other error, the maintainers will investigate. Best of luck, |
@Snailedlt any idea what happened here? |
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.
Alright @PantheraRed I think this looks good now! ✔️
We still need to wait for another review before merging. But if that comes quickly... do you prefer to merge, or wait a few days and see if we get an answer regarding the wordmark version?
Yes, I'd like to wait for few days and see if we get any response regarding the rollup-wordmark. Thank you. 👍 @Snailedlt |
Thanks to @lunatic-fox for plain and line logos. 👍 |
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.
Neat!
Just a couple small suggestion for the line-wordmark version
@@ -0,0 +1 @@ | |||
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 128 128" xml:space="preserve"><g fill="#F33"><path d="M47.899 59.542a5.675 5.675 0 0 0-1.018-.088c-.938 0-1.732.262-2.38.785-.649.523-1.112 1.283-1.388 2.28v9.334h-1.495V58.261h1.47l.025 2.161c.796-1.608 2.073-2.412 3.832-2.412.419 0 .749.055.992.163l-.038 1.369zM49.193 64.856c0-1.307.253-2.483.76-3.53.507-1.047 1.223-1.862 2.149-2.443.925-.582 1.974-.873 3.147-.873 1.809 0 3.275.634 4.397 1.904 1.122 1.268 1.683 2.95 1.683 5.044v.314c0 1.315-.253 2.498-.76 3.549-.507 1.051-1.221 1.862-2.142 2.431s-1.972.854-3.153.854c-1.801 0-3.264-.634-4.391-1.904-1.126-1.268-1.69-2.95-1.69-5.044v-.302zm1.508.415c0 1.625.421 2.959 1.262 4.001.842 1.043 1.946 1.564 3.311 1.564 1.357 0 2.456-.521 3.298-1.564.842-1.042 1.263-2.418 1.263-4.127v-.289c0-1.038-.193-1.989-.578-2.852-.385-.863-.925-1.531-1.621-2.004-.695-.473-1.491-.71-2.387-.71-1.34 0-2.433.525-3.279 1.577-.846 1.051-1.269 2.426-1.269 4.127v.277zM66.116 71.854h-1.508V52.557h1.508v19.297zM71.882 71.854h-1.508V52.557h1.508v19.297zM84.709 70.133c-.905 1.315-2.349 1.972-4.334 1.972-1.449 0-2.55-.421-3.304-1.262-.754-.842-1.139-2.088-1.156-3.738v-8.844h1.495v8.656c0 2.588 1.047 3.882 3.141 3.882 2.178 0 3.555-.9 4.133-2.701v-9.837h1.508v13.593h-1.457l-.026-1.721zM101.029 65.196c0 2.119-.469 3.8-1.407 5.044-.938 1.243-2.194 1.865-3.769 1.865-1.859 0-3.283-.653-4.271-1.96v6.935h-1.495V58.261h1.395l.075 1.922c.98-1.449 2.4-2.173 4.259-2.173 1.625 0 2.9.616 3.826 1.847.925 1.231 1.388 2.94 1.388 5.126v.213zm-1.508-.264c0-1.734-.356-3.103-1.068-4.108s-1.704-1.508-2.977-1.508c-.921 0-1.713.222-2.374.666-.662.444-1.168 1.089-1.52 1.935v6.52c.36.779.875 1.374 1.545 1.784s1.462.616 2.374.616c1.265 0 2.251-.505 2.959-1.514.707-1.009 1.061-2.473 1.061-4.391zM104.22 70.95c0-.293.094-.542.283-.748.188-.205.45-.308.785-.308s.599.103.791.308c.193.205.289.454.289.748s-.096.534-.289.723c-.193.188-.457.282-.791.282-.335 0-.597-.094-.785-.282a.98.98 0 0 1-.283-.723zM112.122 58.261v15.214c0 1.256-.302 2.215-.905 2.877-.603.662-1.491.992-2.663.992-.452 0-.892-.079-1.319-.237l.025-1.233c.369.109.746.163 1.131.163 1.491 0 2.236-.859 2.236-2.577v-15.2h1.495zm-.741-4.937c.327 0 .582.098.766.293.184.196.276.435.276.718s-.092.52-.276.711-.44.287-.766.287c-.318 0-.568-.096-.748-.287a.997.997 0 0 1-.27-.711c0-.283.09-.522.27-.718.18-.196.429-.293.748-.293zM124.509 68.399c0-.746-.299-1.344-.898-1.797-.599-.452-1.501-.806-2.707-1.061-1.206-.256-2.142-.549-2.808-.879-.666-.331-1.16-.735-1.482-1.213-.323-.477-.484-1.055-.484-1.734 0-1.072.448-1.958 1.344-2.657.896-.7 2.043-1.049 3.442-1.049 1.516 0 2.733.375 3.649 1.125.918.749 1.376 1.727 1.376 2.933h-1.508c0-.796-.333-1.457-.999-1.985-.665-.528-1.505-.791-2.519-.791-.988 0-1.782.22-2.381.659-.599.44-.898 1.012-.898 1.715 0 .678.249 1.204.747 1.576.499.373 1.407.714 2.727 1.024 1.319.31 2.307.641 2.965.992.657.352 1.145.775 1.463 1.269.318.494.477 1.093.477 1.797 0 1.148-.463 2.065-1.388 2.751-.926.687-2.134 1.03-3.625 1.03-1.583 0-2.867-.383-3.851-1.15-.984-.766-1.476-1.735-1.476-2.908h1.508c.058.879.429 1.564 1.112 2.054.682.49 1.585.735 2.707.735 1.047 0 1.893-.23 2.538-.691s.969-1.041.969-1.745zM3.467 47.427a.637.637 0 0 0-.634.634v26.515l.004.005c-.337.916-.561 1.668-.691 2.281l-.001.003a.22.22 0 0 1-.002.02v-.008c-.154.728-.185 1.272-.137 1.693v.001c.009.083.024.161.038.235.027.223.115.391.204.572.046.095.111.18.163.257.075.105.154.193.225.273v.001c.059.066.122.123.184.176l.029.023c.047.038.096.075.149.11.002 0 .012.009.034.023l.015.002c.752.464 1.762.401 2.482.054h21.22a.641.641 0 0 0 .568-.923l-.001-.001-5.553-10.993v-.001a.37.37 0 0 1 .145-.488 10.95 10.95 0 0 0 5.505-9.506c0-1.955-.511-3.79-1.412-5.38l-.001-.01c-.075-.127-.149-.254-.225-.377l-.01-.014a10.952 10.952 0 0 0-9.142-5.177H3.467zm0 .265H16.62c3.189.049 6.024 1.501 7.953 3.757l-.04-.028a.315.315 0 0 0-.019-.015v.001a6.747 6.747 0 0 0-.219-.158l-.1-.069v.001l-.067-.043-.115-.075-.051-.033-.114-.07a6.508 6.508 0 0 0-.268-.151l-.013-.007-.048-.026-.002-.001a6.876 6.876 0 0 0-.298-.148l-.073-.035-.098-.042-.138-.06-.071-.031-.061-.025a8.564 8.564 0 0 0-.313-.116l-.082-.028-.017-.005a6.517 6.517 0 0 0-.261-.081l-.103-.031-.096-.024a5.208 5.208 0 0 0-.169-.042l-.054-.013-.066-.016a7.103 7.103 0 0 0-.311-.061l-.021-.004-.068-.013-.011-.002a7.808 7.808 0 0 0-.378-.052l-.117-.01-.162-.014-.09-.008h-.001l-.036-.003a12.147 12.147 0 0 0-1.078.008l-.088.005c-.339.022-.663.06-.973.128a4.73 4.73 0 0 0-.627.182c-.41.151-.805.365-1.203.663a6.448 6.448 0 0 0-.601.513c-.029.027-.058.061-.087.09a11.75 11.75 0 0 0-.854.947l-.001.002c-.029.035-.06.081-.089.118-.161.204-.327.422-.497.657-.083.115-.17.241-.256.364-.132.189-.268.389-.407.598l-.263.398c-.185.286-.376.59-.574.912-.945 1.533-2.078 3.528-3.442 6a729.211 729.211 0 0 0-2.271 4.147l-.001.002-.001.002-.036.068c-.388.715-.751 1.395-1.086 2.035l-.001.002-.001.001c-1.356 2.599-2.272 4.59-2.89 6.155V48.061a.372.372 0 0 1 .372-.369zm16.553 2.471c.204.001.408.012.611.031l.067.006c1.5.153 2.95.807 4.097 1.837a7.818 7.818 0 0 0-3.23-1.11h-.001l-.057-.005a7.674 7.674 0 0 0-.423-.033h-.006a6.472 6.472 0 0 0-.654.006 5.622 5.622 0 0 0-1.208.208c-1.003.297-1.834.907-2.256 1.896-.681 1.588-.108 3.259.819 4.486a6.051 6.051 0 0 0 1.558 1.46c.047.029.092.05.139.077l.019.012.074.04a3.165 3.165 0 0 0 .595.256c.112.033.222.056.329.067l.015.002.02.001c.052.004.102.006.152.005l.01-.001c.056-.002.112-.007.165-.017h.001c.068-.012.131-.029.192-.05h.001l.002-.001a.99.99 0 0 0 .415-.278c.144-.167.226-.379.266-.613.074-.443 0-.972-.133-1.505.093.188.247.461.299.598.188.492.249.845.194 1.159-.003.018-.005.037-.009.055a1.34 1.34 0 0 1-.12.322c-.219.424-.708.867-1.486 1.522-.778.654-1.841 1.521-3.199 2.821-2.725 2.609-5.463 6.648-7.626 10.094-1.081 1.723-2.018 3.297-2.734 4.468-.358.585-.661 1.07-.898 1.42v.001c-.118.175-.22.316-.302.418a1.004 1.004 0 0 1-.168.174c-.642.367-1.684.458-2.379.02l-.004.001c-.004-.004-.007-.007-.022-.017l-.002-.001a1.507 1.507 0 0 1-.128-.095l-.025-.02a1.495 1.495 0 0 1-.157-.15 3.447 3.447 0 0 1-.242-.294c-.135-.181-.21-.414-.274-.613a2.155 2.155 0 0 1-.049-.283v-.001c-.044-.386-.017-.905.135-1.622v-.002c.155-.726.451-1.67.921-2.865.144-.357.28-.7.456-1.105a78.986 78.986 0 0 1 2.103-4.411 124.822 124.822 0 0 1 1.461-2.769v-.001l.002-.003v-.001a783.6 783.6 0 0 1 2.284-4.17c.502-.908.988-1.786 1.424-2.558v-.001l.001-.001.003-.006c1.013-1.79 1.86-3.217 2.604-4.365.055-.085.108-.163.162-.245.179-.272.353-.526.521-.765l.17-.241c.208-.289.41-.559.604-.8l.001-.001.002-.002a11.804 11.804 0 0 1 .917-1.009c.196-.187.387-.35.576-.492a4.43 4.43 0 0 1 1.136-.627 4.434 4.434 0 0 1 .737-.197c.079-.015.157-.031.239-.043l.004-.001c.152-.021.309-.036.469-.047.26-.017.53-.027.82-.026zm1.046.99c1.705.071 3.551.809 4.706 1.99a10.61 10.61 0 0 1 1.375 5.242c0 3.973-2.163 7.433-5.373 9.277l-.001.001a.637.637 0 0 0-.247.837l5.553 10.992a.37.37 0 0 1-.331.54H5.884c.014-.016.026-.028.04-.047v-.001c.09-.112.195-.258.315-.436.234-.346.529-.819.874-1.381l.031-.048c.717-1.173 1.653-2.745 2.732-4.464 2.158-3.438 4.894-7.464 7.585-10.04 1.35-1.293 2.405-2.154 3.186-2.81.781-.656 1.294-1.106 1.55-1.604.064-.124.11-.251.139-.382l.001-.002c.087-.394.018-.829-.19-1.377-.276-.726-.795-1.675-1.538-3.077l-.004-.008-.001-.002-.003-.001-.003-.005a.135.135 0 0 0-.108-.07.134.134 0 0 0-.117.055s-.904 1.275-2.227 3.172c-.05-.061-.102-.118-.149-.181-.479-.634-.853-1.382-.998-2.169a3.529 3.529 0 0 1-.032-.206 3.478 3.478 0 0 1 .242-1.847c.536-1.254 1.783-1.86 3.231-1.945a6.87 6.87 0 0 1 .626-.003zm-.61 3.522c.125.253.525 1.08.805 2.067.163.575.261 1.157.206 1.594-.028.218-.092.398-.197.529a.718.718 0 0 1-.467.259 1.015 1.015 0 0 1-.272.006h-.002l-.003-.001a1.73 1.73 0 0 1-.293-.062l-.048-.016a2.616 2.616 0 0 1-.364-.147 3.673 3.673 0 0 1-.454-.267c-.036-.024-.072-.05-.108-.076a5.97 5.97 0 0 1-.933-.854c1.242-1.779 2.032-2.892 2.13-3.032zM2.408 76.877l.001.009-.001.008v-.017z"/></g></svg> |
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.
Can we make the lines a bit thicker for the logo?
I also wonder if it would look better with hollow text, so the whole icon is lines. What do you think?
This is not a must though. In fact most line-wordmark icons don't have hollow/line text.
I will accept it even if the text is unchanged :)
If you wanna go for hollow text, you can use the djangorest PR as an example: https://github.com/devicons/devicon/pull/1407/files#diff-75a0ca900d5867593762acd10b934d39e826fe5c00a097f0e93a2f9a6380b53a
Examples of hollowed text:
Djangorest: https://github.com/devicons/devicon/pull/1407/files#diff-75a0ca900d5867593762acd10b934d39e826fe5c00a097f0e93a2f9a6380b53a
Unity: https://github.com/devicons/devicon/blob/develop/icons/unity/unity-line-wordmark.svg
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.
Sure, I'll create a sample. 👍
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.
How's this, @Snailedlt?
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.
Yes! Love it!
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.
@Snailedlt done
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.
LGTM! ✔️
Thank you for a great contribution! 🚀
And thank you @lunatic-fox for the plain and line versions
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.
Optimize:
plain-wordmark
line
Co-authored-by: Josélio Júnior <[email protected]>
Co-authored-by: Josélio Júnior <[email protected]>
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.
Approved! ✔
Thank you for the great contribution! 🎉
… line, line-wordmark) (devicons#1497) * new icon: rollup (original, plain) * update rollup plain icon and add rollup line icon * Update devicon.json Co-authored-by: Jørgen Kalsnes Hagen <[email protected]> * Update devicon.json Co-authored-by: Jørgen Kalsnes Hagen <[email protected]> * Fix devicon.json * add rollup wordmark logos * Update rollup-line-wordmark.svg * Update rollup-plain-wordmark.svg Co-authored-by: Josélio Júnior <[email protected]> * Update rollup-line.svg Co-authored-by: Josélio Júnior <[email protected]> Co-authored-by: Jørgen Kalsnes Hagen <[email protected]> Co-authored-by: Josélio Júnior <[email protected]>
Double check these details before you open a PR
develop
branch.devicon.json
file as seen hereThis PR closes NONE
Link to prove your SVG is correct and up-to-date.
https://rollupjs.org/guide/en/