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 Amazon Pay #4793

Merged
Merged

Conversation

PeterShaggyNoble
Copy link
Member

Amazon Pay

Issue: Templarian/MaterialDesign#5612
Alexa rank: n/a

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 this SVG in source footer. This treatment does seem to be permissible under these guidelines but, for our purposes, we may need to consider the full logo:

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

Given our current line-up of Amazon icons, I would suggest the pay variant, and maybe look at updating amazonprime.svg to match the rest of them - but what do other @simple-icons/maintainers think?

image

@adamrusted adamrusted added the in discussion There is an ongoing discussion that should be finished before we can continue label Jan 19, 2021
@fbernhart
Copy link
Contributor

I prefer as well the pay variant. They use it as well on their social media presence: Twitter, YouTube, Facebook, Instagram, LinkedIn

@adamrusted We do have both, Prime and Amazon Prime. --> https://simpleicons.org/?q=prime
See #3245 and #3010 for details.

@adamrusted
Copy link
Member

There's some style guidelines available here which seem to suggest we can only use the full wordmark variation.

@PeterShaggyNoble
Copy link
Member Author

Do you mean the PDF in this zip, @adamrusted? That looks to be specifically for merchants and makes no explicit mention of not using the "pay" version. Anyway, when there's a contradiction between guidelines or lack of clarity - as there so very often is with Amazon - then I think we just go with what best suits our purposes.

@adamrusted
Copy link
Member

Happy to go with this PR if other @simple-icons/maintainers are happy with it!

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.

Merging it 😃 🤑

@service-paradis service-paradis merged commit 0a688f1 into simple-icons:develop Jan 29, 2021
@github-actions github-actions bot removed the in discussion There is an ongoing discussion that should be finished before we can continue label Jan 29, 2021
@PeterShaggyNoble PeterShaggyNoble deleted the add/amazonpay branch January 29, 2021 14:45
ericcornelissen added a commit that referenced this pull request Jan 31, 2021
# New Icons

- Aeroméxico (#4760)
- Air China (#4799)
- Alibaba Cloud (#4819)
- Amazon Pay (#4793)
- BBC (#4846)
- Chainlink (#4857)
- Chakra UI (#4851)
- China Southern Airlines (#4837)
- CountingWorks PRO (#4796)
- Delta (#4844)
- Emirates (#4862)
- Falcon (#4841)
- Jamstack (#4831)
- macOS (#4867)
- NASA (#4850)
- Nrwl (#4859)
- Okta (#4852)
- OYO (#4840)
- Premier League (#4753)
- Redux-Saga (#4803)
- Screencastify (#4806)
- SFML (#4678)
- Statuspal (#4823)
- stylelint (#4838)
- Tampermonkey (#4843)
- TV Time (#4766)
- Vector Logo Zone (#4842)
- Vectorworks (#4764)
- Zoiper (#4854)

# Updated Icons

- Anchor (#4880)
- Google AdSense (#4472)
- Google News (#4474)
- Google Scholar (#4686)
- Google Translate (#4584)
- Opel (#4813)
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

4 participants