-
-
Notifications
You must be signed in to change notification settings - Fork 16
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
added MaxMSP and Orca icons #77
Conversation
@mgxy0 why did you closed it ? |
@LelouchFR Sorry, wrong pull request |
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.
replace the maxmsp icons
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.
It would be interresting to first search a design for them.
orca could have a dark and light (and auto) theme, that matches other icons colors, because it's a little bit out of scope
@mgxy0 on what are you doing your icons ? |
With a simple macos app from app Store... I thought it was enough to upload the svg file. I apologize for the error, what do you recommend I use? |
I recommend you read the contribution guideline (use figma) |
@mgxy0 if you like, you can check the old merged PRs, its can also help you. |
@mgxy0 can you send me the figma file, I'm going to give you instructions on it, will be easier |
@mgxy0 do you need help? |
@mgxy0 please, commit the icons |
I will close it since it's being stale for a very long time |
closes tandpfun/skill-icons#989, closes tandpfun/skill-icons#990