Merge pull request #20 from burnermanx/master

Making an option to not use large icon in PugNotification
halysongoncalves · Jul 14, 2015 · 6f3f2f8 · 6f3f2f8
2 parents 31f5f91 + ebf7b45
commit 6f3f2f8
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/pugnotification/src/main/java/br/com/goncalves/pugnotification/notification/Load.java b/pugnotification/src/main/java/br/com/goncalves/pugnotification/notification/Load.java
@@ -42,8 +42,13 @@ private void createNotifationDefault() {
         this.builder.setContentTitle("");
         this.builder.setContentText("");
         this.builder.setSmallIcon(R.drawable.pugnotification_ic_launcher);
-        this.builder.setLargeIcon(BitmapFactory.decodeResource(mNotification.mContext.getResources(),
-                R.drawable.pugnotification_ic_launcher));
+
+        // FIX: Not setting a default LargeIcon, to make an option to use or not use a large icon in notification. Otherwise I'll get an Android icon plus smallIcon in notification.
+        // I tested it in Sample App and it worked. Just comment .largeIcon() line in PugNotification builder to test.
+
+        //this.builder.setLargeIcon(BitmapFactory.decodeResource(mNotification.mContext.getResources(),
+        //        R.drawable.pugnotification_ic_launcher));
+
         this.builder.setContentIntent(PendingIntent.getBroadcast(mNotification.mContext, 0, new Intent(), PendingIntent.FLAG_UPDATE_CURRENT));
     }