-
Notifications
You must be signed in to change notification settings - Fork 290
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
Close button (X) not visible in notification when using Android 10 Dark theme #1002
Comments
Turns out we used transparency in the 'X' image. I changed the image to not use any alpha channels and add a white border around it in commit 497af31 - could you check the latest nightly build to see if things are better? |
Sorry for not responding sooner! I actually can't test the nightly build: when I download the apk it just gives me the word "error" when I try to install it. |
Interesting: maybe there was something wrong with the build: could you try again? (the APK should get refreshed every 24 hours) |
@adrian-bl Then I can test it |
[x]
):Description
When Android 10 is set to use the built-in Dark theme, notifications are displayed in black. While Vanilla Music's back/play/forward controls are still visible, the X button used to close the notification is not. It is still there, and it does work if you touch it, but since it is black it is invisible.
There has been an official dark theme since Android 9, but in Android 10 the notifications were changed to be black. I saw that there was some talk about dark theme notifications, but only with regards to custom themes. This is a report to get the notifications looking correct with the default stock Android dark theme.
Steps to reproduce the issue
Using stock Android 10, turn on Dark theme (Settings -> Display -> Dark theme).
Play something with Vanilla Music.
Gaze upon the notification.
Screenshots
Notification with Dark theme enabled:
Non-dark theme notification for comparison:
The text was updated successfully, but these errors were encountered: