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 AOL #4954

Merged
merged 2 commits into from Feb 9, 2021
Merged

Add AOL #4954

merged 2 commits into from Feb 9, 2021

Conversation

theDavidBarton
Copy link
Contributor

@theDavidBarton theDavidBarton commented Feb 8, 2021

aol

Issue: closes #4942
Alexa rank: 326 (Retrieved on: 2/6/2021, 2:33PM)

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

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Feb 8, 2021
Copy link
Member

@adamrusted adamrusted left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your work on this @theDavidBarton!
Just having a quick look, it seems the optimizer has deformed the SVG around the o (see slight difference between PR in red and source in black below) - and there seem to be some unnecessary points on the path of the period. Not a deal-breaker by any means, but may be worth reducing down to 4 points total if at all possible (right-hand arrow).

image

@PeterShaggyNoble
Copy link
Member

This looks perfect now, @theDavidBarton 👍🏻 Thanks for your work on it.

@PeterShaggyNoble PeterShaggyNoble merged commit ec9643a into simple-icons:develop Feb 9, 2021
ericcornelissen added a commit that referenced this pull request Feb 14, 2021
# New Icons

- AOL (#4954)
- Blockchain.com (#4818)
- China Eastern Airlines (#4827)
- Cloudera (#4812)
- D-Wave Systems (#4825)
- Major League Hacking (#4915)
- Oxygen (#4839)
- Pug (#4714)
- Textpattern (#4966)
- Windows Terminal (#4956)

# Updated Icons

- Android Auto (#4400)
- Google Street View (#4683)
- Google Tag Manager (#4586)
- LinkedIn (#4985, #4939)
- New Japan Pro-Wrestling (#4930)
- openSUSE (#4904)
- Roundcube (#4495)
- Svelte (#4907)
@adamrusted adamrusted removed their request for review November 15, 2023 16:10
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: AOL
3 participants