UI tweaks

Author: sdsantos

blockstack-sdk/src/main/res/layout/activity_connect.xml

@@ -14,8 +14,7 @@
         <androidx.constraintlayout.widget.ConstraintLayout
             android:layout_width="match_parent"
             android:layout_height="match_parent"
-            android:layout_margin="8dp"
-            android:paddingBottom="8dp">
+            android:paddingBottom="16dp">
 
             <LinearLayout
                 android:id="@+id/connect_icons"
@@ -43,7 +42,8 @@
                     android:layout_marginTop="8dp"
                     android:layout_marginRight="-8dp"
                     android:importantForAccessibility="no"
-                    android:src="@drawable/ic_link" />
+                    android:src="@drawable/ic_link"
+                    android:elevation="1dp"/>
 
                 <ImageView
                     android:id="@+id/connect_connect_icon"
@@ -83,19 +83,19 @@
                 android:layout_height="wrap_content"
                 android:layout_marginTop="4dp"
                 android:importantForAccessibility="no"
-                android:paddingLeft="16dp"
-                android:paddingRight="16dp"
+                android:paddingStart="24dp"
                 android:src="@drawable/ic_lock"
                 app:layout_constraintStart_toStartOf="parent"
                 app:layout_constraintTop_toTopOf="@id/connect_secret_key_desc_title"
-                app:tint="?colorPrimary" />
+                app:tint="?colorPrimary"
+                tools:ignore="RtlSymmetry"/>
 
             <TextView
                 android:id="@+id/connect_secret_key_desc_title"
                 android:layout_width="0dp"
                 android:layout_height="wrap_content"
                 android:layout_marginTop="16dp"
-                android:layout_marginRight="16dp"
+                android:layout_marginHorizontal="16dp"
                 android:gravity="top"
                 android:text="@string/connect_activity_encryption_desc"
                 android:textAppearance="?textAppearanceBody1"
@@ -117,19 +117,19 @@
                 android:layout_height="wrap_content"
                 android:layout_marginTop="4dp"
                 android:importantForAccessibility="no"
-                android:paddingLeft="16dp"
-                android:paddingRight="16dp"
+                android:paddingStart="24dp"
                 android:src="@drawable/ic_track"
                 app:layout_constraintStart_toStartOf="parent"
                 app:layout_constraintTop_toTopOf="@id/connect_tracking_desc_title"
-                app:tint="?colorPrimary" />
+                app:tint="?colorPrimary"
+                tools:ignore="RtlSymmetry"/>
 
             <TextView
                 android:id="@+id/connect_tracking_desc_title"
                 android:layout_width="0dp"
                 android:layout_height="wrap_content"
                 android:layout_marginTop="16dp"
-                android:layout_marginEnd="16dp"
+                android:layout_marginHorizontal="16dp"
                 android:gravity="top"
                 android:text="@string/connect_activity_tracking_desc"
                 android:textAppearance="?textAppearanceBody1"
@@ -143,8 +143,8 @@
 
 
     <com.google.android.material.button.MaterialButton
-        style="?blockstackMainButtonStyle"
         android:id="@+id/connect_get_secret_key"
+        style="?blockstackMainButtonStyle"
         android:layout_height="56dp"
         android:layout_marginLeft="16dp"
         android:layout_marginRight="16dp"
@@ -156,9 +156,10 @@
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:orientation="horizontal">
+
     <com.google.android.material.button.MaterialButton
-        style="?blockstackBottomButtonStyle"
         android:id="@+id/connect_sign_in"
+        style="?blockstackBottomButtonStyle"
         android:layout_height="wrap_content"
         android:layout_width="match_parent"
         android:layout_weight="1"
@@ -167,8 +168,8 @@
         android:text="@string/connect_dialog_restore" />
 
     <com.google.android.material.button.MaterialButton
-        style="?blockstackBottomButtonStyle"
         android:id="@+id/connect_how_it_works"
+        style="?blockstackBottomButtonStyle"
         android:layout_height="wrap_content"
         android:layout_width="match_parent"
         android:layout_weight="1"
@@ -177,5 +178,4 @@
         android:text="@string/connect_dialog_help" />
 
     </LinearLayout>
-
-</LinearLayout>
+</LinearLayout>

blockstack-sdk/src/main/res/layout/activity_connect_how_it_works.xml

@@ -1,11 +1,9 @@
 <?xml version="1.0" encoding="utf-8"?>
-
 <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
     xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="match_parent"
-    android:orientation="vertical"
-    android:padding="16dp">
+    android:orientation="vertical">
 
     <androidx.core.widget.NestedScrollView
         android:layout_width="match_parent"
@@ -15,20 +13,19 @@
         <TextView
             android:id="@+id/textview_help"
             style="?textAppearanceBody1"
+            android:padding="16dp"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:text="@string/connect_help" />
 
-
     </androidx.core.widget.NestedScrollView>
 
     <com.google.android.material.button.MaterialButton
-        style="?blockstackMainButtonStyle"
         android:id="@+id/button_get_started"
+        style="?blockstackMainButtonStyle"
         android:layout_width="match_parent"
-        android:layout_height="wrap_content"
-        android:layout_marginTop="8dp"
-        android:layout_marginBottom="8dp"
+        android:layout_height="56dp"
+        android:layout_marginHorizontal="16dp"
         android:text="@string/org_blockstack_get_started"
         android:textAllCaps="false" />
 
@@ -37,6 +34,7 @@
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:gravity="center"
+        android:padding="16dp"
         android:orientation="horizontal">
 
         <TextView
@@ -49,15 +47,16 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:layout_marginHorizontal="4dp"
-            android:src="@drawable/org_blockstack_black_logo" />
+            android:src="@drawable/org_blockstack_black_logo"
+            android:importantForAccessibility="no"
+            />
 
         <TextView
             style="?textAppearanceSubtitle2"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:text="Blockstack"
+            android:text="@string/blockstack"
             tools:ignore="HardcodedText" />
 
     </LinearLayout>
-
 </LinearLayout>

blockstack-sdk/src/main/res/values/strings.xml

@@ -1,4 +1,5 @@
 <resources xmlns:tools="http://schemas.android.com/tools">
+    <string name="blockstack" translatable="false">Blockstack</string>
     <string name="org_blockstack_login">Get your Secret Key</string>
     <string name="org_blockstack_get_started" tools:ignore="MissingTranslation">Get Started</string>
     <string name="connect_dialog_title" tools:ignore="MissingTranslation">%1$s guarantees your privacy by encrypting everything</string>