mirror of
https://github.com/binwiederhier/ntfy.git
synced 2026-05-09 08:26:00 +02:00
[GH-ISSUE #480] Android 13 Notification Icon invisible against the background #365
Labels
No labels
ai-generated
android-app
android-app
android-app
🪲 bug
build
build
dependencies
docs
enhancement
enhancement
🔥 HOT
in-progress 🏃
ios
prio:low
prio:low
pull-request
question
🔒 security
server
server
unified-push
web-app
website
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ntfy#365
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @s-h-a-r-d on GitHub (Nov 9, 2022).
Original GitHub issue: https://github.com/binwiederhier/ntfy/issues/480
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 commented on GitHub (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.
@s-h-a-r-d commented on GitHub (Nov 14, 2022):
I can confirm the "always green" icon as well.
@binwiederhier commented on GitHub (Nov 19, 2022):
I have not been able to reproduce either case in the emulator, but it occurs on my phone as well. Very odd.
@alexhorner commented on GitHub (Nov 24, 2022):
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.
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 commented on GitHub (Nov 26, 2022):
Should be fixed in https://github.com/binwiederhier/ntfy-android/pull/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
@alexhorner commented on GitHub (Nov 26, 2022):
@binwiederhier All tested, looking snazzy!
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 commented on GitHub (Nov 27, 2022):
I briefly checked and I don't think this can be toggled programatically.
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 commented on GitHub (Nov 27, 2022):
Done. Will be in the next release.
@alexhorner commented on GitHub (Nov 27, 2022):
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?
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 commented on GitHub (Nov 27, 2022):
Interesting. Yeah please feel free to create two new issues for this.