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 Smallcase icon #10983

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Add Smallcase icon #10983

wants to merge 5 commits into from

Conversation

jglamp
Copy link
Contributor

@jglamp jglamp commented May 21, 2024

smallcase_preview

Issue: closes #10937

Popularity metric: SimilarWeb rank: 37,125.

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

Pulled SVG directly from link provided in issue. Added hex code from SVG provided. Fixes lint issue from previous pull request.

@simple-icons simple-icons bot added the new icon Issues or pull requests for adding a new icon label May 21, 2024
@adamrusted
Copy link
Member

As per section 8 of their terms and conditions we should not add this to our collection without receiving explicit permission prior. Would you like to reach out to them @jglamp?
image

@adamrusted adamrusted added the permission required Permission should be sought out from the trademark owner before including this icon in our project. label May 22, 2024
@jglamp
Copy link
Contributor Author

jglamp commented May 22, 2024

Sent an email to them today asking for permission. Will update once I hear more.

@PratyushJayachandran
Copy link
Member

Can someone from @zerodha take a look at this?

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 permission required Permission should be sought out from the trademark owner before including this icon in our project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request: Smallcase
3 participants