Skip to content
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

Magic #10307

Merged
merged 11 commits into from
Jan 18, 2024
Merged

Magic #10307

merged 11 commits into from
Jan 18, 2024

Conversation

durgeshgowdac
Copy link
Contributor

Issue: closes #10303

Website: https://www.magic.link/
Guidelines: https://magic.link/brand-assets

Similarweb rank: 83,573
Similarweb link: https://www.similarweb.com/website/magic.link/#overview

Checklist

  • 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

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Jan 17, 2024
@durgeshgowdac
Copy link
Contributor Author

@PeterShaggyNoble All checks have passed. You can merge now

Copy link
Member

@PeterShaggyNoble PeterShaggyNoble left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@durgeshgowdac, which file did you base your icon off? I'm seeing significant differences when comparing it (in red) to \Magic Logo🪄\SVG\01-Icon\2-Icon_Magic_Black.svg from the zip file of logos they make available.

And we should update the source to match the guidelines.

_data/simple-icons.json Outdated Show resolved Hide resolved
@durgeshgowdac
Copy link
Contributor Author

@PeterShaggyNoble I have made the changes, please check.

Copy link
Member

@PeterShaggyNoble PeterShaggyNoble left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks spot on now, @durgeshgowdac 👍🏻 Nice work.

@PeterShaggyNoble PeterShaggyNoble merged commit 3fca3e5 into simple-icons:develop Jan 18, 2024
3 checks passed
mondeja added a commit that referenced this pull request Jan 21, 2024
# 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)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new icon Issues or pull requests for adding a new icon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request: magic
2 participants