From e1b2b200bf71cfc9e33cfdd32d4340581c0b6b5d Mon Sep 17 00:00:00 2001
From: Gina Trapani <ginatrapani@gmail.com>
Date: Sat, 12 Feb 2011 10:09:59 -0800
Subject: [PATCH] Improved task list item presentation * Strikethrough
 completed items * Removed priority parentheses * Aligned item number
 horizontally * Set item number visibility to GONE when preference is
 unchecked * Closes #129

---
 res/layout/list_item.xml                       | 16 ++++++++--------
 src/com/todotxt/todotxttouch/TodoTxtTouch.java | 18 +++++++++++++++---
 2 files changed, 23 insertions(+), 11 deletions(-)

diff --git a/res/layout/list_item.xml b/res/layout/list_item.xml
index 76155fc4..fb60d147 100644
--- a/res/layout/list_item.xml
+++ b/res/layout/list_item.xml
@@ -23,24 +23,24 @@ You should have received a copy of the GNU General Public License along with Tod
 
 @author Gina Trapani <ginatrapani[at]gmail[dot]com>
 @author mathias <mathias[at]ws7862[dot](none)>
-@author mathias <mathias[at]x2[dot](none)>
 @license http://www.gnu.org/licenses/gpl.html
 @copyright 2009-2011 Gina Trapani, mathias
  -->
 
 <RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android">
 
-	<TextView android:id="@+id/taskprio" android:layout_width="wrap_content"
-		android:layout_height="wrap_content" android:padding="4dip"
-		android:textStyle="bold" android:textSize="16dip" />
-
 	<TextView android:id="@+id/taskid" android:layout_width="wrap_content"
+		android:layout_height="wrap_content" android:gravity="bottom"
+		android:textSize="10dip" android:layout_alignBaseline="@+id/taskprio"
+		android:textColor="@color/grey" android:paddingLeft="4dip" />
+
+	<TextView android:id="@+id/taskprio" android:layout_width="wrap_content"
 		android:layout_height="wrap_content" android:padding="4dip"
-		android:textSize="10dip" android:layout_below="@id/taskprio"
-		android:textColor="@color/black" />
+		android:textSize="16dip" android:textStyle="bold"
+		android:layout_toRightOf="@id/taskid" />
 
 	<TextView android:id="@+id/tasktext" android:layout_width="wrap_content"
 		android:layout_height="wrap_content" android:padding="4dip"
-		android:layout_toRightOf="@id/taskprio" android:textSize="16dip" />
+		android:textSize="16dip" android:layout_toRightOf="@id/taskprio" />
 
 </RelativeLayout>
diff --git a/src/com/todotxt/todotxttouch/TodoTxtTouch.java b/src/com/todotxt/todotxttouch/TodoTxtTouch.java
index e1e2bf7b..fa379407 100644
--- a/src/com/todotxt/todotxttouch/TodoTxtTouch.java
+++ b/src/com/todotxt/todotxttouch/TodoTxtTouch.java
@@ -55,6 +55,7 @@
 import android.content.SharedPreferences.Editor;
 import android.content.SharedPreferences.OnSharedPreferenceChangeListener;
 import android.content.res.Resources;
+import android.graphics.Paint;
 import android.os.AsyncTask;
 import android.os.Bundle;
 import android.text.SpannableString;
@@ -831,7 +832,8 @@ public View getView(int position, View convertView, ViewGroup parent) {
 				if (task.getPriority() == Priority.NONE) {
 					holder.taskprio.setText("   ");
 				} else {
-					holder.taskprio.setText(task.getPriority().inFileFormat());
+					holder.taskprio
+							.setText(task.getPriority().inScreenFormat());
 				}
 				SpannableString ss = new SpannableString(task.inScreenFormat());
 				Util.setGray(ss, task.getProjects());
@@ -857,11 +859,21 @@ public View getView(int position, View convertView, ViewGroup parent) {
 				default:
 					holder.taskprio.setTextColor(res.getColor(R.color.black));
 				}
+				if (task.isCompleted()) {
+					Log.v(TAG, "Striking through " + task.getText());
+					holder.tasktext.setPaintFlags(holder.tasktext
+							.getPaintFlags() | Paint.STRIKE_THRU_TEXT_FLAG);
+				} else {
+					holder.tasktext.setPaintFlags(holder.tasktext
+							.getPaintFlags() & ~Paint.STRIKE_THRU_TEXT_FLAG);
+				}
+
 				// hide line numbers unless show preference is checked
 				if (!m_app.m_prefs.getBoolean("showlinenumberspref", false)) {
-					holder.taskid.setTextColor(res.getColor(R.color.white));
+					holder.taskid.setVisibility(View.GONE);
+				} else {
+					holder.taskid.setVisibility(View.VISIBLE);
 				}
-
 			}
 			return convertView;
 		}