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 Clarivate icon #9142

Merged
merged 5 commits into from Aug 10, 2023
Merged

Add Clarivate icon #9142

merged 5 commits into from Aug 10, 2023

Conversation

PratyushJayachandran
Copy link
Member

clarivate

Issue: closes #9103

Similarweb rank: 10,994. See https://www.similarweb.com/website/clarivate.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 Media Resource Kit PDF from here https://clarivate.com/newsroom/media-resources/
Color: Taken from website header #FFFFFF

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Jul 29, 2023
adamrusted
adamrusted requested changes Jul 29, 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.

I'm seeing some differences between your icon and the SVG in the website header. Could you take another look please @PratyushJayachandran 🙏

@PratyushJayachandran
Copy link
Member Author

PratyushJayachandran commented Jul 30, 2023 •
edited

I'm seeing some differences

I had taken from the PDF but I have redone it from website header SVG. Please review again.
clarivate

adamrusted
adamrusted approved these changes Aug 10, 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.

LGTM, thanks @PratyushJayachandran!
I'll update the source to just the homepage, and get this merged.

@adamrusted adamrusted merged commit fc6b38c into simple-icons:develop Aug 10, 2023
3 checks passed
@github-actions github-actions bot mentioned this pull request Aug 13, 2023
Merged
mondeja added a commit that referenced this pull request Aug 13, 2023
@mondeja
Release 5 new icons and 5 updated icons (v9.10.0)
c11c1d8 
# New Icons

- Clarivate (#9142)
- Distrokid (#9143)
- Google Bard (#9114)
- ImprovMX (#9004)
- InfoQ (#9148)

# Updated Icons

- Astro (#9201)
- SmugMug (#9237)
- Sourcegraph (#9174)
- Zcash (#9247)
- Zoom (#9211)
@PratyushJayachandran PratyushJayachandran mentioned this pull request Aug 13, 2023
Merged
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@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.

Request: Clarivate
2 participants
@PratyushJayachandran @adamrusted