From 32ce1f71202ca525069b0f8291278e59bd022b22 Mon Sep 17 00:00:00 2001
From: Christopher Eby <kreed@kreed.org>
Date: Thu, 18 Mar 2010 20:05:29 -0500
Subject: [PATCH] Update widget play/pause icon according to state

---
 .../{hidden_play_pause.xml => hidden_pause.xml} |  0
 res/drawable/hidden_play.xml                    | 10 ++++++++++
 res/layout/one_cell_widget.xml                  | 11 +++++++++--
 src/org/kreed/vanilla/OneCellWidget.java        | 17 ++++++++++++++++-
 4 files changed, 35 insertions(+), 3 deletions(-)
 rename res/drawable/{hidden_play_pause.xml => hidden_pause.xml} (100%)
 create mode 100644 res/drawable/hidden_play.xml

diff --git a/res/drawable/hidden_play_pause.xml b/res/drawable/hidden_pause.xml
similarity index 100%
rename from res/drawable/hidden_play_pause.xml
rename to res/drawable/hidden_pause.xml
diff --git a/res/drawable/hidden_play.xml b/res/drawable/hidden_play.xml
new file mode 100644
index 00000000..c013bf28
--- /dev/null
+++ b/res/drawable/hidden_play.xml
@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="utf-8"?>
+<selector xmlns:android="http://schemas.android.com/apk/res/android">
+	<item
+		android:state_pressed="true"
+		android:drawable="@drawable/play" />
+	<item
+		android:state_focused="true"
+		android:drawable="@drawable/play" />
+	<item android:drawable="@drawable/empty" />
+</selector>
\ No newline at end of file
diff --git a/res/layout/one_cell_widget.xml b/res/layout/one_cell_widget.xml
index 2a6f3eb6..23d582f3 100644
--- a/res/layout/one_cell_widget.xml
+++ b/res/layout/one_cell_widget.xml
@@ -12,11 +12,18 @@
 		android:layout_width="fill_parent"
 		android:orientation="vertical" >
 		<ImageButton
-			android:id="@+id/play_pause"
-			android:background="@drawable/hidden_play_pause"
+			android:id="@+id/play"
+			android:background="@drawable/hidden_play"
 			android:layout_height="wrap_content"
 			android:layout_width="fill_parent"
 			android:layout_weight="1" />
+		<ImageButton
+			android:id="@+id/pause"
+			android:background="@drawable/hidden_pause"
+			android:layout_height="wrap_content"
+			android:layout_width="fill_parent"
+			android:layout_weight="1"
+			android:visibility="gone" />
 		<ImageButton
 			android:id="@+id/next"
 			android:background="@drawable/hidden_next"
diff --git a/src/org/kreed/vanilla/OneCellWidget.java b/src/org/kreed/vanilla/OneCellWidget.java
index b267948b..6f61923f 100644
--- a/src/org/kreed/vanilla/OneCellWidget.java
+++ b/src/org/kreed/vanilla/OneCellWidget.java
@@ -25,6 +25,7 @@ import android.content.ComponentName;
 import android.content.Context;
 import android.content.Intent;
 import android.util.TypedValue;
+import android.view.View;
 import android.widget.RemoteViews;
 
 public class OneCellWidget extends AppWidgetProvider {
@@ -63,7 +64,21 @@ public class OneCellWidget extends AppWidgetProvider {
 	{
 		RemoteViews views = new RemoteViews(context.getPackageName(), R.layout.one_cell_widget);
 
-		views.setOnClickPendingIntent(R.id.play_pause, PendingIntent.getBroadcast(context, 0, new Intent(PlaybackService.TOGGLE_PLAYBACK), 0));
+		int toggle;
+
+		// Unfortunately we have to have two views since we can not call
+		// setBackgroundResource from RemoteViews
+		if (playing) {
+			toggle = R.id.pause;
+			views.setViewVisibility(R.id.play, View.GONE);
+			views.setViewVisibility(R.id.pause, View.VISIBLE);
+		} else {
+			toggle = R.id.play;
+			views.setViewVisibility(R.id.pause, View.GONE);
+			views.setViewVisibility(R.id.play, View.VISIBLE);
+		}
+
+		views.setOnClickPendingIntent(toggle, PendingIntent.getBroadcast(context, 0, new Intent(PlaybackService.TOGGLE_PLAYBACK), 0));
 		views.setOnClickPendingIntent(R.id.next, PendingIntent.getBroadcast(context, 0, new Intent(PlaybackService.NEXT_SONG), 0));
 
 		if (song == null) {