Skip to content

new icon: TOML (original)#2667

Open
macmv wants to merge 1 commit intodevicons:developfrom
macmv:toml
Open

new icon: TOML (original)#2667
macmv wants to merge 1 commit intodevicons:developfrom
macmv:toml

Conversation

@macmv
Copy link

@macmv macmv commented Feb 24, 2026

Double check these details before you open a PR

  • PR does not match another non-stale PR currently opened
  • PR name matches the format new icon: Icon name (versions separated by comma). More details here
  • PR's base is the develop branch.
  • Your icons are inside a folder as seen here
  • SVG matches the standards laid out here
  • A new object is added in the devicon.json file at the correct alphabetic position as seen here

Link to prove your SVG is correct and up-to-date.

This is the SVG from the spec: https://toml.io/en/. I modified it slightly to be the correct color standalone. It's quite similar to the original in the TOML github too, although that one has different colors.

@macmv macmv changed the title *new icon: <i>TOML</i> (<i>original</i>)* new icon: TOML (original) Feb 24, 2026
Copy link
Contributor

@weh weh left a comment

Choose a reason for hiding this comment

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

Hi @macmv,

thanks for your pull request. There are some things you need to update here.

The Icon files should have a single line with a second empty line in the file. This can be done when you optimize the image e.g. with svgomg. I am not sure if svgo also replaces the polygon elements with a path, or if that should be done manually.

now tho use the icon in the font we also need a second file (in your case toml-plain.svg which has only a single svg path with a single color. I think this should not be a big thing to do with this icon.

If you need help or have questions, check out our wiki, or just ask, we are happy to help.

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