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 monochrome icon variant for android 13 #170

Merged
merged 2 commits into from
Aug 30, 2023
Merged

Conversation

rec0de
Copy link
Contributor

@rec0de rec0de commented Aug 10, 2023

Hi!

I recently upgraded to android 13 and noticed that your app does not yet support monochrome app icons for theming. This makes the app stand out quite a bit when placed next to other apps that do support theming, so I added a simple monochrome version of the app icon.

[privacy]

Here's a before and after showing the themed and un-themed icon next to some other themed apps:
unthemed
themed

@udenr
Copy link
Collaborator

udenr commented Aug 28, 2023

On my Pixel 6 Pro (emulator, API 33) it looks like this:
grafik
Which device did you use?

@rec0de
Copy link
Contributor Author

rec0de commented Aug 28, 2023

Ah, I see that you added an inset for the non-monochrome version in the app icon xml - I've added that to the monochrome icon as well. Didn't catch that on my phone as the theme seems to provide its own inset for all icons.

I've also slightly bumped the inset on the regular icon so it displays better on round icons.

@udenr
Copy link
Collaborator

udenr commented Aug 30, 2023

Looks good to me now!
grafik

@udenr udenr merged commit 72f23a8 into SecUSo:master Aug 30, 2023
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.

2 participants