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

Ferrari Corp (#2059) #2360

Merged
merged 10 commits into from Feb 4, 2020
Merged

Ferrari Corp (#2059) #2360

merged 10 commits into from Feb 4, 2020

Conversation

runxel
Copy link
Member

@runxel runxel commented Jan 13, 2020

Issue:
#2059

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

The wordmark was in the header of the website. We could add the horse, but that's just an extra, the wordmark stands on it's own normally.
Color is debatable. I think it must be red: #F20300 (eyedropped from website).
But it could also be the yellow #ffff01 which is used as background, when you have the horse.


Addition: The scope is now slightly turned; it's about the Ferrari corporation as whole, not only the cars, that's why it is the wordmark and not the horse – which will be added with another PR.

@runxel runxel added the new icon Issues or pull requests for adding a new icon label Jan 13, 2020
@PeterShaggyNoble
Copy link
Member

From their websites' stylesheets, it looks like the colour should be #eb2e2c. Although a strong case could probably also be made for rosso corsa, #d40000.

@runxel runxel added the in discussion There is an ongoing discussion that should be finished before we can continue label Jan 13, 2020
@PeterShaggyNoble
Copy link
Member

PeterShaggyNoble commented Jan 13, 2020

@runxel, you might need to up the precision on the logo; there are a couple of hairline differences in the e and the a between your version (red) and the original (green).

Ferrari

However, in the absence of any brand guidelines that I could find, I think we should use the horse on its own, as they do in their favicon and splash screen - it lends itself better to a 24*24 icon. What do the other @simple-icons/maintainers think?

@runxel
Copy link
Member Author

runxel commented Jan 13, 2020

@runxel, you might need to up the precision on the logo; there are a couple of hairline differences in the e and the a between your version (red) and the original (green).

Oh yeah, that was being expected.
Their a has some problems I've taken care of. Also on of the handles at the e is also most probably false.
I've to kindly decline to revert at least the change on the a 😅

The brand guide lines are behind a log in form, I think, btw.

@phatbhoy67
Copy link
Contributor

There is an official .svg version of the horse in the header of their store website.

@runxel
Copy link
Member Author

runxel commented Jan 13, 2020

Good find, @phatbhoy67!

I'd still say, the horse alone is not enough to present the brand :/
Maybe together with the wordmark?

@phatbhoy67
Copy link
Contributor

The wordmark is iconic. The "prancing horse" motif is their icon.

@runxel
Copy link
Member Author

runxel commented Jan 13, 2020

I know, I know... So if everybody is for just the horse I will do so!

@phatbhoy67
Copy link
Contributor

If you asked me to describe it without looking I'd probably include the shield. 🤔

@PeterShaggyNoble
Copy link
Member

The version with the SF in the shield is the logo for Scuderia Ferrari.

But they do have many different versions of it. Looking at a random car they have the shield version on the side panels, a rectangular version on the bonnet and the prancing horse on its own on the grille, which is the version they use on their website.

An option could be to add both, as was done with Yamaha; the shield version as Scuderia Ferrari and the version below (extracted from their website source) as Ferrari.

Ferrari

@runxel
Copy link
Member Author

runxel commented Jan 15, 2020

An option could be to add both, as was done with Yamaha; the shield version as Scuderia Ferrari and the version below (extracted from their website source) as Ferrari.

Wait, don't you mean the shield with the horse as Scuderia Ferrari, and Ferrari with the wordmark?
Otherwise it's just the horse two times... 🐴

@PeterShaggyNoble
Copy link
Member

I'm suggesting the horse on the shield with the SF Scuderia Ferrari and the horse on its own, as above, for Ferrari.

@runxel
Copy link
Member Author

runxel commented Jan 15, 2020

Okay, I see.
Nevertheless I think that we don't need the horse twice. (doesn't matter with or without shield)
I would at least expect the wordmark for once.
But lets listen to someone else from the @simple-icons/maintainers! :)

@ericcornelissen
Copy link
Contributor

I would say the horse (without shield) is recognizable for Ferrari, personally I think it is more recognizable than the wordmark. The horse is used in various forms (in a shield, in a square, in isolation) and is used to represent their brand on all of their social media (based on which we could argue it should be inside rectangle). Also, it is more appropriate for our purposes, I think.

I might be confused about the difference between Ferrari and Ferrari Corporate but for Ferrari I think we should definitely use the horse (without shield). The horse with shield could be added, on request, for Scuderia Ferrari. Potentially we could add the Ferrari wordmark for Ferrari Corporate?

But correct me if I got the "Ferrari" brand incorrectly here 😅

@phatbhoy67
Copy link
Contributor

Just the horse without a-neigh accoutrements for me 👀

@runxel
Copy link
Member Author

runxel commented Jan 17, 2020

Okay great! 💯
To sum it up:

  • Ferrari → horse, nothing else.
  • Scuderia Ferrari → horse in shield (later PR)
  • Ferrari Corp. (this PR) → the wordmark.

Maybe you want to make a Ferrari PR, @PeterShaggyNoble, since you already have it all together? :)
That would be sick.

Ferrari

Update the naming according to simple-icons#2360 (comment)
This allows for the other Ferrari related symbols (horse!) to be added.
@PeterShaggyNoble

This comment has been minimized.

@runxel runxel mentioned this pull request Jan 17, 2020
Man I'm dumb. Always forget to rename the title element....
@runxel runxel changed the title Ferrari (#2059) Ferrari Corp (#2059) Jan 17, 2020
@PeterShaggyNoble PeterShaggyNoble removed the in discussion There is an ongoing discussion that should be finished before we can continue label Jan 17, 2020
@ericcornelissen
Copy link
Contributor

Just wondering, why Ferrari Corp rather than Ferrari Corperate (or the native Ferrari N.V., see Wikipedia) 🤔

@runxel
Copy link
Member Author

runxel commented Jan 19, 2020

Just wondering, why Ferrari Corp rather than Ferrari Corperate (or the native Ferrari N.V., see Wikipedia) 🤔

Good point, like always!
Will change that to Ferrari N.V.
Does the file be named ferrarin-dot-v-dot then? 😅

@ericcornelissen
Copy link
Contributor

Does the file be named ferrarin-dot-v-dot then?

I think so 🤔😅

Will change that to Ferrari N.V.

I'm equally fine with Ferrari Corporate which would improve the filename 😄 Any other opinions on which name we should use @simple-icons/maintainers

@PeterShaggyNoble PeterShaggyNoble mentioned this pull request Jan 21, 2020
3 tasks
@PeterShaggyNoble
Copy link
Member

All looks good now, @runxel but, before merging, where did we land on the colour?

@runxel
Copy link
Member Author

runxel commented Jan 31, 2020

You already got the rosso corsa, so I will take the red from the stylesheet :)

@PeterShaggyNoble
Copy link
Member

Thanks for all your work on this one, @runxel

@PeterShaggyNoble PeterShaggyNoble merged commit 3674683 into simple-icons:develop Feb 4, 2020
@runxel runxel deleted the ferrari branch February 4, 2020 12:23
ericcornelissen added a commit that referenced this pull request Feb 5, 2020
# New icons

- Elixir (#2242)
- Cinema4d (#2376)
- SEAT (#2351)
- MAAS (#2391)
- Fila (#2405)
- Ferrari (#2431)
- OpenBSD (#2426)
- Zigbee (#2382)
- Google Translate (#2430)
- LG (#2401)
- FIFA (#2404)
- Fluentd (#2415)
- Google Messages (#2367)
- JCB (#2389)
- Google Cast (#2352)
- Vuetify (#2417)
- Spinnaker icon (#2443)
- Microsoft SQL Server (#2453, 5c9c622)
- pr.co (#2456)
- Khronos Group (#2448)
- OpenGL (#2450)
- Vulkan (#2451)
- WebGL (#2452)
- WebRTC (#2454)
- Symphony (#2468)
- Veritas (#2326)
- Beatport (#2471)
- Hugo (#2455)
- Font Awesome (#2447)
- pre-commit (#2476)
- Aircall (#2466)
- Apache ECharts (#2473)
- WooCommerce (#2461)
- MIDI (#2483)
- Serverless (#2484)
- Woo (#2479)
- Katana (#2497)
- Houdini (#2498)
- Google Cardboard (#2502)
- Nuke (#2496)
- Concourse (#2427)
- NixOS (#2429)
- MariaDB (#2506)
- MariaDB Foundation (#2402)
- Google Classroom (#2482)
- Open Containers Initiative (#2490)
- Fastly (#2500)
- Peugeot (#2515)
- WP Rocket (#2432)
- Affinity Photo (#2423)
- Google Nearby (#2508)
- haveibeenpwned (#2517)
- Ferrari Corp (#2360)
- Affinity Designer (#2422)
- LabVIEW (#2435)
- Affinity Publisher (#2424)
- Affinity (#2425)
- NFC (#2501)

# Updated icons

- Twitch (#2118)
- Pandora (#2428)
- Babel (#2499)
@runxel runxel mentioned this pull request Mar 6, 2020
48 tasks
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