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 Pointy #4952

Merged
merged 3 commits into from
Mar 13, 2021
Merged

Add Pointy #4952

merged 3 commits into from
Mar 13, 2021

Conversation

PeterShaggyNoble
Copy link
Member

@PeterShaggyNoble PeterShaggyNoble commented Feb 8, 2021

Pointy

Issue: n/a (request from a client)
Alexa rank: ~162k

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

Icon from SVG found in source header, colour from stylesheet.

@PeterShaggyNoble PeterShaggyNoble added the new icon Issues or pull requests for adding a new icon label Feb 8, 2021
@adamrusted
Copy link
Member

I think linking to https://www.pointy.com/gb/vend or https://www.pointy.com/ie/vend should be fine. Though the SVG has a slight imperfection, it's easily corrected by just deleting the inner point of the issue and re-joining the SVG. Following that process, I've been able to get the below path - which I've compared against yours with some differences, which may just be from me optimizing it...

<svg role="img" viewBox="0 0 24 24" xmlns="http://www.w3.org/2000/svg"><title>Pointy</title><path d="M8.076.025S4.52.234 2.833 2.751c-1.58 2.351-1.465 5.145-1.1 8.121C2.096 13.831 2.587 24 2.587 24c.002.003 11.235-11.526 11.23-11.506 1.75-1.805 2.408-4.468 2.395-5.961-.037-4.274-3.461-6.815-8.136-6.508zm.777 10.774c-1.991 0-3.604-1.632-3.604-3.645 0-2.015 1.614-3.649 3.604-3.649s3.642 1.512 3.642 3.527c0 2.011-1.652 3.767-3.642 3.767zm2.765-3.741a1.58 1.58 0 1 1-3.162 0 1.58 1.58 0 0 1 3.162 0zm10.879 1.431s-2.325.158-3.644.57c-1.317.413-2.502 1.076-2.502 1.076s.495-.852.705-2.361c.207-1.511-.14-2.652-.14-2.652l5.581 3.367z"/></svg>

@PeterShaggyNoble
Copy link
Member Author

PeterShaggyNoble commented Feb 24, 2021

I've been able to get the below path

Thanks, @adamrusted, that lines up perfectly with the source, other than the imperfection. After wasting so much time on that S3 icon, I've run with your path here so we'll need one of the other @simple-icons/maintainers to review it before merging.

I've compared against yours with some differences, which may just be from me optimizing it

The are differences between the 2 source files, which would account for that.

@fbernhart
Copy link
Contributor

LGTM 👍🏼 🐦

@fbernhart fbernhart merged commit 3f266c9 into simple-icons:develop Mar 13, 2021
ericcornelissen added a commit that referenced this pull request Mar 14, 2021
# New Icons

- Amazon S3 (#4999)
- Commerzbank (#5028)
- DatoCMS (#5184)
- Enpass (#5174)
- Hyper (#5197)
- Mercurial (#5157)
- Pointy (#4952)
- Private Internet Access (#5158)
- Stimulus (#5209)
- SymPy (#4554)
- XState (#4971)

# Updated Icons

- AlliedModders (#5107)
- Code Climate (#5194)
- Elsevier (#5175)
- Firefox (#5203)
- Git (#5213)
- Google Ads (#4471)
- Monzo (#5219)
- Notepad++ (#5187)
- pre-commit (#5226)
- SparkFun (#5196)
- Stack Overflow (#5217)
@PeterShaggyNoble PeterShaggyNoble deleted the add/pointy branch March 15, 2021 10:45
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.

None yet

3 participants