-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
Added ZAP Icon #10230
Added ZAP Icon #10230
Conversation
Update ZAP source link Signed-off-by: HAHWUL <hahwul@gmail.com> Fixed bug in icon Added ZAP Icon Signed-off-by: HAHWUL <hahwul@gmail.com>
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.
Thanks for the contribution, @hahwul 👍🏻 My first instinct would have been to give it the same monochrome treatment you did, but, after reviewing their site, I think we should follow the treatment seen in their website footer.
Also, I'm getting #00549e
from the stylesheet for the colour.
And, as it's one of the very first things mentioned on their homepage, we should include the full name as an alias.
Co-authored-by: Peter Noble <15157491+PeterShaggyNoble@users.noreply.github.com>
Co-authored-by: Peter Noble <15157491+PeterShaggyNoble@users.noreply.github.com>
@PeterShaggyNoble |
Signed-off-by: HAHWUL <hahwul@gmail.com>
@PeterShaggyNoble
|
Well, that's odd, the new version is passing our linting despite being off-center and too high; we'll need to look into that. That aside, I am seeing some differences between your version and the one in their website's footer. Here's an example, with yours in red. Would you mind having another look at that, please? |
Hello again @PeterShaggyNoble :D I adjusted it to be 24x24 to match both dimensions, which might have caused a very subtle change in proportions. Could this potentially be an issue? The files included in the pull request have been modified to have dimensions of 24x24. |
The might be the cause of the linter passing when it shouldn't, alright. But it doesn't account for the differences I'm seeing; if I resize yours to the correct relative height they're still there. Could you try again, keeping the height relative to the width when you scale it down to 24dp? |
memoSvg and viewbox
Size checkLinting |
Signed-off-by: HAHWUL <hahwul@gmail.com>
I'm still seeing the same differences in that new version, @hahwul, when comparing with the SVG in the website's footer. |
Signed-off-by: HAHWUL <hahwul@gmail.com>
@PeterShaggyNoble |
Gave it a try myself and it was trickier than it first looked. Came up with the following, though, if you want to use it:
|
Signed-off-by: HAHWUL <hahwul@gmail.com>
I experimented with different solutions but couldn't resolve the issue. Ultimately, I utilized the data you provided and made a commit. Everything seems fine on my local environment, but if there are any problem, please feel free to let me know! |
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 both @hahwul and @PeterShaggyNoble!
# New Icons - 2K (#10322) - Air Transat (#10121) - Animal Planet (#10314) - BentoBox (#10284) - Deepgram (#10281) - Gameloft (#10291) - Gitconnected (#10258) - KuCoin (#10326) - Magic (#10307) - Metasploit (#10274) - MUO (#10296) - Pushbullet (#10305) - Rockstar Games (#10294) - Roon (#10044) - Salla (#10308) - Saudia (#9178) - Take-Two Interactive Software (#10302) - TeePublic (#10279) - Tilda Publishing (#10325) - Top.gg (#10240) - Uphold (#10304) - Vectary (#10297) - ZAP (#10230) # Updated Icons - Google Chat (#10257) - Tauri (#9537) - Yoast (#9398)
Issue: closes #10217
Similarweb rank:
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24