Skip to content

Conversation

@s0ul141
Copy link

@s0ul141 s0ul141 commented Jul 27, 2024

Adding a New Icon

@canaleal
Copy link
Member

Hey @s0ul141, I see that you're a first-time contributor to Devicons. First of all, welcome! Also, when submitting new icons, please make sure your Pull Requests follow the steps outlined in the guidelines.

Right now, the icon has not been added to the devicon.json, and the SVGs are not created properly or named correctly.

If you need any help creating the SVGs feel free to reach out and we'll be happy to help.

@s0ul141
Copy link
Author

s0ul141 commented Jul 27, 2024

Hey @s0ul141, I see that you're a first-time contributor to Devicons. First of all, welcome! Also, when submitting new icons, please make sure your Pull Requests follow the steps outlined in the guidelines.

Right now, the icon has not been added to the devicon.json, and the SVGs are not created properly or named correctly.

If you need any help creating the SVGs feel free to reach out and we'll be happy to help.

Yes please guide me about creating SVGs

@canaleal
Copy link
Member

Sure!

First, instead of using PNGs, use SVGs. We can use this SVG for the original and this for the original-wordmark.

Note that the only supported SVG versions are original, original-wordmark, plain, plain-wordmark, line, and line-wordmark. You can read all about the different versions here.

Here is a long tutorial on how to update the SVGs inside a program called inkscape and how to optimize them.

@s0ul141
Copy link
Author

s0ul141 commented Jul 27, 2024 via email

@s0ul141
Copy link
Author

s0ul141 commented Aug 1, 2024

Added Seaborn and Geopandas

@canaleal
Copy link
Member

@s0ul141 You'll need to split each SVG into their own PRs. (Geopandas, seaborn, statsmodels)

@canaleal
Copy link
Member

canaleal commented Mar 8, 2025

Closing as it's been some time since any comments and the MR will have to be split into 3 sperate PR anways.

New MRs should be made for:

  • geopandas
  • seaborn
  • statsmodels

@canaleal canaleal closed this Mar 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants