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 Pearson icon #9030

Merged
merged 8 commits into from Jul 23, 2023
Merged

Add Pearson icon #9030

merged 8 commits into from Jul 23, 2023

Conversation

PratyushJayachandran
Copy link
Member

pearson
Issue: closes #5865

Similarweb rank: 1,504. See https://www.similarweb.com/website/pearson.com

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

SVG: Taken from website https://www.pearson.com
Color: Taken from logo #000000. Also mentioned in the branding guidelines which I cannot find a landing page for.

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Jul 16, 2023
@adamrusted
Copy link
Member

Also worth noting the exact same logo is used for Pearson VUE, the online testing platform. We might want to add this in as an AKA alias.

@PratyushJayachandran
Copy link
Member Author

Also worth noting the exact same logo is used for Pearson VUE, the online testing platform. We might want to add this in as an AKA alias.

Added and pushed commit. Thanks for the suggestion 💯

@LitoMore
Copy link
Member

LitoMore commented Jul 20, 2023 •
edited

image

@PratyushJayachandran I picked the resource from their homepage, but I found some differences.

@PratyushJayachandran
Copy link
Member Author

image

@PratyushJayachandran I picked the resource from their homepage, but I found some differences.

I have tried to fix it. Hope the new commit resolves it. Thanks for the review.

@adamrusted adamrusted requested a review from LitoMore July 21, 2023 07:02
@LitoMore LitoMore removed their assignment Jul 21, 2023
LitoMore
LitoMore requested changes Jul 21, 2023
View reviewed changes
Copy link
Member

@LitoMore LitoMore left a comment

Choose a reason for hiding this comment

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

image

Hey @PratyushJayachandran, please review your changes before committing. Thank you.

@PratyushJayachandran
Copy link
Member Author

PratyushJayachandran commented Jul 21, 2023 •
edited

Hey @PratyushJayachandran, please review your changes before committing. Thank you.

Oops my bad. It was looking fine in my editing app but then after optimization it added tags which I manually removed. Usually it works but I forgot to recheck this time.

I'll fix it.

@PratyushJayachandran
Copy link
Member Author

Hey @PratyushJayachandran, please review your changes before committing. Thank you.

I fixed it.

adamrusted
adamrusted approved these changes Jul 23, 2023
View reviewed changes
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.

Icon itself is a perfect match, just an amendment to the JSON, which I should've proposed instead of the 'aka' entry and then we're good to go. I'll merge this in myself - save you doing it and re-requesting a review @PratyushJayachandran 🎉

_data/simple-icons.json Outdated Show resolved Hide resolved
@adamrusted adamrusted merged commit a9c96b2 into simple-icons:develop Jul 23, 2023
3 checks passed
@PratyushJayachandran PratyushJayachandran deleted the Add-Pearson branch July 23, 2023 19:53
@github-actions github-actions bot mentioned this pull request Jul 30, 2023
Merged
mondeja added a commit that referenced this pull request Jul 30, 2023
@mondeja
Release 44 new icons and 12 updated icons (v9.8.0)
64a2035 
# New Icons

- Adroll (#9040)
- Airbrake.io (#8976)
- Alteryx (#9008)
- Amazon Identity Access Management (#8739)
- Apifox (#9123)
- BentoML (#9056)
- BisectHosting (#9027)
- Bombardier (#9029)
- Brevo (#9075)
- bspwm (#9025)
- Chatwoot (#7005)
- Creality (#9130)
- EasyEDA (#9135)
- F# (#9011)
- Google Docs (#8981)
- Google Forms (#9080)
- HashiCorp (#9037)
- Intuit (#9032)
- IRIS (#5796)
- Jovian (#8989)
- Miraheze (#9054)
- NBC (#5431)
- NgRx (#9082)
- Open Source Hardware (#9125)
- Opera GX (#8750)
- Palo Alto Networks (#9038)
- Paychex (#8998)
- Pearson (#9030)
- Phoenix Framework (#9007)
- Podcast Index (#9100)
- PythonAnywhere (#9033)
- Qlik (#8964)
- Radix UI (#9079)
- remark (#9022)
- Sage (#9031)
- Semrush (#9036)
- SST (#9063)
- Stryker (#7388)
- SVG.js (#9083)
- Thangs (#9023)
- Thurgauer Kantonalbank (#9012)
- Tina (#9097)
- Voron Design (#8961)
- Zod (#9078)

# Updated Icons

- Arduino (#9145)
- Bose (#9144)
- Zendesk (#9124)
- Zenn (#9124)
- Zerply (#9124)
- Zettlr (#9124)
- Zig (#9124)
- Zigbee (#9124)
- Zilch (#9124)
- ZincSearch (#9124)
- Zingat (#9124)
- Zorin (#9124)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LitoMore LitoMore LitoMore requested changes

@adamrusted adamrusted adamrusted approved these changes

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.

Pearson
3 participants
@PratyushJayachandran @adamrusted @LitoMore