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 Elk icon #10704

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Add Elk icon #10704

wants to merge 1 commit into from

Conversation

uncenter
Copy link
Member

elk

Issue: closes #10699.

Popularity metric: The Similarweb rank is 219,110. See https://www.similarweb.com/website/elk.zone/#overview.

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 Mar 26, 2024
adamrusted
adamrusted requested changes Apr 1, 2024
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 spotting some differences when comparing, granted my software didn't want to play nicely with the mask initially...

image

Can another @simple-icons/pr-reviewers verify the differences seen above?

@uncenter
Copy link
Member Author

uncenter commented Apr 1, 2024 •
edited

It's not just you! I'm seeing that now as well. Curiously I've gone and redone the icon and I'm still seeing these differences... ugh masks are the worst!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adamrusted adamrusted adamrusted requested 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: Elk
2 participants
@uncenter @adamrusted