From c44a3d01b921a500ea03c25bb14abc06db57ef17 Mon Sep 17 00:00:00 2001
From: Torsten Grote <t@grobox.de>
Date: Wed, 28 Feb 2018 12:53:22 -0300
Subject: [PATCH] Fix notification light
---
.../briar/android/AndroidNotificationManagerImpl.java | 9 +++++++--
.../briar/android/util/BriarNotificationBuilder.java | 5 +++++
2 files changed, 12 insertions(+), 2 deletions(-)
diff --git a/briar-android/src/main/java/org/briarproject/briar/android/AndroidNotificationManagerImpl.java b/briar-android/src/main/java/org/briarproject/briar/android/AndroidNotificationManagerImpl.java
index 5c66a8d9ca..698809cac7 100644
--- a/briar-android/src/main/java/org/briarproject/briar/android/AndroidNotificationManagerImpl.java
+++ b/briar-android/src/main/java/org/briarproject/briar/android/AndroidNotificationManagerImpl.java
@@ -11,6 +11,7 @@ import android.net.Uri;
import android.support.annotation.StringRes;
import android.support.annotation.UiThread;
import android.support.v4.app.TaskStackBuilder;
+import android.support.v4.content.ContextCompat;
import org.briarproject.bramble.api.Multiset;
import org.briarproject.bramble.api.contact.ContactId;
@@ -170,9 +171,13 @@ class AndroidNotificationManagerImpl implements AndroidNotificationManager,
@TargetApi(26)
private void createNotificationChannel(String channelId,
@StringRes int name) {
- notificationManager.createNotificationChannel(
+ NotificationChannel nc =
new NotificationChannel(channelId, appContext.getString(name),
- IMPORTANCE_DEFAULT));
+ IMPORTANCE_DEFAULT);
+ nc.enableLights(true);
+ nc.setLightColor(
+ ContextCompat.getColor(appContext, R.color.briar_green_light));
+ notificationManager.createNotificationChannel(nc);
}
@Override
diff --git a/briar-android/src/main/java/org/briarproject/briar/android/util/BriarNotificationBuilder.java b/briar-android/src/main/java/org/briarproject/briar/android/util/BriarNotificationBuilder.java
index 6e80004e4c..d51c0dad14 100644
--- a/briar-android/src/main/java/org/briarproject/briar/android/util/BriarNotificationBuilder.java
+++ b/briar-android/src/main/java/org/briarproject/briar/android/util/BriarNotificationBuilder.java
@@ -6,6 +6,8 @@ import android.support.annotation.ColorRes;
import android.support.v4.app.NotificationCompat;
import android.support.v4.content.ContextCompat;
+import org.briarproject.briar.R;
+
import static android.support.v4.app.NotificationCompat.VISIBILITY_PRIVATE;
import static android.support.v4.app.NotificationCompat.VISIBILITY_SECRET;
@@ -17,6 +19,9 @@ public class BriarNotificationBuilder extends NotificationCompat.Builder {
// Auto-cancel does not fire the delete intent, see
// https://issuetracker.google.com/issues/36961721
setAutoCancel(true);
+
+ setLights(ContextCompat.getColor(context, R.color.briar_green_light),
+ 750, 500);
}
public BriarNotificationBuilder setColorRes(@ColorRes int res) {
--
GitLab