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