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
Add iDEAL icon #1778
Add iDEAL icon #1778
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.
Colour and SVG look good to me, nice work 👌 Just one question @Paulsky: did you optimize the SVG using SVGO or SVGOMG?
Small tip regarding GitHub(/MarkDown): our Pull Request template looks a little like
...
- [ ] I updated the JSON data in `_data/simple-icons.json`
- [ ] I optimized the icon with SVGO or SVGOMG
- [ ] The SVG `viewbox` is `0 0 24 24`
...
Then, if you fill them in like this
...
- [x] I updated the JSON data in `_data/simple-icons.json`
- [x] I optimized the icon with SVGO or SVGOMG
- [x] The SVG `viewbox` is `0 0 24 24`
...
You get (e.g.) this
- I updated the JSON data in
_data/simple-icons.json
Instead of this
- [x ] I updated the JSON data in
_data/simple-icons.json
@ericcornelissen Sorry about that! I updated my comment. I optimized the SVG with SVGO. :) |
Okay good, just wanted to make sure 👍 Thanks for working on this, it is much appreciated!
Not a problem at all, just want to try to help you understand GitHub 🙂 |
Glad I could be of any help :) Thank you for approving/merging this PR and for your help/advice! |
# New icons - LibreOffice (#1763) - IBM (#1768) - Apache Airflow (#1766) - Cisco (#1769) - Epson (#1770) - HP (#1759) - CentOS (#1764) - Minetest (#1751) - Genius (#1777) - Lenovo (#1772) - iDEAL (#1778) - Plesk (#1781) - Bancontact (#1779) - Krita (#1775) - XRP (#1783) - Ripple (#1784) - SVG (#1790) - Nec (#1796) - Nim (#1776) - Habr (#1804) - Symantec (#1797) - Analogue (#1802) - AdGuard (#1810) - Strapi (#1787) - GIMP (#1813) - uBlock Origin (#1815) # Updated icons - Apple Podcasts (#1803 and #1805)
Issue: #1746
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description
It's the main color in the manual: https://www.ideal.nl/cms/files/Manual_iDEAL_logo.pdf
I used the SVG from this comment: iDeal Icon #1746 (comment)
No