From 2da411c1ecd3995c5f2afb9a361cd1efcd5aab44 Mon Sep 17 00:00:00 2001
From: John Zhen Mo <zhenmogukl@gmail.com>
Date: Wed, 15 Nov 2017 20:42:05 -0800
Subject: [PATCH] -#836: Modified notifications texts to use custom colors
 instead of device-specific styles.

---
 app/src/main/res/layout/player_notification.xml          | 4 ++--
 app/src/main/res/layout/player_notification_expanded.xml | 6 +++---
 app/src/main/res/layout/player_popup_notification.xml    | 4 ++--
 app/src/main/res/values/colors.xml                       | 3 +++
 4 files changed, 10 insertions(+), 7 deletions(-)

diff --git a/app/src/main/res/layout/player_notification.xml b/app/src/main/res/layout/player_notification.xml
index 157615bb7..2a3e7aff3 100644
--- a/app/src/main/res/layout/player_notification.xml
+++ b/app/src/main/res/layout/player_notification.xml
@@ -34,22 +34,22 @@
 
             <TextView
                 android:id="@+id/notificationSongName"
-                style="@android:style/TextAppearance.StatusBar.EventContent.Title"
                 android:layout_width="match_parent"
                 android:ellipsize="end"
                 android:layout_height="wrap_content"
                 android:maxLines="1"
                 android:textSize="14sp"
+                android:textColor="@color/background_title_color"
                 tools:text="Lorem ipsum dolor sit amet, consectetur adipiscing elit. Duis nec aliquam augue, eget cursus est. Ut id tristique enim, ut scelerisque tellus. Sed ultricies ipsum non mauris ultricies, commodo malesuada velit porta."/>
 
             <TextView
                 android:id="@+id/notificationArtist"
                 android:layout_width="match_parent"
-                style="@android:style/TextAppearance.StatusBar.EventContent"
                 android:ellipsize="end"
                 android:layout_height="wrap_content"
                 android:maxLines="1"
                 android:textSize="12sp"
+                android:textColor="@color/background_subtext_color"
                 tools:text="Duis posuere arcu condimentum lobortis mattis."/>
         </LinearLayout>
 
diff --git a/app/src/main/res/layout/player_notification_expanded.xml b/app/src/main/res/layout/player_notification_expanded.xml
index d37087312..7d59720e0 100644
--- a/app/src/main/res/layout/player_notification_expanded.xml
+++ b/app/src/main/res/layout/player_notification_expanded.xml
@@ -46,22 +46,22 @@
 
         <TextView
             android:id="@+id/notificationSongName"
-            style="@android:style/TextAppearance.StatusBar.EventContent.Title"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:ellipsize="end"
             android:maxLines="2"
             android:textSize="14sp"
+            android:textColor="@color/background_title_color"
             tools:text="Lorem ipsum dolor sit amet, consectetur adipiscing elit. Duis nec aliquam augue, eget cursus est. Ut id tristique enim, ut scelerisque tellus. Sed ultricies ipsum non mauris ultricies, commodo malesuada velit porta."/>
 
         <TextView
             android:id="@+id/notificationArtist"
-            style="@android:style/TextAppearance.StatusBar.EventContent"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:ellipsize="end"
             android:maxLines="1"
             android:textSize="12sp"
+            android:textColor="@color/background_subtext_color"
             tools:text="Duis posuere arcu condimentum lobortis mattis."/>
     </LinearLayout>
 
@@ -80,7 +80,6 @@
 
     <TextView
         android:id="@+id/notificationTime"
-        style="@android:style/TextAppearance.StatusBar.EventContent"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:layout_marginLeft="8dp"
@@ -92,6 +91,7 @@
         android:ellipsize="end"
         android:maxLines="1"
         android:textSize="12sp"
+        android:textColor="@color/background_subtext_color"
         tools:text="Duis posuere"/>
 
     <RelativeLayout
diff --git a/app/src/main/res/layout/player_popup_notification.xml b/app/src/main/res/layout/player_popup_notification.xml
index ab436a1fa..c8a06db47 100644
--- a/app/src/main/res/layout/player_popup_notification.xml
+++ b/app/src/main/res/layout/player_popup_notification.xml
@@ -29,22 +29,22 @@
 
         <TextView
             android:id="@+id/notificationSongName"
-            style="@android:style/TextAppearance.StatusBar.EventContent.Title"
             android:layout_width="match_parent"
             android:ellipsize="end"
             android:layout_height="wrap_content"
             android:maxLines="1"
             android:textSize="14sp"
+            android:textColor="@color/background_title_color"
             tools:text="Lorem ipsum dolor sit amet, consectetur adipiscing elit. Duis nec aliquam augue, eget cursus est. Ut id tristique enim, ut scelerisque tellus. Sed ultricies ipsum non mauris ultricies, commodo malesuada velit porta."/>
 
         <TextView
             android:id="@+id/notificationArtist"
             android:layout_width="match_parent"
-            style="@android:style/TextAppearance.StatusBar.EventContent"
             android:ellipsize="end"
             android:layout_height="wrap_content"
             android:maxLines="1"
             android:textSize="12sp"
+            android:textColor="@color/background_subtext_color"
             tools:text="Duis posuere arcu condimentum lobortis mattis."/>
 
     </LinearLayout>
diff --git a/app/src/main/res/values/colors.xml b/app/src/main/res/values/colors.xml
index 3750bdb78..b77a2d229 100644
--- a/app/src/main/res/values/colors.xml
+++ b/app/src/main/res/values/colors.xml
@@ -39,7 +39,10 @@
     <color name="duration_text_color">#EEFFFFFF</color>
     <color name="playlist_stream_count_text_color">#ffffff</color>
     <color name="video_overlay_color">#66000000</color>
+
     <color name="background_notification_color">#323232</color>
+    <color name="background_title_color">#ffffff</color>
+    <color name="background_subtext_color">#999999</color>
 
     <color name="subscribe_background_color">#e53935</color>
     <color name="subscribe_text_color">#fff</color>