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 Audiomack icon #2230

Merged
merged 2 commits into from Dec 27, 2019
Merged

Add Audiomack icon #2230

merged 2 commits into from Dec 27, 2019

Conversation

ayo-audiomack
Copy link
Contributor

@ayo-audiomack ayo-audiomack commented Dec 26, 2019

Issue: Closes #2229

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

Creative assets from Audiomack's CCO @purenine and the official style guide

@PeterShaggyNoble
Copy link
Member

Thanks for your work on this one, @aaya-audiomack. However, could you update the SVG to fill the full width of the canvas?

@ayo-audiomack
Copy link
Contributor Author

Thanks for catching the error @PeterShaggyNoble. Does this look okay now?

@runxel runxel added the new icon Issues or pull requests for adding a new icon label Dec 26, 2019
@PeterShaggyNoble
Copy link
Member

Thanks for the quick update, @aaya-audiomack, this looks good to me now but, as I'm on my phone, I'll leave it to one of the other @simple-icons/maintainers to review more thoroughly before merging.

@runxel
Copy link
Member

runxel commented Dec 27, 2019

Looks fantastic! Thank you for the PR @aaya-audiomack 👍

@runxel runxel merged commit 5e3a344 into simple-icons:develop Dec 27, 2019
ericcornelissen added a commit that referenced this pull request Jan 7, 2020
# New icons

- GNU Bash (#2185)
- Z-Wave (#2212)
- Convertio (#2213)
- Material UI (#2211)
- Datadog (#2231)
- Audiomack (#2230)
- New York Times (#2224)
- EA (#2237)
- Square Enix (#2236)
- Authy (#2233)
- Activision (#2235)
- ActiGraph (#2234)
- Celery (#2110)
- Photocrowd (#2193)
- WebAssembly (#2244)
- Alipay (#2241)
- ZDF (#2199)
- NDR (#2200)
- Sat.1 (#2203)
- TELE5 (#2206)
- Battle.net (#2215)
- Phonegap (#2247)
- GNU Emacs (#2248)
- Corona Engine (#2245)
- Asciinema (#2218)
- RTLZWEI (#2202)
- figshare (#2220)
- Tomorrowland (#2217)
- iFood (#2209)
- Litecoin (#2210)
- Freelancer (#2254)
- Fandom (#2263)
- Unicode (#2257)
- Pexels (#2261)
- Airtable (#2274)
- XMPP (#2085, #2275)
- Red Hat Open Shift (#2277)
- FITE (#2262)
- Qi (#2255)
- Sencha (#2246)
- Neovim (#2251)
- Qualtrics (#2285)
- VMware (#2265)

# Updated icons

- JSFiddle (#2214)
- Campaign Monitor (#2259)
- Helm (#2273)
- Accusoft (#2291)
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.

Request: Audiomack Icon
3 participants