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 iHeartRadio #3106
Add iHeartRadio #3106
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution, @thrash4life 👍
Unfortunately, from their brand guidelines:
DON'T use the Logo Mark itself should never be used without the correct usage of the Logo Type.
So we'll need to use the stacked version with the wordmark included here. I'd also suggest updating the source URL to https://brand.iheart.com/logo
Crap - I need some help here @ericcornelissen and @PeterShaggyNoble I can't reduce the SVG down any further. |
This comment has been minimized.
This comment has been minimized.
@thrash4life I agree, their branding is inconsistent. I just uploaded the Stitcher logo and and wondering the same thing. |
@thrash4life and @NovaGL, it is not uncommon for brands to specify something in their brand guidelines that they don't adhere to themselves, typically in places such as app icons. From my/our perspective, it is quite reasonable for them to do this (especially in places such as app icons) while requiring that outsiders adhere to the guidelines. This is simply to ensure that you don't use their logo to mean something else. Therefore, generally speaking, we prefer to follow explicit guidelines and include text if necessary even if app icons (or similar) do not include text. |
Can you elaborate @thrash4life? I did not do a thorough comparison to the original, but this seems to match the source quite well. |
Other than it being a shade over |
I created the SVG from an EPS rather than from the AI, so now there's a single path (no clip path) which also permitted the SI Preview Generator to render as expected.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update, @thrash4life
However, your path is coming in at 23.179
high, rather than 24
. You should also be able to drop the precision a good bit on it when optimising it (although I haven't checked myself yet).
Looks perfect to me know - thanks for your efforts on this one, @thrash4life 👍 |
It was fun and I was happy to contribute again. Thanks for the encouragement and support @PeterShaggyNoble and @ericcornelissen |
Looking forward to your future contributions 🙂 |
Issue: Closes #3102
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24