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

Jump to bottom

Update Linktree icon #7962

Merged
merged 5 commits into from Oct 16, 2022
Merged

Update Linktree icon #7962

merged 5 commits into from Oct 16, 2022

Conversation

Poukame
Copy link
Contributor

@Poukame Poukame commented Oct 15, 2022

linktree

Issue: closes #7955

Similarweb rank:
#432000

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

Got the new hex color from their favicon.

@github-actions github-actions bot added the update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both label Oct 15, 2022
dirien
dirien requested changes Oct 16, 2022
View reviewed changes
Copy link
Member

@dirien dirien left a comment

Choose a reason for hiding this comment

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

Hi ,

Thanks for your contribution to simple-icons!

I see some differences:
image

I used the header icon from the website. Maybe check for the precision?

@Poukame
Copy link
Contributor Author

Poukame commented Oct 16, 2022

Hello @dirien,

I've refined it slighly. Which icon did you compare to? The tree alone or logo with text?
In my process, I used svg-grabber on their website and took the tree logo alone.

When I diff with this one, I'm near 100% match.

Hope it's gonna be ok.

2022-10-16_14-23

dirien
dirien approved these changes Oct 16, 2022
View reviewed changes
Copy link
Member

@dirien dirien left a comment

Choose a reason for hiding this comment

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

Hi @Poukame ,

Thanks for your contribution to simple-icons!

It's a perfect fit:
image

LitoMore
LitoMore reviewed Oct 16, 2022
View reviewed changes
_data/simple-icons.json Outdated Show resolved Hide resolved
@dirien dirien merged commit 3f99027 into simple-icons:develop Oct 16, 2022
@Poukame Poukame deleted the linktree branch October 21, 2022 07:49
@github-actions github-actions bot mentioned this pull request Oct 23, 2022
Merged
mondeja added a commit that referenced this pull request Oct 23, 2022
@mondeja
Release 5 new icons and 1 updated icon (v7.17.0)
749e2bf 
# New Icons

- Backstage (#7949, #7965)
- E3 (#7876)
- Elgato (#7884)
- Iced (#7961, #7966)
- StandardJS (#7929)

# Updated Icons

- Linktree (#7962)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@LitoMore LitoMore LitoMore left review comments

@dirien dirien dirien approved these changes

Assignees
No one assigned
Labels
update icon/data Issues or pull requests regarding icons that are outdated, this can be the SVG or color or both
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Linktree icon out of date
3 participants
@Poukame @LitoMore @dirien