Skip to content

feat: add folder icons for evaluation - #3617

Open
Farhan-Alam-Jamal wants to merge 2 commits into
material-extensions:mainfrom
Farhan-Alam-Jamal:feat/folder-evaluation-icon
Open

Farhan-Alam-Jamal wants to merge 2 commits into
material-extensions:mainfrom
Farhan-Alam-Jamal:feat/folder-evaluation-icon

Conversation

@Farhan-Alam-Jamal

Copy link
Copy Markdown

Add a dedicated folder icon for evaluation

Checks:

  • npm run check — all 4 checks passed (availability, usage, conflict, structure)
  • npm run check-colors — all Material Design palette colors valid
  • npm run build — bundle + open variants generated successfully
  • npm test — 158 tests passed, 0 failed

closes #3616

@github-actions github-actions Bot added the icons PR with new icons label Oct 8, 2026
@github-actions

github-actions Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

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.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for creating a PR, could you please read the https://github.com/material-extensions/vscode-material-icon-theme/blob/main/CONTRIBUTING.md#design-folder-icons and see how we typically place the motive in front of the folder icon? Thanks

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks for the review, @PKief! I'll read through the folder icon design guidelines, adjust the placement of the symbol relative to the folder, and push an update shortly

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.

[Icon Request]: evaluation folder

2 participants