Work on add subkey dialog design

2 files changed, 88 insertions, 73 deletions

diff --git a/OpenKeychain/src/main/res/layout/create_key_dialog.xml b/OpenKeychain/src/main/res/layout/add_subkey_dialog.xml
index 31c5fbefc..a4bd4ed6a 100644
--- a/OpenKeychain/src/main/res/layout/create_key_dialog.xml
+++ b/OpenKeychain/src/main/res/layout/add_subkey_dialog.xml
@@ -6,64 +6,90 @@
     <TableLayout
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
+        android:paddingTop="8dp"
         android:paddingLeft="16dp"
         android:paddingRight="16dp"
         android:stretchColumns="1">
 
+        <TextView
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_gravity="center_vertical"
+            android:text="@string/key_creation_el_gamal_info" />
+
         <TableRow>
 
             <TextView
-                android:id="@+id/label_expiry"
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
                 android:layout_gravity="center_vertical"
-                android:paddingRight="10dip"
-                android:text="@string/label_expiry" />
+                android:text="@string/label_algorithm" />
 
-            <Button
-                android:id="@+id/expiry"
+            <Spinner
+                android:id="@+id/add_subkey_algorithm"
                 android:layout_width="match_parent"
-                android:layout_height="40dp"
-                android:text="@string/none"
-                android:background="@drawable/button_edgy" />
+                android:layout_height="wrap_content"
+                android:padding="4dp" />
         </TableRow>
 
-        <TableRow android:id="@+id/row_certify">
+        <TableRow>
 
             <TextView
-                android:id="@+id/label_usage"
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
                 android:layout_gravity="center_vertical"
-                android:paddingRight="10dip"
-                android:text="@string/label_usage" />
+                android:text="@string/label_key_size" />
 
-            <CheckBox
-                android:id="@+id/chkCertify"
-                android:enabled="false"
-                android:layout_width="wrap_content"
+            <Spinner
+                android:id="@+id/add_subkey_size"
+                android:layout_width="match_parent"
                 android:layout_height="wrap_content"
-                android:text="@string/flag_certify" />
+                android:gravity="right"
+                android:padding="4dp" />
         </TableRow>
 
-        <TableRow android:id="@+id/row_sign">
+        <TextView
+            android:id="@+id/add_subkey_custom_key_size_label"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_gravity="center_vertical"
+            android:text="@string/key_size_custom_info"
+            android:visibility="gone" />
+
+        <EditText
+            android:id="@+id/add_subkey_custom_key_size_input"
+            android:layout_width="0dip"
+            android:layout_height="wrap_content"
+            android:layout_weight="1"
+            android:inputType="number"
+            android:visibility="gone" />
+
+        <TextView
+            android:id="@+id/add_subkey_custom_key_size_info"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_gravity="center_vertical" />
+
+        <TableRow>
 
             <TextView
-                android:id="@+id/label_usage2"
+                android:id="@+id/label_expiry"
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
                 android:layout_gravity="center_vertical"
                 android:paddingRight="10dip"
-                android:text="@string/label_usage" />
+                android:text="@string/label_expiry" />
 
             <CheckBox
-                android:id="@+id/chkSign"
+                android:id="@+id/add_subkey_no_expiry"
+                android:checked="true"
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
-                android:text="@string/flag_sign" />
+                android:text="@string/btn_no_date" />
+
         </TableRow>
 
-        <TableRow android:id="@+id/row_encrypt">
+        <TableRow>
 
             <TextView
                 android:layout_width="wrap_content"
@@ -71,49 +97,49 @@
                 android:layout_gravity="center_vertical"
                 android:paddingRight="10dip" />
 
-            <CheckBox
-                android:id="@+id/chkEncrypt"
+            <DatePicker
+                android:id="@+id/add_subkey_expiry_date_picker"
+                android:layout_gravity="center_horizontal"
+                android:visibility="gone"
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
-                android:text="@string/flag_encrypt" />
+                android:spinnersShown="true"
+                android:calendarViewShown="false" />
+
         </TableRow>
 
-        <TableRow android:id="@+id/row_authenticate">
+        <TableRow>
 
             <TextView
+                android:id="@+id/label_usage"
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
                 android:layout_gravity="center_vertical"
-                android:paddingRight="10dip" />
+                android:paddingRight="10dip"
+                android:text="@string/label_usage" />
+
 
             <CheckBox
-                android:id="@+id/chkAuthenticate"
+                android:id="@+id/add_subkey_flag_certify"
+                android:enabled="false"
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
-                android:text="@string/flag_authenticate" />
+                android:text="@string/flag_certify" />
         </TableRow>
 
-        <TextView
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:layout_gravity="center_vertical"
-            android:padding="4dp"
-            android:text="@string/key_creation_el_gamal_info" />
-
         <TableRow>
 
             <TextView
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
                 android:layout_gravity="center_vertical"
-                android:padding="4dp"
-                android:text="@string/label_algorithm" />
+                android:paddingRight="10dip" />
 
-            <Spinner
-                android:id="@+id/create_key_algorithm"
-                android:layout_width="match_parent"
+            <CheckBox
+                android:id="@+id/add_subkey_flag_sign"
+                android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
-                android:padding="4dp" />
+                android:text="@string/flag_sign" />
         </TableRow>
 
         <TableRow>
@@ -122,40 +148,29 @@
                 android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
                 android:layout_gravity="center_vertical"
-                android:padding="4dp"
-                android:text="@string/label_key_size" />
+                android:paddingRight="10dip" />
 
-            <Spinner
-                android:id="@+id/create_key_size"
-                android:layout_width="match_parent"
+            <CheckBox
+                android:id="@+id/add_subkey_flag_encrypt"
+                android:layout_width="wrap_content"
                 android:layout_height="wrap_content"
-                android:gravity="right"
-                android:padding="4dp" />
+                android:text="@string/flag_encrypt" />
         </TableRow>
 
-        <TextView
-            android:id="@+id/custom_key_size_label"
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:layout_gravity="center_vertical"
-            android:padding="4dp"
-            android:text="@string/key_size_custom_info"
-            android:visibility="gone" />
+        <TableRow>
 
-        <EditText
-            android:id="@+id/custom_key_size_input"
-            android:layout_width="0dip"
-            android:layout_height="wrap_content"
-            android:layout_weight="1"
-            android:inputType="number"
-            android:visibility="gone" />
+            <TextView
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:layout_gravity="center_vertical"
+                android:paddingRight="10dip" />
 
-        <TextView
-            android:id="@+id/custom_key_size_info"
-            android:layout_width="wrap_content"
-            android:layout_height="wrap_content"
-            android:layout_gravity="center_vertical"
-            android:padding="4dp" />
+            <CheckBox
+                android:id="@+id/add_subkey_flag_authenticate"
+                android:layout_width="wrap_content"
+                android:layout_height="wrap_content"
+                android:text="@string/flag_authenticate" />
+        </TableRow>
 
     </TableLayout>
 
diff --git a/OpenKeychain/src/main/res/layout/expiry_dialog.xml b/OpenKeychain/src/main/res/layout/edit_subkey_expiry_dialog.xml
index 14fff0b3f..062f07863 100644
--- a/OpenKeychain/src/main/res/layout/expiry_dialog.xml
+++ b/OpenKeychain/src/main/res/layout/edit_subkey_expiry_dialog.xml
@@ -4,8 +4,8 @@
     android:layout_width="match_parent"
     android:layout_height="match_parent">
 
-    <DatePicker xmlns:android="http://schemas.android.com/apk/res/android"
-        android:id="@+id/datePicker"
+    <DatePicker
+        android:id="@+id/edit_subkey_expiry_date_picker"
         android:layout_gravity="center_horizontal"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"