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 Quizlet icon #10721

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

Conversation

EricRisher
Copy link

@EricRisher EricRisher commented Mar 27, 2024

quizlet

Issue: closes #10718.

Popularity metric: The Similarweb rank is 259 globally. See https://www.similarweb.com/website/quizlet.com/#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

Name

As it appears on their website

Icon

Derived from the official svg icon from https://commons.wikimedia.org/wiki/File:Quizlet_Logo_2021.svg

Color

Extracted from original svg icon color

  • #4255FF

@github-actions github-actions bot added the new icon Issues or pull requests for adding a new icon label Mar 27, 2024
@uncenter
Copy link
Member

Did you mean to open both #10720 and this PR?

@uncenter uncenter changed the title added quizlet svg logo Add Quizlet icon Mar 27, 2024
@EricRisher
Copy link
Author

Did you mean to open both #10720 and this PR?

Sorry this is my first ever pull request on a open source project, opened both by accident.

@uncenter
Copy link
Member

uncenter commented Mar 27, 2024

No worries! Looks like you figured it out :D

I've edited your original comment to fix the checkboxes and link to the original issue. Would you also like to try making a preview image for the icon? See https://wasm.simpleicons.org/preview.

icons/quizlet.svg Outdated Show resolved Hide resolved
@uncenter
Copy link
Member

uncenter commented Mar 27, 2024

Preview image looks good, nice job! I believe the issue with the SVG (which is making the check fail) is that you have an extra space before viewBox="0 0 24 24".

_data/simple-icons.json Outdated Show resolved Hide resolved
EricRisher and others added 2 commits March 27, 2024 22:04
Co-authored-by: uncenter <47499684+uncenter@users.noreply.github.com>
@EricRisher
Copy link
Author

Thanks for the help :)

Copy link
Member

Choose a reason for hiding this comment

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

This is a weird one - it looks like it's been overly simplified in some places, leading to issues like this and
this, but in other points there are weird additional points.

I'd suggest regenerating from the source, and only cutting points out where absolutely necessary.

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request: Quizlet
3 participants