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 OnePlus #3007

Merged
merged 3 commits into from
May 3, 2020
Merged

Add OnePlus #3007

merged 3 commits into from
May 3, 2020

Conversation

EinsEli
Copy link
Contributor

@EinsEli EinsEli commented Apr 29, 2020

oneplus

Issue: Closes #2995

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 logo is from wikipedia and the colors from the same image.

@ericcornelissen ericcornelissen added the new icon Issues or pull requests for adding a new icon label Apr 29, 2020
@runxel
Copy link
Member

runxel commented May 3, 2020

Hey @EinsEli! Thank you for your PR!
The Travis build currently fails, because the title element has to have "icon" behind the brand's name:
<title>OnePlus icon</title>.
Please add it, then we are good to go. Otherwise the svg looks great :)

@EinsEli
Copy link
Contributor Author

EinsEli commented May 3, 2020

Should be fixed now :)

@runxel
Copy link
Member

runxel commented May 3, 2020

Wonderful! Thanks again, @EinsEli! 👍

@runxel runxel merged commit a872013 into simple-icons:develop May 3, 2020
ericcornelissen added a commit that referenced this pull request May 10, 2020
# New Icons

- Badoo (#3004)
- Bose (#3038)
- Audio-Technica (#3018)
- Apache CloudStack (#3023)
- Open Badges (#2764)
- Yoast (#2967)
- OnePlus (#3007)
- Awesome Lists (#3011)

# Updated Icons

- Adobe Acrobat Reader (#3029)
- Roots (#3024)
- Adobe After Effects (#3030)
- Adobe Dreamweaver (#3030)
- Adobe Audition (#3031)
- Adobe Illustrator (#3033)
- Adobe InDesign (#3033)
- Adobe (#2769)
- gulp (#2566)
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.

Icon for OnePlus
3 participants