Add GitHub Copilot #9500
Merged
Add GitHub Copilot #9500
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
github-actions
bot
added
the
new icon
Closed
DenverCoder1
force-pushed
the
github-copilot
branch
from
ff5b54e
to
b433157
Compare
adamrusted
added
the
awaiting reply
adamrusted
removed
the
awaiting reply
PeterShaggyNoble
requested changes
Dec 1, 2023
There was a problem hiding this comment.
I'm seeing a few tiny differences throughout, such as this one in both eyes (your version in red), which can likely be fixed by upping the precision.
|
I downloaded the icon from https://primer.style/foundations/icons/copilot-24, unioned the paths and used the center align tool in Inkscape, and optimized with https://jakearchibald.github.io/svgomg/ (number precision set to 3) |
DenverCoder1
force-pushed
the
github-copilot
branch
2 times, most recently
from
1f7acc4
to
0da0c43
Compare
PeterShaggyNoble
approved these changes
Dec 11, 2023
There was a problem hiding this comment.
My apologies, @DenverCoder1, it turns out the differences I was seeing were a result of the source icon not being centrally aligned along the horizontal axis 🤦🏻♂️ After fixing that, the previous version lined up perfectly.
mondeja
added a commit
that referenced
this pull request
Dec 17, 2023
# New Icons - Airtel (#9709) - Biome (#10110) - Charles (#9906) - Clever Cloud (#10083) - Code::Blocks (#7335) - ComicFury (#10013) - daily.dev (#10058) - date-fns (#10093) - DBeaver (#8218) - Embark (#10124) - ERPNext (#9245) - Frappe (#9245) - GitHub Copilot (#9500) - Goldman Sachs (#9910) - Google Bigtable (#9439) - Greasy Fork (#10095) - Helium (#10049) - KX (#10051) - LibreOffice Calc (#9923) - LinuxServer (#9616) - Logitech G (#10063) - Metacritic (#10078) - NetEase Cloud Music (#10062) - OSF (#7700) - pipx (#10072) - Proteus (#8793) - Qase (#9907) - Red Bull (#6016) - Runrun.it (#10103) - Semantic UI (#10029) - Showwcase (#8593) - SimpleLogin (#10041) - SRG SSR (#10037) - Télé-Québec (#10076) - THE FINALS (#10125) - The Odin Project (#9161) - Thunderstore (#10025) - Tindie (#9996) - TrillerTV (#10097, #10112) - Uniqlo (#10102) - Uniqlo (#9754) - Virgin Atlantic (#10064) - Wondershare (#10117) # Updated Icons - Android (#10085) - Bugatti (#10099) - Channel 4 (#10100) - Citroën (#10101) - Citroën (#10101) - Counter-Strike (#10106) - Cypress (#10070) - Dolby (#10107) - Facebook (#10069) - Glassdoor (#10071) - IVECO (#10068) - Kinopoisk (#10084) - LibreOffice Base (#10059) - NBC (#10067) - Riot Games (#10105) - Slides (#10089)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue: closes #9499
Similarweb rank: 80
Checklist
_data/simple-icons.jsonviewboxis0 0 24 24Description
Note: there are two variations of the GitHub Copilot logo:
SVG v1: primer.style/design/foundations/icons/copilot-16
SVG v2: primer.style/design/foundations/icons/copilot-24
The 24x24 was the original icon in this PR, but I changed it to the 16x16 variation based on feedback from @LitoMore at #9499 (comment).
Alternative variation (not in this PR)