Skip to content

added icon for context#1106

Merged
PKief merged 3 commits into
material-extensions:mainfrom
matemora:contexts-folder
Jul 12, 2021
Merged

added icon for context#1106
PKief merged 3 commits into
material-extensions:mainfrom
matemora:contexts-folder

Conversation

@matemora
Copy link
Copy Markdown
Contributor

Hello friend!
This is my first time ever doing a pull request. Feel free to give any tips on thing i can do better next time!

This PR was motivated by a common practice i have while programming react applications, that is creating a contexts folder. So i thought i would be nice to have a folder icon for that =)

The process i followed was based on the .md file for contributions. I downloaded the .svg icons for folder, folder-open and sitemap from material icons. Then i edited on GIMP, saved as png, and exported to .svg via the recommended links on the contributing .md.

Thank you in advance for considering this PR! =)

@PKief PKief self-requested a review June 23, 2021 16:59
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.

Hey, sorry for the late reply. I just looked at your PR for a context folder icon and I like the idea very much. I had to adjust the folder color a little bit so that the contrast in File Explorer is even better and you can see the icon well. Thank you very much for your contribution. The icon will be part of the next update :)

@PKief PKief merged commit a53cc5e into material-extensions:main Jul 12, 2021
@matemora
Copy link
Copy Markdown
Contributor Author

Thank you very very much for the consideration! It feels really nice to have my contribution accepted =)

@alamenai
Copy link
Copy Markdown

THANK YOU

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.

3 participants