From 0c3b78eca035c31e0e498547d417f99cdc5b1294 Mon Sep 17 00:00:00 2001
From: Karim Abou Zeid <karim23697@gmail.com>
Date: Tue, 27 Jan 2015 14:39:26 +0100
Subject: [PATCH] Design changes

- changed some style elements
- playing queue now much better looking
---
 .../materialmusic/adapter/PlayingQueueAdapter.java | 10 ++++++----
 app/src/main/res/layout/item_playlist.xml          | 14 ++++++--------
 app/src/main/res/values/styles.xml                 | 14 +++++++++++---
 3 files changed, 23 insertions(+), 15 deletions(-)

diff --git a/app/src/main/java/com/kabouzeid/materialmusic/adapter/PlayingQueueAdapter.java b/app/src/main/java/com/kabouzeid/materialmusic/adapter/PlayingQueueAdapter.java
index dfcf697a..fcb5d7f0 100644
--- a/app/src/main/java/com/kabouzeid/materialmusic/adapter/PlayingQueueAdapter.java
+++ b/app/src/main/java/com/kabouzeid/materialmusic/adapter/PlayingQueueAdapter.java
@@ -34,13 +34,15 @@ public class PlayingQueueAdapter extends ArrayAdapter<Song> {
             convertView = LayoutInflater.from(context).inflate(R.layout.item_playlist, parent, false);
         }
         final TextView title = (TextView) convertView.findViewById(R.id.song_title);
-        final ImageView albumArt = (ImageView) convertView.findViewById(R.id.album_art);
+        final ImageView playingIndicator = (ImageView) convertView.findViewById(R.id.playing_indicator);
 
         title.setText(song.title);
-        if (app.getMusicPlayerRemote().getCurrentSongId() == song.id) {
-            albumArt.setImageResource(R.drawable.ic_speaker_white_48dp);
+        if (app.getMusicPlayerRemote().getPosition() == position) {
+            playingIndicator.setVisibility(View.VISIBLE);
+            playingIndicator.setImageResource(R.drawable.ic_speaker_white_48dp);
         } else {
-            albumArt.setImageBitmap(null);
+            playingIndicator.setVisibility(View.GONE);
+            playingIndicator.setImageBitmap(null);
         }
         return convertView;
     }
diff --git a/app/src/main/res/layout/item_playlist.xml b/app/src/main/res/layout/item_playlist.xml
index 0424aa11..1fa269e3 100644
--- a/app/src/main/res/layout/item_playlist.xml
+++ b/app/src/main/res/layout/item_playlist.xml
@@ -7,14 +7,6 @@
     android:descendantFocusability="blocksDescendants"
     android:orientation="horizontal"
     >
-
-    <ImageView
-        android:layout_gravity="center"
-        android:id="@id/album_art"
-        android:layout_width="24dp"
-        android:layout_height="24dp"
-        android:layout_marginLeft="16dp"/>
-
     <TextView
         android:id="@+id/song_title"
         android:layout_width="0dp"
@@ -26,6 +18,12 @@
         android:singleLine="true"
         android:textAppearance="@style/TextAppearance.AppCompat.Body2"/>
 
+    <ImageView
+        android:id="@+id/playing_indicator"
+        style="@style/PlayingIndicator"
+        android:layout_marginRight="-8dp"
+        />
+
     <ImageView
         android:id="@+id/menu"
         style="@style/OverFlowButton"
diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml
index aaad2d34..9e93e57f 100644
--- a/app/src/main/res/values/styles.xml
+++ b/app/src/main/res/values/styles.xml
@@ -13,9 +13,9 @@
     <style name="Theme.MaterialMusic.Notification.Title" parent="@android:style/TextAppearance.StatusBar.EventContent.Title"/>
 
     <style name="OverFlowButton">
-        <item name="android:padding">20dp</item>
-        <item name="android:layout_width">64dp</item>
-        <item name="android:layout_height">64dp</item>
+        <item name="android:padding">12dp</item>
+        <item name="android:layout_width">48dp</item>
+        <item name="android:layout_height">48dp</item>
         <item name="android:src">@drawable/ic_overflow</item>
         <item name="android:scaleType">fitXY</item>
         <item name="android:background">?selectableItemBackground</item>
@@ -24,6 +24,14 @@
         <item name="android:focusable">false</item>
     </style>
 
+    <style name="PlayingIndicator">
+        <item name="android:layout_width">36dp</item>
+        <item name="android:layout_height">36dp</item>
+        <item name="android:padding">8dp</item>
+        <item name="android:layout_gravity">center</item>
+
+    </style>
+
     <style name="PlayPauseFab" parent="PlayPauseFabParent"/>
 
     <style name="NotificationButton" parent="NotificationButtonParent">