From 773b78a363d6237018cabb9c32c0ab4a11cc7eed Mon Sep 17 00:00:00 2001
From: Henrik Tunedal <tunedal@gmail.com>
Date: Mon, 7 Mar 2011 17:47:17 +0100
Subject: [PATCH] Improve listview item layout

---
 res/layout/apklistitem.xml | 55 +++++++++++++++++------------
 res/layout/applistitem.xml | 71 +++++++++++++++++++++-----------------
 2 files changed, 72 insertions(+), 54 deletions(-)

diff --git a/res/layout/apklistitem.xml b/res/layout/apklistitem.xml
index 774fc7557..b95e8587f 100644
--- a/res/layout/apklistitem.xml
+++ b/res/layout/apklistitem.xml
@@ -1,28 +1,39 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
-	android:id="@+id/vw1" android:layout_width="fill_parent"
-	android:layout_height="wrap_content" android:orientation="horizontal">
+<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
+		android:layout_width="fill_parent"
+		android:layout_height="wrap_content"
+		android:orientation="vertical"
+		android:paddingTop="5dp"
+		android:paddingBottom="5dp"
+		android:paddingLeft="10dp"
+		android:paddingRight="10dp">
 
-	<RelativeLayout android:layout_width="fill_parent"
-		android:layout_height="wrap_content" android:orientation="vertical">
+  <TextView android:id="@+id/buildtype"
+	    android:textSize="12sp"
+	    android:layout_alignParentRight="true"
+	    android:layout_height="wrap_content"
+	    android:layout_width="wrap_content" />
 
-		<TextView android:id="@+id/buildtype" android:textSize="12sp"
-			android:layout_alignParentRight="true" android:layout_height="wrap_content"
-			android:layout_width="wrap_content" />
+  <TextView android:id="@+id/version"
+	    android:textStyle="bold"
+	    android:singleLine="true"
+	    android:ellipsize="marquee"
+	    android:layout_width="fill_parent"
+	    android:layout_height="wrap_content"
+	    android:textSize="18sp" />
 
-		<TextView android:id="@+id/version" android:textStyle="bold"
-			android:singleLine="true" android:ellipsize="marquee"
-			android:layout_width="fill_parent" android:layout_height="wrap_content"
-			android:textSize="18sp" />
+  <TextView android:id="@+id/status"
+	    android:textSize="12sp"
+	    android:layout_below="@id/version"
+	    android:layout_alignParentRight="false"
+	    android:layout_height="wrap_content"
+	    android:layout_width="wrap_content" />
 
-		<TextView android:id="@+id/status" android:textSize="12sp"
-			android:layout_below="@id/version" android:layout_alignParentRight="false"
-			android:layout_height="wrap_content" android:layout_width="wrap_content" />
+  <TextView android:id="@+id/size"
+	    android:textSize="12sp"
+	    android:layout_below="@id/version"
+	    android:layout_alignParentRight="true"
+	    android:layout_height="wrap_content"
+	    android:layout_width="wrap_content" />
 
-		<TextView android:id="@+id/size" android:textSize="12sp"
-			android:layout_below="@id/version" android:layout_alignParentRight="true"
-			android:layout_height="wrap_content" android:layout_width="wrap_content" />
-
-	</RelativeLayout>
-
-</LinearLayout>
+</RelativeLayout>
diff --git a/res/layout/applistitem.xml b/res/layout/applistitem.xml
index dacff8aef..486d6d438 100644
--- a/res/layout/applistitem.xml
+++ b/res/layout/applistitem.xml
@@ -1,39 +1,46 @@
 <?xml version="1.0" encoding="UTF-8"?>
-<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
-	android:id="@+id/vw1" android:layout_width="fill_parent"
-	android:layout_height="wrap_content" android:orientation="vertical">
+<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
+		android:orientation="vertical"
+		android:layout_width="fill_parent"
+		android:layout_height="wrap_content"
+		android:paddingTop="0dp"
+		android:paddingBottom="5dp"
+		android:paddingLeft="5dp"
+		android:paddingRight="10dp">
 
-	<LinearLayout android:id="@+id/vw2" android:layout_width="fill_parent"
-		android:layout_height="wrap_content" android:orientation="horizontal">
+  <ImageView android:id="@+id/icon"
+	     android:scaleType="centerInside"
+	     android:layout_width="50dp"
+	     android:layout_height="50dp"
+	     android:padding="5dp"/>
 
-		<ImageView android:id="@+id/icon" android:cropToPadding="true"
-			android:padding="4px" android:scaleType="centerInside"
-			android:layout_height="50px" android:layout_width="50px" />
+  <TextView android:id="@+id/name"
+	    android:textSize="18sp"
+	    android:textStyle="bold"
+	    android:singleLine="true"
+	    android:ellipsize="marquee"
+	    android:layout_width="fill_parent"
+	    android:layout_height="wrap_content"
+	    android:layout_toRightOf="@id/icon"/>
 
-		<RelativeLayout android:layout_width="fill_parent"
-			android:layout_height="wrap_content" android:orientation="vertical">
+  <TextView android:id="@+id/status"
+	    android:textSize="12sp"
+	    android:layout_width="fill_parent"
+	    android:layout_height="wrap_content"
+	    android:layout_toRightOf="@id/icon"
+	    android:layout_below="@id/name"/>
 
-			<TextView android:id="@+id/name" android:textStyle="bold"
-				android:singleLine="true" android:ellipsize="marquee"
-				android:layout_width="fill_parent" android:layout_height="wrap_content"
-				android:textSize="18sp" />
+  <TextView android:id="@+id/license"
+	    android:textSize="12sp"
+	    android:layout_width="wrap_content"
+	    android:layout_height="wrap_content"
+	    android:layout_below="@id/name"
+	    android:layout_alignParentRight="true"/>
 
-			<TextView android:id="@+id/license" android:textSize="12sp"
-				android:layout_below="@id/name" android:layout_alignParentRight="true"
-				android:layout_height="wrap_content" android:layout_width="wrap_content" />
+  <TextView android:id="@+id/summary"
+	    android:layout_width="fill_parent"
+	    android:layout_height="wrap_content"
+	    android:layout_below="@id/icon"
+	    android:paddingLeft="5dp"/>
 
-			<TextView android:id="@+id/status" android:layout_toLeftOf="@id/license"
-				android:layout_below="@id/name" android:layout_alignParentLeft="true"
-				android:textSize="12sp" android:layout_height="wrap_content"
-				android:layout_width="fill_parent" />
-
-
-		</RelativeLayout>
-
-	</LinearLayout>
-
-	<TextView android:id="@+id/summary" android:layout_height="wrap_content"
-		android:layout_width="fill_parent" />
-
-</LinearLayout>
-	
\ No newline at end of file
+</RelativeLayout>