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 arXiv icon #1856

Merged
merged 1 commit into from Nov 2, 2019
Merged

Conversation

foo-dogsquared
Copy link
Contributor

Issue: #1854

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

Seems like the wordmark is more appropriate here since it is pretty recognized and it is the face of their social media (Twitter for a larger example).

@birjj
Copy link
Contributor

birjj commented Oct 29, 2019

I agree that the wordmark seems to be the right choice here (for context it's also used as the favicon of their blog and their logo on GitHub), but I could probably be convinced to use the favicon instead too, if a high-res version is found as a reference.

Given that that's a subjective thing I'll let one of the other @simple-icons/maintainers merge this in.

@birjj birjj added the new icon Issues or pull requests for adding a new icon label Oct 29, 2019
@phatbhoy67
Copy link
Contributor

phatbhoy67 commented Oct 31, 2019

Hey @birjolaxew, there is an official .svg for the favicon in this "about" page source.

@birjj
Copy link
Contributor

birjj commented Nov 1, 2019

Hmm. I personally still think that the wordmark is more appropriate than the logomark, solely because the logomark only seems to be used as their favicon and nowhere else. Thoughts @simple-icons/maintainers (maybe specifically @ericcornelissen, since everyone else is already involved)?

Copy link
Contributor

@ericcornelissen ericcornelissen left a comment

Choose a reason for hiding this comment

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

I agree that the wordmark is a better choice her for the reason pointes out by @birjolaxew

@ericcornelissen ericcornelissen merged commit b7b7c7e into simple-icons:develop Nov 2, 2019
@birjj birjj mentioned this pull request Nov 3, 2019
@foo-dogsquared foo-dogsquared deleted the arxiv branch June 22, 2021 08:30
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