From 2ce9b38747390511505cbd5f39a1531c060ccd4b Mon Sep 17 00:00:00 2001
From: Karim Abou Zeid <karim23697@gmail.com>
Date: Thu, 28 Jan 2016 17:49:37 +0100
Subject: [PATCH] Improved the Palette APIs speed and readded colored footers
 to artist and song adapters.

---
 app/build.gradle                              |  2 --
 .../adapter/album/AlbumAdapter.java           | 21 ++++++------
 .../adapter/artist/ArtistAdapter.java         | 17 +++++-----
 .../gramophone/adapter/song/SongAdapter.java  | 17 +++++-----
 .../palette/CustomTextColorGenerator.java     | 33 -------------------
 .../gramophone/util/PhonographColorUtil.java  |  2 +-
 6 files changed, 30 insertions(+), 62 deletions(-)
 delete mode 100644 app/src/main/java/com/kabouzeid/gramophone/glide/palette/CustomTextColorGenerator.java

diff --git a/app/build.gradle b/app/build.gradle
index b87f1f78..206540c9 100644
--- a/app/build.gradle
+++ b/app/build.gradle
@@ -139,6 +139,4 @@ dependencies {
     compile 'com.github.bumptech.glide:glide:3.6.1'
     compile 'com.github.kabouzeid:RecyclerView-FastScroll:v1.2-kmod'
     compile 'com.github.kabouzeid:app-theme-helper:0.7.8kmod'
-
-    compile 'com.github.kabouzeid:GlidePalette:e34717bced@aar'
 }
diff --git a/app/src/main/java/com/kabouzeid/gramophone/adapter/album/AlbumAdapter.java b/app/src/main/java/com/kabouzeid/gramophone/adapter/album/AlbumAdapter.java
index 1498d54e..dc6933df 100644
--- a/app/src/main/java/com/kabouzeid/gramophone/adapter/album/AlbumAdapter.java
+++ b/app/src/main/java/com/kabouzeid/gramophone/adapter/album/AlbumAdapter.java
@@ -12,6 +12,8 @@ import android.view.View;
 import android.view.ViewGroup;
 
 import com.bumptech.glide.Glide;
+import com.kabouzeid.appthemehelper.util.ColorUtil;
+import com.kabouzeid.appthemehelper.util.MaterialValueHelper;
 import com.kabouzeid.gramophone.R;
 import com.kabouzeid.gramophone.adapter.base.AbsMultiSelectAdapter;
 import com.kabouzeid.gramophone.adapter.base.MediaEntryViewHolder;
@@ -104,16 +106,15 @@ public class AlbumAdapter extends AbsMultiSelectAdapter<AlbumAdapter.ViewHolder,
     }
 
     private void setColors(int color, ViewHolder holder) {
-        // TODO
-//        if (holder.paletteColorContainer != null) {
-//            holder.paletteColorContainer.setBackgroundColor(color);
-//            if (holder.title != null) {
-//                holder.title.setTextColor(PhonographColorUtil.getPrimaryTextColorForBackground(activity, color));
-//            }
-//            if (holder.text != null) {
-//                holder.text.setTextColor(PhonographColorUtil.getSecondaryTextColorForBackground(activity, color));
-//            }
-//        }
+        if (holder.paletteColorContainer != null) {
+            holder.paletteColorContainer.setBackgroundColor(color);
+            if (holder.title != null) {
+                holder.title.setTextColor(MaterialValueHelper.getPrimaryTextColor(activity, ColorUtil.isColorLight(color)));
+            }
+            if (holder.text != null) {
+                holder.text.setTextColor(MaterialValueHelper.getSecondaryTextColor(activity, ColorUtil.isColorLight(color)));
+            }
+        }
     }
 
     protected void loadAlbumCover(Album album, final ViewHolder holder) {
diff --git a/app/src/main/java/com/kabouzeid/gramophone/adapter/artist/ArtistAdapter.java b/app/src/main/java/com/kabouzeid/gramophone/adapter/artist/ArtistAdapter.java
index 99bcf176..15d80b17 100644
--- a/app/src/main/java/com/kabouzeid/gramophone/adapter/artist/ArtistAdapter.java
+++ b/app/src/main/java/com/kabouzeid/gramophone/adapter/artist/ArtistAdapter.java
@@ -15,6 +15,8 @@ import com.bumptech.glide.Glide;
 import com.bumptech.glide.Priority;
 import com.bumptech.glide.load.engine.DiskCacheStrategy;
 import com.bumptech.glide.request.target.Target;
+import com.kabouzeid.appthemehelper.util.ColorUtil;
+import com.kabouzeid.appthemehelper.util.MaterialValueHelper;
 import com.kabouzeid.gramophone.R;
 import com.kabouzeid.gramophone.adapter.base.AbsMultiSelectAdapter;
 import com.kabouzeid.gramophone.adapter.base.MediaEntryViewHolder;
@@ -109,14 +111,13 @@ public class ArtistAdapter extends AbsMultiSelectAdapter<ArtistAdapter.ViewHolde
 
     private void setColors(int color, ViewHolder holder) {
         if (holder.paletteColorContainer != null) {
-            // TODO
-//            holder.paletteColorContainer.setBackgroundColor(color);
-//            if (holder.title != null) {
-//                holder.title.setTextColor(PhonographColorUtil.getPrimaryTextColorForBackground(activity, color));
-//            }
-//            if (holder.text != null) {
-//                holder.text.setTextColor(PhonographColorUtil.getSecondaryTextColorForBackground(activity, color));
-//            }
+            holder.paletteColorContainer.setBackgroundColor(color);
+            if (holder.title != null) {
+                holder.title.setTextColor(MaterialValueHelper.getPrimaryTextColor(activity, ColorUtil.isColorLight(color)));
+            }
+            if (holder.text != null) {
+                holder.text.setTextColor(MaterialValueHelper.getSecondaryTextColor(activity, ColorUtil.isColorLight(color)));
+            }
         }
     }
 
diff --git a/app/src/main/java/com/kabouzeid/gramophone/adapter/song/SongAdapter.java b/app/src/main/java/com/kabouzeid/gramophone/adapter/song/SongAdapter.java
index 06950ae1..abfbcb0b 100644
--- a/app/src/main/java/com/kabouzeid/gramophone/adapter/song/SongAdapter.java
+++ b/app/src/main/java/com/kabouzeid/gramophone/adapter/song/SongAdapter.java
@@ -13,6 +13,8 @@ import android.view.ViewGroup;
 
 import com.afollestad.materialcab.MaterialCab;
 import com.bumptech.glide.Glide;
+import com.kabouzeid.appthemehelper.util.ColorUtil;
+import com.kabouzeid.appthemehelper.util.MaterialValueHelper;
 import com.kabouzeid.gramophone.R;
 import com.kabouzeid.gramophone.adapter.base.AbsMultiSelectAdapter;
 import com.kabouzeid.gramophone.adapter.base.MediaEntryViewHolder;
@@ -106,14 +108,13 @@ public class SongAdapter extends AbsMultiSelectAdapter<SongAdapter.ViewHolder, S
 
     private void setColors(int color, ViewHolder holder) {
         if (holder.paletteColorContainer != null) {
-            // TODO
-//            holder.paletteColorContainer.setBackgroundColor(color);
-//            if (holder.title != null) {
-//                holder.title.setTextColor(PhonographColorUtil.getPrimaryTextColorForBackground(activity, color));
-//            }
-//            if (holder.text != null) {
-//                holder.text.setTextColor(PhonographColorUtil.getSecondaryTextColorForBackground(activity, color));
-//            }
+            holder.paletteColorContainer.setBackgroundColor(color);
+            if (holder.title != null) {
+                holder.title.setTextColor(MaterialValueHelper.getPrimaryTextColor(activity, ColorUtil.isColorLight(color)));
+            }
+            if (holder.text != null) {
+                holder.text.setTextColor(MaterialValueHelper.getSecondaryTextColor(activity, ColorUtil.isColorLight(color)));
+            }
         }
     }
 
diff --git a/app/src/main/java/com/kabouzeid/gramophone/glide/palette/CustomTextColorGenerator.java b/app/src/main/java/com/kabouzeid/gramophone/glide/palette/CustomTextColorGenerator.java
deleted file mode 100644
index a6268a7e..00000000
--- a/app/src/main/java/com/kabouzeid/gramophone/glide/palette/CustomTextColorGenerator.java
+++ /dev/null
@@ -1,33 +0,0 @@
-package com.kabouzeid.gramophone.glide.palette;
-
-import android.support.annotation.ColorInt;
-import android.support.v7.graphics.Palette;
-
-import com.github.florent37.glidepalette.common.BaseColorGenerator;
-
-/**
- * @author Karim Abou Zeid (kabouzeid)
- */
-public class CustomTextColorGenerator extends BaseColorGenerator {
-    private final int titleDark;
-    private final int titleLight;
-    private final int bodyDark;
-    private final int bodyLight;
-
-    public CustomTextColorGenerator(@ColorInt int titleDark, @ColorInt int titleLight, @ColorInt int bodyDark, @ColorInt int bodyLight) {
-        this.titleDark = titleDark;
-        this.titleLight = titleLight;
-        this.bodyDark = bodyDark;
-        this.bodyLight = bodyLight;
-    }
-
-    @Override
-    public int getTitleTextColor(Palette.Swatch swatch) {
-        return super.getTitleTextColor(swatch);
-    }
-
-    @Override
-    public int getBodyTextColor(Palette.Swatch swatch) {
-        return super.getBodyTextColor(swatch);
-    }
-}
diff --git a/app/src/main/java/com/kabouzeid/gramophone/util/PhonographColorUtil.java b/app/src/main/java/com/kabouzeid/gramophone/util/PhonographColorUtil.java
index ad54c20e..7c6ca979 100644
--- a/app/src/main/java/com/kabouzeid/gramophone/util/PhonographColorUtil.java
+++ b/app/src/main/java/com/kabouzeid/gramophone/util/PhonographColorUtil.java
@@ -14,7 +14,7 @@ import java.util.Comparator;
  * @author Karim Abou Zeid (kabouzeid)
  */
 public class PhonographColorUtil {
-    public static final int PALETTE_BITMAP_SIZE = 100;
+    public static final int PALETTE_BITMAP_SIZE = 10;
 
     @Nullable
     public static Palette generatePalette(Bitmap bitmap) {