-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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 Mintlify icon #10943
base: develop
Are you sure you want to change the base?
Add Mintlify icon #10943
Conversation
_data/simple-icons.json
Outdated
} | ||
}, | ||
{ | ||
"title": "Mintlify", | ||
"hex": "18e299", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"hex": "18e299", | |
"hex": "18E299", |
_data/simple-icons.json
Outdated
{ | ||
"title": "Mintlify", | ||
"hex": "18e299", | ||
"source": "https://github.com/simple-icons/simple-icons/blob/develop/icons/mintlify.svg" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can't link to our own repository for the source 😅
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I concur - you can use the homepage of the site as your source, as there's an SVG version of the icon there.
I've fixed the issues with the data JSON, @deevee47 - can you please make sure you are either using our prettier / eslint config, or disable your format-on-save functionality in your editor to prevent that happening again? |
It would also be nice if you could add a preview image of the icon using https://wasm.simpleicons.org/preview, as suggested in the comment of the pull request template. |
Issue: closes #10934
Popularity metric:
Checklist
_data/simple-icons.json
viewbox
is0 0 24 24
Description