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 N26 #3133

Merged
merged 2 commits into from Jun 2, 2020
Merged

Add N26 #3133

merged 2 commits into from Jun 2, 2020

Conversation

vigonotion
Copy link
Contributor

@vigonotion vigonotion commented May 29, 2020

N26

Issue: -

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

Adds the N26 logo (app-based bank from Germany)

Alexa Rank: ~11,834

Logo has been extracted from the website header (top left).

Color #5FAD97 has been extracted from their app logo: https://n26.com/en-de/logo-assets

@PeterShaggyNoble PeterShaggyNoble added the new icon Issues or pull requests for adding a new icon label May 29, 2020
@PeterShaggyNoble
Copy link
Member

Nice work, @vigonotion - SVG looks perfect to me 👍

Not entirely sold on the choice of colour, but I can see why you went with it. My first instinct would have been to go with the #266678 shade as it's the most dominant on their site. If others agree that the lighter shade is the correct choice then I'd suggest #48ac98 from their stylesheet, as eyedropping from an image isn't always 100% reliable.

@PeterShaggyNoble PeterShaggyNoble added the in discussion There is an ongoing discussion that should be finished before we can continue label May 29, 2020
@vigonotion
Copy link
Contributor Author

#266678 is only dominant on their website, which is not really used that much as N26 is mainly an app based bank. I can confirm the color #48ac98, the Android app icon actually also uses that color.

Also look at the first video in the app description, especially the splash screen in the beginning, to get an idea of their branding:
https://play.google.com/store/apps/details?id=de.number26.android

@PeterShaggyNoble
Copy link
Member

OK, I'm happy to run with that colour, so. Thanks for your work on this one, @vigonotion.

@PeterShaggyNoble PeterShaggyNoble removed the in discussion There is an ongoing discussion that should be finished before we can continue label Jun 2, 2020
@PeterShaggyNoble PeterShaggyNoble merged commit 4b87214 into simple-icons:develop Jun 2, 2020
ericcornelissen pushed a commit that referenced this pull request Jun 7, 2020
# New Icons

- GeeksforGeeks (#3072)
- Jasmine (#3152)
- KTM (#2997)
- Metro de la Ciudad de México (#3156)
- Metro de Madrid (#3153)
- N26 (#3133)
- Prime Video (#3025)
- Scratch (#2872)
- Shenzhen Metro (#3158)
- Tokyo Metro (#3157)
- TuneIn (#3115)
- VirusTotal (#3182)

# Updated Icons

- Bitwarden (#3150)
- SpaceX (#3159)
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

2 participants