Skip to content

feat(icon): add macaulay2 icon#3349

Merged
PKief merged 4 commits intomaterial-extensions:mainfrom
SepehrAkbari:icon/macaulay2
Feb 21, 2026
Merged

feat(icon): add macaulay2 icon#3349
PKief merged 4 commits intomaterial-extensions:mainfrom
SepehrAkbari:icon/macaulay2

Conversation

@SepehrAkbari
Copy link
Copy Markdown
Contributor

Description

This PR adds the official icon of Macaulay2, a software system for research in algebraic geometry, to files with ".m2" extension.

Macaulay2 Website
Macaulay2 Repository

The screenshot below shows the changes:

m2-test

Contribution Guidelines

@github-actions github-actions bot added the icons PR with new icons label Feb 7, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Feb 7, 2026

Preview

Thank you for creating a pull request. This preview shows you how your icon will look on the different themes:

Generated preview

Check how your icon fits in a 16x16 grid with our Pixel Perfect Checker by following this link.

You can find more information on how to contribute in the contribution guidelines.

@PKief
Copy link
Copy Markdown
Member

PKief commented Feb 13, 2026

@SepehrAkbari thanks for the PR. We have to align the icon to be more pixel perfect, you can see that the shapes of the SVG are not following our grid in this preview:
https://pixp.pages.dev/material-extensions/vscode-material-icon-theme/pull/3349

Additionally, we should have at least one pixel whitespace around the icon

@SepehrAkbari
Copy link
Copy Markdown
Contributor Author

@PKief Thanks for the comment. This version should be pixel perfect, and have a bigger margin.

@PKief
Copy link
Copy Markdown
Member

PKief commented Feb 17, 2026

@SepehrAkbari thanks for the adjustments. However, based on the preview I can still see that some paths are cutting our grid lines although it looks like we could easily adjust it to follow the grid:

image

Please make it possible that edges are not in the middle of a square/pixel but that the squares are completely filled

@PKief
Copy link
Copy Markdown
Member

PKief commented Feb 17, 2026

let me know if you need some help, then I can fix it once I'm free.

@SepehrAkbari
Copy link
Copy Markdown
Contributor Author

@PKief Thanks for clarifying. Does this work? Let me know if anything else needs modifying.

Copy link
Copy Markdown
Member

@PKief PKief left a comment

Choose a reason for hiding this comment

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

It looks ok, now! Thanks for the contribution.

@PKief PKief merged commit 6e8da9c into material-extensions:main Feb 21, 2026
5 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

Merge Successful

Thanks for your contribution! 🎉

The changes will be part of the upcoming update on the Marketplace.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

icons PR with new icons

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants