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

Android 13 Notification Icon invisible against the background #480

Closed
s-h-a-r-d opened this issue Nov 9, 2022 · 10 comments
Closed

Android 13 Notification Icon invisible against the background #480

s-h-a-r-d opened this issue Nov 9, 2022 · 10 comments
Labels
android-app ntfy Android app 🪲 bug Something isn't working in-progress 🏃 I'm working on this right now

Comments

@s-h-a-r-d
Copy link
Contributor

tempFileForShare_20221109-121817

Device: Samsung Galaxy S22 Plus
OS: Android 13

My device updated to Android 13 and now the background color of the notification icon is same as the icon color itself. Which makes the icon almost invisible.

I guess the background color is configured somewhere and could be changed to a different color. BUT I think changing the notification icon to be fully white could be a better solution.

@clmbmb
Copy link

clmbmb commented Nov 14, 2022

Related to the above:

Before upgrading to Android 13 (Samsung Galaxy S20+) the notification icon in the status bar was 'adaptable', white on dark background, black on light background.

At the moment the icon is green and stays green in both situations. I'm attaching screenshots.

Screenshot_20221114_090614_Settings

Screenshot_20221114_090555_Settings

@s-h-a-r-d
Copy link
Contributor Author

I can confirm the "always green" icon as well.

@binwiederhier
Copy link
Owner

I have not been able to reproduce either case in the emulator, but it occurs on my phone as well. Very odd.

@alexhorner
Copy link

I have not been able to reproduce either case in the emulator, but it occurs on my phone as well. Very odd.

I too have this issue on my S22. I think (from my limited android dev experience, so please dismiss me if you know better) that you can set various different icon types in some kind of manifest, and provide them as a bunch of resources, some of those resources intentionally being monochrome icons for situations such as these.

image

As you can see, FitnessSyncer also has this issue partially, but they don't appear to have a theme colour set. The other apps such as Automate have monochrome icons correctly configured so do not exhibit this issue.

@binwiederhier
Copy link
Owner

binwiederhier commented Nov 26, 2022

Should be fixed in binwiederhier/ntfy-android#46 - Please try out the .apk and let me know if it works for you, and maybe post a screenshot too.

@alexhorner @s-h-a-r-d @clmbmb

NOTE: Be sure to back up your config first (Settings -> Backup)

https://drive.google.com/file/d/12Q9mejiBP1EZ84OFjvHyyPPoAklkTBPd/view?usp=share_link

@binwiederhier binwiederhier added the in-progress 🏃 I'm working on this right now label Nov 26, 2022
@binwiederhier binwiederhier pinned this issue Nov 26, 2022
@alexhorner
Copy link

alexhorner commented Nov 26, 2022

@binwiederhier All tested, looking snazzy!

Screenshot_20221126_201534_Nova7.jpg

May I also make the recommendation that you configure the subscription service notification category to be minimised by default? As you can see in my screenshot, I have manually configured it to be minimised.

Thanks for your awesome work!

EDIT: And maybe make that notification also undismissable?

@binwiederhier
Copy link
Owner

May I also make the recommendation that you configure the subscription service notification category to be minimised by default?

I briefly checked and I don't think this can be toggled programatically.

EDIT: And maybe make that notification also undismissable?

You mean make it so that it can be swiped away? That's not possible for foreground service notifications. It's desired behavior in Android.

@binwiederhier
Copy link
Owner

Done. Will be in the next release.

@binwiederhier binwiederhier unpinned this issue Nov 27, 2022
@alexhorner
Copy link

May I also make the recommendation that you configure the subscription service notification category to be minimised by default?

I briefly checked and I don't think this can be toggled programatically.

Quasseldroid, Conversations and Automate as you can see in my screenshots have it minimised by default. I believe Quasseldroid and Conversations are open source so maybe that could be of help?

EDIT: And maybe make that notification also undismissable?

You mean make it so that it can be swiped away? That's not possible for foreground service notifications. It's desired behavior in Android.

The opposite. Both Quasseldroid and Automate are undismissable (cannot be swiped away). Conversations and ntfy both are dismissable (can be swiped away) right now, and don't seem to reappear without killing them and starting them again.

I will open a separate issue for this.

@binwiederhier
Copy link
Owner

Interesting. Yeah please feel free to create two new issues for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
android-app ntfy Android app 🪲 bug Something isn't working in-progress 🏃 I'm working on this right now
Projects
None yet
Development

No branches or pull requests

4 participants