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

Add: Farcaster #10458

Merged
merged 7 commits into from
Feb 27, 2024
Merged

Conversation

tytwitchell
Copy link
Contributor

@tytwitchell tytwitchell commented Feb 13, 2024

Issue: closes #10402

Similarweb rank: 347470

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

Description

  • the hex and svg were provided in the issue description

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Feb 13, 2024
@tytwitchell tytwitchell deleted the add--Farcaster-Icon branch February 13, 2024 19:03
@tytwitchell tytwitchell restored the add--Farcaster-Icon branch February 13, 2024 19:04
@tytwitchell tytwitchell reopened this Feb 13, 2024
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.

Thanks for the contribution, @tytwitchell 👍🏻 However, your icon currently exceeds the bounds of our 24*24 canvas; could you fix that, please?

@tytwitchell
Copy link
Contributor Author

@PeterShaggyNoble this has been corrected

@adamrusted
Copy link
Member

Thanks for this one, @tytwitchell - though I would think we'd want it with the rounded rectangle around the outside, see the preview below. I've included my path below based on their CSS on the site, which you're welcome to use - but we'll need another maintainer to review if you do!
image

<svg role="img" viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg"><title>Farcaster</title><path d="M18.24.24H5.76C2.5789.24 0 2.8188 0 6v12c0 3.1811 2.5789 5.76 5.76 5.76h12.48c3.1812 0 5.76-2.5789 5.76-5.76V6C24 2.8188 21.4212.24 18.24.24m.8155 17.1662v.504c.2868-.0256.5458.1905.5439.479v.5688h-5.1437v-.5688c-.0019-.2885.2576-.5047.5443-.479v-.504c0-.22.1525-.402.358-.458l-.0095-4.3645c-.1589-1.7366-1.6402-3.0979-3.4435-3.0979-1.8038 0-3.2846 1.3613-3.4435 3.0979l-.0096 4.3578c.2276.0424.5318.2083.5395.4648v.504c.2863-.0256.5457.1905.5438.479v.5688H4.3915v-.5688c-.0019-.2885.2575-.5047.5438-.479v-.504c0-.2529.2011-.4548.4536-.4724v-7.895h-.4905L4.2898 7.008l2.6405-.0005V5.0419h9.9495v1.9656h2.8219l-.6091 2.0314h-.4901v7.8949c.2519.0177.453.2195.453.4724"/></svg>

@tytwitchell
Copy link
Contributor Author

thanks @adamrusted! I just added the styles and re-tested locally. thank you and your team for your patience with this one.

If you need anything else feel free to let me know.

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.

I'm seeing some small differences (PR in red) throughout both the left & right side of the cut-out. As we're manually recreating this from CSS, though, I'd be happy to let them slide if someone else decides they're insignificant enough.

Copy link
Member

@service-paradis service-paradis left a comment

Choose a reason for hiding this comment

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

On my side, I got much smaller differences than yours @PeterShaggyNoble . LGTM, thank you @tytwitchell!

@service-paradis service-paradis merged commit 33a61ea into simple-icons:develop Feb 27, 2024
3 checks passed
mondeja added a commit that referenced this pull request Mar 3, 2024
# New Icons

- Contributor Covenant (#10521) (@nirvikpurkait)
- Farcaster (#10458) (@tytwitchell)
- foobar2000 (#10529) (@adamrusted)
- Google Chronicle (#10548) (@durgeshgowdac)
- Image.sc (#10527) (@adamrusted)

# Updated Icons

- Pop!_OS (#10510) (@Mvlprem)
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: Farcaster
4 participants