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 United Nations icon #7677

Merged
merged 14 commits into from Sep 16, 2023
Merged

Conversation

PaulaBarszcz
Copy link
Contributor

unitednations

Issue: closes #7609

Similarweb rank: 4,021 (https://www.similarweb.com/website/un.org/#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

Description

Svg with UN logo downloaded from:
https://commons.wikimedia.org/wiki/File:Logo_of_the_United_Nations_(B%26W).svg

I grouped paths in the UN logo using the built-in option in Inkscape.

The hex code for the UN blue was not easy to find. Even their website doesn't mention the hex code (https://www.un.org/en/about-us/un-emblem-and-flag) and there are different options found online.
My proposition is to use: #009FDC , which I found by using Color Picker on the UN flag found on Wikimedia: https://commons.wikimedia.org/wiki/File:Flag_of_the_United_Nations.svg

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Aug 15, 2022
@PaulaBarszcz PaulaBarszcz changed the title Add United Nations logo Add United Nations icon Aug 15, 2022
@LitoMore
Copy link
Member

@simple-icons/icon-review According to our contributing doc; the United Nations is acceptable, right?

@jwr1
Copy link
Contributor

jwr1 commented Apr 13, 2023

United Nations seems to describe itself as an international organization, which is allowed:

  • Allowed: International organizations and NGOs with supranational interests

Copy link
Member

@adamrusted adamrusted left a comment

Choose a reason for hiding this comment

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

The shape of this matches up perfectly, but there are a lot of extraeneous paths that we can get rid of where the landmasses overlap the circles. Would you mind having another look at this one @PaulaBarszcz?

@LitoMore
Copy link
Member

@adamrusted Those extraneous paths should be fixed now.

Copy link
Member

@PratyushJayachandran PratyushJayachandran left a comment

Choose a reason for hiding this comment

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

@adamrusted Can you review the suggested changes against new source?

_data/simple-icons.json Outdated Show resolved Hide resolved
icons/unitednations.svg Outdated Show resolved Hide resolved
_data/simple-icons.json Outdated Show resolved Hide resolved
Co-authored-by: PratyushJayachandran <35000807+PratyushJayachandran@users.noreply.github.com>
Co-authored-by: PratyushJayachandran <35000807+PratyushJayachandran@users.noreply.github.com>
Co-authored-by: PratyushJayachandran <35000807+PratyushJayachandran@users.noreply.github.com>
_data/simple-icons.json Outdated Show resolved Hide resolved
Co-authored-by: LitoMore <LitoMore@users.noreply.github.com>
_data/simple-icons.json Outdated Show resolved Hide resolved
@LitoMore LitoMore merged commit cb04095 into simple-icons:develop Sep 16, 2023
4 checks passed
mondeja added a commit that referenced this pull request Sep 17, 2023
# New Icons

- Akaunting (#7660)
- Amazon Simple Email Service (#8731)
- Avira (#7635)
- i3 (#7336)
- Internet Computer (#7706)
- Lutris (#9089)
- Mantine (#7633)
- Meilisearch (#7564)
- Mixpanel (#7404)
- Sails.js (#9525)
- Sphinx (#9315)
- SWR (#7399)
- TourBox (#9548)
- United Nations (#7677)
- uTorrent (#9565)

# Updated Icons

- Beatport (#9346)
- C Sharp (#9521)
- LG (#9542)
- Threads (#9552)
@adamrusted adamrusted removed their request for review November 15, 2023 16:08
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: United Nations
5 participants