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.

Sign up for GitHub

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

Jump to bottom

Add YouTube TV #2480

Merged
merged 4 commits into from Feb 14, 2020
Merged

Add YouTube TV #2480

merged 4 commits into from Feb 14, 2020

Conversation

PeterShaggyNoble
Copy link
Member

YouTube TV

Issue: Closes #2457

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

@PeterShaggyNoble PeterShaggyNoble added the new icon Issues or pull requests for adding a new icon label Jan 27, 2020
@ericcornelissen
Copy link
Contributor

Though I'm not against adding this version of the logo, since it is not used anywhere else other than their Android App icon (as far as I can tell), it might be better to go with the logo version of the play buttin + "TV". As seen in their Twitter avatar) and in the promotional material for the app.

What do you think @PeterShaggyNoble ?

@ericcornelissen ericcornelissen added the in discussion There is an ongoing discussion that should be finished before we can continue label Feb 14, 2020
@PeterShaggyNoble
Copy link
Member Author

From this page, it looks like that version is only for YouTube's own use, which would mean we'd have to use the full wordmark if we wanted to use the lockup. Given the options of the full wordmark and the app icon, I think the app icon is the best option for us.

They do use the icon for all their apps (e.g., https://apps.apple.com/us/app/youtube-watch-listen-stream/id544007664 & https://www.amazon.com/Google-LLC-YouTube-TV/dp/B07XF2P9KR) so should be easily recognisable.

@ericcornelissen
Copy link
Contributor

Thanks for the clarification, in that case I think this is good to merge 👍

@ericcornelissen ericcornelissen merged commit 36abec3 into simple-icons:develop Feb 14, 2020
@PeterShaggyNoble PeterShaggyNoble deleted the youtubetv branch February 14, 2020 22:58
@ericcornelissen ericcornelissen removed the in discussion There is an ongoing discussion that should be finished before we can continue label Feb 16, 2020
@ericcornelissen ericcornelissen mentioned this pull request Feb 16, 2020
Merged
ericcornelissen added a commit that referenced this pull request Feb 17, 2020
@ericcornelissen
Release 17 new icons and 10 updated icons (v2.5.0)
4e02525 
# New icons

- Google Tag Manager (#2195)
- Untangle (#2524)
- Dynamics 365 (#2523)
- AdonisJS (#2540)
- Android Studio (#2520)
- Google My Business (#2521)
- Alpine Linux (#2544)
- Apache Solr (#2576)
- YouTube Gaming (#2531)
- Bugsnag (#2545)
- Google Sheets (#2368)
- KLM (#2588)
- YouTube TV (#2480)
- Pepsi (#2595)
- Andela (#2599)
- Pleroma (#2590)
- Lighthouse (#2591)

# Updated icons

- Docker (#2561)
- CodePen (#2045)
- WhatsApp (#2581)
- Kirby (#2578)
- Reddit (#2580)
- Kickstarter (#2577)
- Quantopian (#2579)
- Google Analytics (#2564)
- Tripadvisor (#2510)
- Microsoft Word (#2557)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
new icon Issues or pull requests for adding a new icon
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

YouTube TV
2 participants
@PeterShaggyNoble @ericcornelissen