D8966: Add a foreground notification to the background service
Matthijs Tijink
noreply at phabricator.kde.org
Thu Nov 23 20:38:54 UTC 2017
mtijink added inline comments.
INLINE COMMENTS
> nicolasfella wrote in BackgroundService.java:294
> Why set a explicit color? I would go with the default and remove the line
This way the notification color ties in with the app color (orange), see https://developer.android.com/reference/android/app/Notification.html#color. More apps do this (and personally, I like it 😉).
If you disagree, I can change it.
REPOSITORY
R225 KDE Connect - Android application
REVISION DETAIL
https://phabricator.kde.org/D8966
To: mtijink, #kde_connect
Cc: nicolasfella
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20171123/25d6fd19/attachment.html>
More information about the KDEConnect
mailing list