From e3d009e743131e32dc5a4dbe42134359acb45c98 Mon Sep 17 00:00:00 2001
From: Kosh <kosh20111@gmail.com>
Date: Sat, 17 Jun 2017 22:58:23 +0800
Subject: [PATCH 1/4] added cardview custom background color & provide an
 example

---
 .../ExampleMaterialAboutActivity.java         |  5 ++++
 app/src/main/res/layout/content_main.xml      |  9 +++++-
 app/src/main/res/values/styles.xml            | 16 ++++++++--
 .../layout/mal_material_about_list_card.xml   | 30 +++++++++----------
 library/src/main/res/values/attrs.xml         |  1 +
 library/src/main/res/values/styles.xml        |  5 ++++
 6 files changed, 48 insertions(+), 18 deletions(-)

diff --git a/app/src/main/java/com/danielstone/materialaboutlibrarydemo/ExampleMaterialAboutActivity.java b/app/src/main/java/com/danielstone/materialaboutlibrarydemo/ExampleMaterialAboutActivity.java
index 864f633..9835c14 100644
--- a/app/src/main/java/com/danielstone/materialaboutlibrarydemo/ExampleMaterialAboutActivity.java
+++ b/app/src/main/java/com/danielstone/materialaboutlibrarydemo/ExampleMaterialAboutActivity.java
@@ -30,6 +30,7 @@ public class ExampleMaterialAboutActivity extends MaterialAboutActivity {
     public static final int THEME_LIGHT_DARKBAR = 1;
     public static final int THEME_DARK_LIGHTBAR = 2;
     public static final int THEME_DARK_DARKBAR = 3;
+    public static final int THEME_DARK_CUSTOM_CARDVIEW = 4;
 
     protected int colorIcon = R.color.mal_color_icon_light_theme;
 
@@ -120,6 +121,10 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
                 setTheme(R.style.AppTheme_MaterialAboutActivity_Dark_DarkActionBar);
                 colorIcon = R.color.mal_color_icon_dark_theme;
                 break;
+            case THEME_DARK_CUSTOM_CARDVIEW:
+                setTheme(R.style.AppTheme_MaterialAboutActivity_Light_CustomCardView);
+                colorIcon = R.color.mal_color_icon_dark_theme;
+                break;
         }
 
         super.onCreate(savedInstanceState);
diff --git a/app/src/main/res/layout/content_main.xml b/app/src/main/res/layout/content_main.xml
index 98a39e6..f076933 100644
--- a/app/src/main/res/layout/content_main.xml
+++ b/app/src/main/res/layout/content_main.xml
@@ -55,5 +55,12 @@
         android:tag="3"
         android:text="Launch Activity - Dark with Dark Action Bar" />
 
-
+    <Button
+        style="@style/Widget.AppCompat.Button.Colored"
+        android:layout_width="wrap_content"
+        android:layout_height="wrap_content"
+        android:layout_marginTop="@dimen/mal_baseline"
+        android:onClick="onActivityButtonPressed"
+        android:tag="4"
+        android:text="Launch Activity - Custom CardView Background" />
 </LinearLayout>
diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index 5c5bb91..fe0187f 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -20,6 +20,7 @@
         <item name="colorAccent">@color/colorAccent</item>
     </style>
 
+
     <style name="AppTheme.MaterialAboutActivity.Fragment" parent="Theme.Mal.Dark.DarkActionBar">
         <!-- Customize your theme here. -->
         <item name="colorPrimary">@color/colorPrimary</item>
@@ -41,6 +42,17 @@
         <item name="colorAccent">@color/colorAccent</item>
     </style>
 
+    <style name="AppTheme.MaterialAboutActivity.Light.CustomCardView" parent="Theme.Mal.Light.DarkActionBar">
+        <!-- Customize your theme here. -->
+        <item name="colorPrimary">@color/colorPrimary</item>
+        <item name="colorPrimaryDark">@color/colorPrimaryDark</item>
+        <item name="colorAccent">@color/colorAccent</item>
+        <item name="mal_card_background">@color/colorPrimaryDark</item>
+        <item name="android:textColorPrimary">#eee</item>
+        <item name="mal_color_primary">#eee</item>
+        <item name="mal_color_secondary">#ffe0e0e0</item>
+    </style>
+
     <style name="AppTheme.MaterialAboutActivity.Light.LightActionBar" parent="Theme.Mal.Light.LightActionBar">
         <!-- Customize your theme here. -->
         <item name="colorPrimary">@color/colorLightBar</item>
@@ -48,8 +60,8 @@
         <item name="colorAccent">@color/colorAccent</item>
     </style>
 
-    <style name="AppTheme.AppBarOverlay" parent="ThemeOverlay.AppCompat.Dark.ActionBar" />
+    <style name="AppTheme.AppBarOverlay" parent="ThemeOverlay.AppCompat.Dark.ActionBar"/>
 
-    <style name="AppTheme.PopupOverlay" parent="ThemeOverlay.AppCompat.Light" />
+    <style name="AppTheme.PopupOverlay" parent="ThemeOverlay.AppCompat.Light"/>
 
 </resources>
diff --git a/library/src/main/res/layout/mal_material_about_list_card.xml b/library/src/main/res/layout/mal_material_about_list_card.xml
index 30e0166..f6515d6 100644
--- a/library/src/main/res/layout/mal_material_about_list_card.xml
+++ b/library/src/main/res/layout/mal_material_about_list_card.xml
@@ -1,25 +1,26 @@
 <?xml version="1.0" encoding="utf-8"?>
-<android.support.v7.widget.CardView xmlns:android="http://schemas.android.com/apk/res/android"
+<android.support.v7.widget.CardView
+    xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
-    xmlns:app="http://schemas.android.com/apk/res-auto"
+    android:layout_marginEnd="@dimen/mal_baseline_half"
     android:layout_marginLeft="@dimen/mal_baseline_half"
-    android:layout_marginStart="@dimen/mal_baseline_half"
     android:layout_marginRight="@dimen/mal_baseline_half"
-    android:layout_marginEnd="@dimen/mal_baseline_half"
+    android:layout_marginStart="@dimen/mal_baseline_half"
     android:layout_marginTop="@dimen/mal_baseline_half"
+    android:paddingBottom="@dimen/mal_baseline"
+    android:paddingEnd="@dimen/mal_baseline_half"
     android:paddingLeft="@dimen/mal_baseline_half"
-    android:paddingStart="@dimen/mal_baseline_half"
     android:paddingRight="@dimen/mal_baseline_half"
-    android:paddingEnd="@dimen/mal_baseline_half"
+    android:paddingStart="@dimen/mal_baseline_half"
     android:paddingTop="@dimen/mal_baseline"
-    android:paddingBottom="@dimen/mal_baseline"
+    app:cardBackgroundColor="?mal_card_background"
     app:cardCornerRadius="@dimen/mal_card_radius"
     app:cardElevation="@dimen/mal_card_elevation"
     app:cardPreventCornerOverlap="false"
-    app:cardUseCompatPadding="true"
-    >
+    app:cardUseCompatPadding="true">
 
     <LinearLayout
         android:layout_width="match_parent"
@@ -30,24 +31,23 @@
             android:id="@+id/mal_list_card_title"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
+            android:layout_marginBottom="@dimen/mal_baseline_half"
+            android:layout_marginEnd="@dimen/mal_baseline"
             android:layout_marginLeft="@dimen/mal_baseline"
-            android:layout_marginStart="@dimen/mal_baseline"
             android:layout_marginRight="@dimen/mal_baseline"
-            android:layout_marginEnd="@dimen/mal_baseline"
+            android:layout_marginStart="@dimen/mal_baseline"
             android:layout_marginTop="@dimen/mal_baseline"
-            android:layout_marginBottom="@dimen/mal_baseline_half"
             android:textAppearance="@style/TextAppearance.AppCompat.Body2"
             android:textColor="?mal_color_primary"
-            tools:text="Author" />
+            tools:text="Author"/>
 
         <android.support.v7.widget.RecyclerView
             android:id="@+id/mal_card_recyclerview"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:clipToPadding="false"
-            android:paddingTop="@dimen/mal_baseline_half"
             android:paddingBottom="@dimen/mal_baseline_half"
-            />
+            android:paddingTop="@dimen/mal_baseline_half"/>
 
     </LinearLayout>
 
diff --git a/library/src/main/res/values/attrs.xml b/library/src/main/res/values/attrs.xml
index 7dc82da..452f970 100644
--- a/library/src/main/res/values/attrs.xml
+++ b/library/src/main/res/values/attrs.xml
@@ -5,4 +5,5 @@
     
     <attr name="mal_color_primary" format="reference|color" />
     <attr name="mal_color_secondary" format="reference|color" />
+    <attr name="mal_card_background" format="reference|color" />
 </resources>
\ No newline at end of file
diff --git a/library/src/main/res/values/styles.xml b/library/src/main/res/values/styles.xml
index a2f3024..1a5fa1c 100644
--- a/library/src/main/res/values/styles.xml
+++ b/library/src/main/res/values/styles.xml
@@ -8,6 +8,7 @@
         <item name="mal_popupOverlay">@style/Theme.Mal.Light.PopupOverlay</item>
         <item name="mal_color_primary">@color/mal_text_primary</item>
         <item name="mal_color_secondary">@color/mal_text_secondary</item>
+        <item name="mal_card_background">@color/cardview_light_background</item>
     </style>
 
     <style name="Theme.Mal.Light.DarkActionBar" parent="Theme.AppCompat.Light.NoActionBar">
@@ -17,6 +18,7 @@
         <item name="mal_popupOverlay">@style/Theme.Mal.Light.PopupOverlay</item>
         <item name="mal_color_primary">@color/mal_text_primary</item>
         <item name="mal_color_secondary">@color/mal_text_secondary</item>
+        <item name="mal_card_background">@color/cardview_light_background</item>
     </style>
 
     <style name="Theme.Mal.Light.LightActionBar" parent="Theme.AppCompat.Light.NoActionBar">
@@ -26,6 +28,7 @@
         <item name="mal_popupOverlay">@style/Theme.Mal.Light.PopupOverlay</item>
         <item name="mal_color_primary">@color/mal_text_primary</item>
         <item name="mal_color_secondary">@color/mal_text_secondary</item>
+        <item name="mal_card_background">@color/cardview_light_background</item>
     </style>
 
     <style name="Theme.Mal.Dark.DarkActionBar" parent="Theme.AppCompat.NoActionBar">
@@ -35,6 +38,7 @@
         <item name="mal_popupOverlay">@style/Theme.Mal.Dark.PopupOverlay</item>
         <item name="mal_color_primary">@color/mal_text_primary_dark</item>
         <item name="mal_color_secondary">@color/mal_text_secondary_dark</item>
+        <item name="mal_card_background">@color/cardview_dark_background</item>
     </style>
 
     <style name="Theme.Mal.Dark.LightActionBar" parent="Theme.AppCompat.NoActionBar">
@@ -44,6 +48,7 @@
         <item name="mal_popupOverlay">@style/Theme.Mal.Dark.PopupOverlay</item>
         <item name="mal_color_primary">@color/mal_text_primary_dark</item>
         <item name="mal_color_secondary">@color/mal_text_secondary_dark</item>
+        <item name="mal_card_background">@color/cardview_dark_background</item>
     </style>
 
     <style name="Theme.Mal.Light.PopupOverlay" parent="ThemeOverlay.AppCompat.Light" />

From 4c7b6908ded5b40ce4a449416db0ab2153f69883 Mon Sep 17 00:00:00 2001
From: Kosh <kosh20111@gmail.com>
Date: Sat, 17 Jun 2017 23:03:00 +0800
Subject: [PATCH 2/4] renamed variable

---
 .../ExampleMaterialAboutActivity.java                         | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/app/src/main/java/com/danielstone/materialaboutlibrarydemo/ExampleMaterialAboutActivity.java b/app/src/main/java/com/danielstone/materialaboutlibrarydemo/ExampleMaterialAboutActivity.java
index 9835c14..2f4fd33 100644
--- a/app/src/main/java/com/danielstone/materialaboutlibrarydemo/ExampleMaterialAboutActivity.java
+++ b/app/src/main/java/com/danielstone/materialaboutlibrarydemo/ExampleMaterialAboutActivity.java
@@ -30,7 +30,7 @@ public class ExampleMaterialAboutActivity extends MaterialAboutActivity {
     public static final int THEME_LIGHT_DARKBAR = 1;
     public static final int THEME_DARK_LIGHTBAR = 2;
     public static final int THEME_DARK_DARKBAR = 3;
-    public static final int THEME_DARK_CUSTOM_CARDVIEW = 4;
+    public static final int THEME_CUSTOM_CARDVIEW = 4;
 
     protected int colorIcon = R.color.mal_color_icon_light_theme;
 
@@ -121,7 +121,7 @@ protected void onCreate(@Nullable Bundle savedInstanceState) {
                 setTheme(R.style.AppTheme_MaterialAboutActivity_Dark_DarkActionBar);
                 colorIcon = R.color.mal_color_icon_dark_theme;
                 break;
-            case THEME_DARK_CUSTOM_CARDVIEW:
+            case THEME_CUSTOM_CARDVIEW:
                 setTheme(R.style.AppTheme_MaterialAboutActivity_Light_CustomCardView);
                 colorIcon = R.color.mal_color_icon_dark_theme;
                 break;

From c0f539c842f5d97e6aa1e5969721646e1a7045ec Mon Sep 17 00:00:00 2001
From: Kosh <kosh20111@gmail.com>
Date: Sat, 17 Jun 2017 23:16:50 +0800
Subject: [PATCH 3/4] renamed variable & made activity context to be wrapped in
 weakReference to avoid leakage.

---
 .../materialaboutlibrary/MaterialAboutActivity.java  | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/library/src/main/java/com/danielstone/materialaboutlibrary/MaterialAboutActivity.java b/library/src/main/java/com/danielstone/materialaboutlibrary/MaterialAboutActivity.java
index e1b6072..0916af3 100644
--- a/library/src/main/java/com/danielstone/materialaboutlibrary/MaterialAboutActivity.java
+++ b/library/src/main/java/com/danielstone/materialaboutlibrary/MaterialAboutActivity.java
@@ -24,6 +24,8 @@
 import com.danielstone.materialaboutlibrary.util.DefaultViewTypeManager;
 import com.danielstone.materialaboutlibrary.util.ViewTypeManager;
 
+import java.lang.ref.WeakReference;
+
 public abstract class MaterialAboutActivity extends AppCompatActivity {
 
     private MaterialAboutList list = new MaterialAboutList.Builder().build();
@@ -173,22 +175,22 @@ protected void setScrollToolbar(boolean scrollToolbar) {
 
     private static class ListTask extends AsyncTask<String, String, MaterialAboutList> {
 
-        private MaterialAboutActivity context;
+        private WeakReference<MaterialAboutActivity> context;
 
         ListTask(MaterialAboutActivity context) {
-            this.context = context;
+            this.context = new WeakReference<>(context);
         }
 
         @Override
         protected MaterialAboutList doInBackground(String... params) {
-            return isCancelled() ? null : context.getMaterialAboutList(context);
+            return isCancelled() || context.get() == null ? null : context.get().getMaterialAboutList(context.get());
         }
 
         @Override
         protected void onPostExecute(MaterialAboutList materialAboutList) {
             super.onPostExecute(materialAboutList);
-            if (!context.isFinishing()) {
-                context.onTaskFinished(materialAboutList);
+            if (!context.get().isFinishing()) {
+                context.get().onTaskFinished(materialAboutList);
             }
             context = null;
         }

From e033ca559a13e3f79204cc2817ee765bb9cafb02 Mon Sep 17 00:00:00 2001
From: Kosh <kosh20111@gmail.com>
Date: Sat, 17 Jun 2017 23:22:21 +0800
Subject: [PATCH 4/4] added FastHub to readme

---
 README.md | 1 +
 1 file changed, 1 insertion(+)

diff --git a/README.md b/README.md
index 1cef923..68483fb 100644
--- a/README.md
+++ b/README.md
@@ -197,6 +197,7 @@ Check out a working example in [`Demo.java`][3].
 - [ComicsDB Client](https://play.google.com/store/apps/details?id=cz.kutner.comicsdbclient.comicsdbclient), code available on [GitHub](https://github.com/tukak/comicsdbclient)
 - [Android About Box](https://github.com/eggheadgames/android-about-box) (library) - an opinionated About Box for Android
 - [Skin Widget for Minecraft](https://play.google.com/store/apps/details?id=com.rabross.android.minecraftskinwidget)
+- [FastHub for GitHub](https://play.google.com/store/apps/details?id=com.fastaccess.github&hl=en)
 
 ## License