Skip to content
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

Add support for monochrome icon (Android 13+ only) #446

Merged
merged 2 commits into from Jul 22, 2023

Conversation

milindgoel15
Copy link
Contributor

@milindgoel15 milindgoel15 commented Jul 19, 2023

closes #396

I was not able to test on a real device as Android Studio won't let me build the project even the debug build due to some ffmpeg error
image

Let me know if the icon needs adjustments or needs to reduce/increase size.

@anilbeesetti
Copy link
Owner

anilbeesetti commented Jul 20, 2023

@milindgoel15, Try rebasing or merging main branch to pull latest changes. It will build now

@milindgoel15
Copy link
Contributor Author

milindgoel15 commented Jul 20, 2023

@milindgoel15, Try rebasing or merging main branch to pull latest changes. It will build now

Yep does now. This is how it will look. What do you think? First one is with stroke width 3 and second is 4. i think 4 is better.

stroke width 3 stroke width 4

@anilbeesetti
Copy link
Owner

Check this: I think it is better, don't you think
screenshot_1690000744

@milindgoel15
Copy link
Contributor Author

Yep filled is okay too.

@anilbeesetti anilbeesetti merged commit 2623288 into anilbeesetti:main Jul 22, 2023
3 checks passed
@milindgoel15
Copy link
Contributor Author

The viewport height and width aren't as per Google android docs. So do make sure ur icon fits perfectly in all devices.

Different viewports can mess up icons on custom skins like MIUI.

@anilbeesetti
Copy link
Owner

Thanks for the suggestion. So far i didn't encounter any issues with the icon. But i'll look into docs

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.

Add monochrome icon
2 participants