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 Yamaha Corporation icon #1869

Merged
merged 4 commits into from Nov 1, 2019
Merged

Add Yamaha Corporation icon #1869

merged 4 commits into from Nov 1, 2019

Conversation

phatbhoy67
Copy link
Contributor

**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

See issue #1553

Source is official .svg available from header of https://www.yamaha.com/en/.

The logo construction is as outlined in Yamaha logo history.

Chosen the hex value #4B1E78, which is used in the logo.

Built on GitPod, working as expected.

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

birjj commented Nov 1, 2019

It is a bit confusing, but I think the official name is "Yamaha Corporation"? They refer to the logo as "the Yamaha Logo", but it seems that the company itself uses the full name consistently. It also mentions:

1987. The Company changed its name to Yamaha Corporation to mark the 100th anniversary of its founding. Removed the tuning forks mark and emphasized "Yamaha."

This would also make it more inline with the Yamaha Motor Corporation (#1868). Could you update @phatbhoy67?

@phatbhoy67 phatbhoy67 changed the title Add Yamaha icon Add Yamaha Corporation icon Nov 1, 2019
@birjj birjj merged commit 1677bf3 into simple-icons:develop Nov 1, 2019
@birjj
Copy link
Contributor

birjj commented Nov 1, 2019

👍 Thanks for taking care of this one!

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

3 participants