Skip to content

feat(icon): nginx folder icon#3197

Open
lorypelli wants to merge 1 commit intomaterial-extensions:mainfrom
lorypelli:patch-2
Open

feat(icon): nginx folder icon#3197
lorypelli wants to merge 1 commit intomaterial-extensions:mainfrom
lorypelli:patch-2

Conversation

@lorypelli
Copy link
Contributor

Description

Added icon if the folder is named nginx (like php does have)

Contribution Guidelines

@github-actions github-actions bot added the icons PR with new icons label Oct 1, 2025
@lucas-labs
Copy link
Member

I think you've missed the svg files for the icon in your commit.

@lorypelli
Copy link
Contributor Author

I think you've missed the svg files for the icon in your commit.

Isn't it the same as the other one for file icon?

@PKief
Copy link
Member

PKief commented Oct 2, 2025

@lorypelli you have to create new SVG files for the folders. We don't pick up the file icons automatically. This PR requires:

  • folder-nginx.svg
  • folder-nginx-open.svg

You can find more details in our contribution guidelines: https://github.com/material-extensions/vscode-material-icon-theme/blob/main/CONTRIBUTING.md#create-icon-as-svg

Feel free to ask if you need further support!

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.

3 participants