From 48c843beae9c29ffcf0e413b89fd7b017f5c5f78 Mon Sep 17 00:00:00 2001
From: Vincent Breitmoser <valodim@mugenguild.com>
Date: Sun, 5 Oct 2014 02:15:43 +0200
Subject: even more layout neatification

---
 .../keychain/ui/MultiCertifyKeyFragment.java       |  1 +
 .../src/main/res/layout/certify_key_item.xml       | 51 ++++++++++++++--------
 2 files changed, 35 insertions(+), 17 deletions(-)

(limited to 'OpenKeychain/src/main')

diff --git a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/MultiCertifyKeyFragment.java b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/MultiCertifyKeyFragment.java
index 026051715..a3be8f8e1 100644
--- a/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/MultiCertifyKeyFragment.java
+++ b/OpenKeychain/src/main/java/org/sufficientlysecure/keychain/ui/MultiCertifyKeyFragment.java
@@ -124,6 +124,7 @@ public class MultiCertifyKeyFragment extends LoaderFragment
 
         mUserIdsAdapter = new MultiUserIdsAdapter(mActivity, null, 0);
         mUserIds.setAdapter(mUserIdsAdapter);
+        mUserIds.setDividerHeight(0);
 
         getLoaderManager().initLoader(0, null, this);
 
diff --git a/OpenKeychain/src/main/res/layout/certify_key_item.xml b/OpenKeychain/src/main/res/layout/certify_key_item.xml
index 297e0944b..c578473d3 100644
--- a/OpenKeychain/src/main/res/layout/certify_key_item.xml
+++ b/OpenKeychain/src/main/res/layout/certify_key_item.xml
@@ -6,31 +6,45 @@
     android:orientation="vertical"
     android:singleLine="true">
 
-    <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
-                  android:id="@+id/user_id_header"
+    <LinearLayout android:id="@+id/user_id_header"
                   android:layout_width="match_parent"
                   android:layout_height="wrap_content"
-                  android:orientation="horizontal"
-                  android:singleLine="true"
-                  android:clickable="true">
+                  android:orientation="vertical"
+                  android:clickable="true"
+                  android:layout_marginLeft="8dip"
+                  android:layout_marginTop="16dip">
 
-        <TextView
-            android:id="@+id/user_id_header_id"
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:text="0123 4567 890a bcde"
-            android:textAppearance="?android:attr/textAppearanceMedium" />
+        <LinearLayout android:layout_width="match_parent"
+                      android:layout_height="wrap_content"
+                      android:orientation="horizontal">
+
+            <TextView
+                android:id="@+id/user_id_header_id"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:text="ID: 0123 4567 890a bcde"
+                android:textAppearance="?android:attr/textAppearanceMedium"
+                />
+
+        </LinearLayout>
+
+        <View
+            android:layout_width="match_parent"
+            android:layout_height="1dip"
+            android:background="?android:attr/listDivider"
+            android:paddingLeft="24dip"
+            android:paddingRight="24dip"
+            android:layout_marginBottom="4dip"/>
 
     </LinearLayout>
 
-    <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
-                  android:id="@+id/user_id_body"
+    <LinearLayout android:id="@+id/user_id_body"
                   android:layout_width="match_parent"
                   android:layout_height="wrap_content"
-                  android:minHeight="?android:attr/listPreferredItemHeight"
                   android:orientation="horizontal"
                   android:singleLine="true"
-                  android:layout_marginLeft="15dip">
+                  android:layout_marginLeft="8dip"
+                  android:layout_marginTop="4dip">
 
         <CheckBox
             android:id="@+id/user_id_item_check_box"
@@ -47,14 +61,17 @@
             android:layout_marginTop="4dp"
             android:layout_marginBottom="4dp"
             android:layout_height="wrap_content"
-            android:layout_weight="1">
+            android:layout_weight="1"
+            android:minHeight="48dip">
 
             <TextView
                 android:id="@+id/user_id_item_name"
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
                 android:text="alice@example.com"
-                android:textAppearance="?android:attr/textAppearanceMedium" />
+                android:textAppearance="?android:attr/textAppearanceMedium"
+                android:layout_weight="1"
+                android:gravity="center_vertical"/>
 
             <TextView
                 android:id="@+id/user_id_item_addresses"
-- 
cgit v1.2.3