-
Notifications
You must be signed in to change notification settings - Fork 119
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(generators): Added generation ic_launcher_round.png in assets icon #216
base: master
Are you sure you want to change the base?
Conversation
Just tested it, and it worked 👍 |
Thanks for the PR. I've tried it with one of my projects but it seems like some of the rounded icons have a white strip at some of their edges, despite the background being navy in the original square icon. Sometimes they have this at top, bottom, left, and right all together, and sometimes a combination of these. |
Thank you for your answer. I guess it's all about the margin of 90% I added to get the icon center in a smaller square. I use a mask and rewrite the icon on it with the compose "copyopacity". I didn't experience the bug you describe, I will try with different icon size, transparency and format. For the moment I still don't know how to prevent that except remove the margin. |
Hello dear. When do you plan to release this feature? Best regards |
console.log(`Wrote ${destinationPath + '.mask.png'}`); | ||
}); | ||
|
||
gm(path.normalize(psdSafeSourcePath)) |
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.
Composing of images depends on creating *.mask.png. So i think this gm chain for compose original png with *.mask.png should be places inside of .writeAsync
then(() => {
when writeAsync is asynchronous.
Fantastic job. I've tested it in my project and it works well. 👍 I've also added some comments to pull review commit's code - there is one potential problem with async writing of mask file and using it for composing opacity. I think this PR should be merged and |
Works great on react native 0.59.4 |
Maybe one more approving review? |
Works great, although it looks like Android 10 is pushing |
Oh I was not aware of that. I think you are right. Need to be done as squircle icons. |
I'm looking at this for the splash as well -> https://developer.android.com/preview/features/darktheme The default generated splash background color on a transparent PNG is Update: Manually setting |
now must be merging :( |
Could you please merge this PR |
Hi, thanks for submitting a Pull Request! 🎈
Before submitting, please be sure to check a few things:
Commit names formatting
This repository is automatically published to npm with
semantic-release.
In order to manage that, your commit should follow the AngularJS commit
conventions
as explained here
You can use
yarn commit
to write a commit respecting this convention.