-
-
Notifications
You must be signed in to change notification settings - Fork 630
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add new icons for directive folder #2690
Conversation
PreviewThank you for creating a pull request. This preview shows you how your icons will look on the different themes: Check how your icons fit 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. |
This comment was marked as off-topic.
This comment was marked as off-topic.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@okineadev is right, the folder icon in the background is not the same one as we're using for all the other folder icons. You can compare the folder icons here: Correct one: Your one: Take a look at the shape of the folder icons. It looks different. Try to make it look similar as in the other folder icons. For any further questions take a look into the Contribution Guidelines and paste further questions here. |
It’s right, I prefer the first icon version too. |
Merge SuccessfulThanks for your contribution! 🎉 The changes will be part of the upcoming update on the Marketplace. |
Description
Added icons for directive folder.
Contribution Guidelines