From 67a867ac69046b929eb777c0d241dafcc171f2ee Mon Sep 17 00:00:00 2001
From: Alex <askutarenko@askutarenko-macbookpro.roam.corp.google.com>
Date: Thu, 18 Aug 2016 16:12:55 -0500
Subject: [PATCH] ui fixes

check list text size
important info text size
important info item focuses keyboard
about text is in center

resolves #38
---
 .../ImportantInformationItemActivity.java     | 14 +++-
 app/src/main/res/layout/activity_about.xml    | 71 +++++++++++--------
 .../layout/activity_important_information.xml |  4 --
 app/src/main/res/layout/fragment_dummy.xml    | 34 ---------
 .../main/res/layout/view_checklist_item.xml   |  3 +-
 .../view_important_information_item.xml       |  4 +-
 6 files changed, 58 insertions(+), 72 deletions(-)
 delete mode 100644 app/src/main/res/layout/fragment_dummy.xml

diff --git a/app/src/main/java/com/g11x/checklistapp/ImportantInformationItemActivity.java b/app/src/main/java/com/g11x/checklistapp/ImportantInformationItemActivity.java
index d6f955e..39d4e5c 100644
--- a/app/src/main/java/com/g11x/checklistapp/ImportantInformationItemActivity.java
+++ b/app/src/main/java/com/g11x/checklistapp/ImportantInformationItemActivity.java
@@ -23,8 +23,9 @@
 import android.os.Bundle;
 import android.support.v7.app.AppCompatActivity;
 import android.view.View;
+import android.view.WindowManager;
 import android.widget.Button;
-import android.widget.TextView;
+import android.widget.EditText;
 
 import com.g11x.checklistapp.data.Database;
 
@@ -44,8 +45,17 @@ public void onClick(View view) {
     });
   }
 
+  @Override
+  protected void onStart() {
+    super.onStart();
+
+    EditText editText = (EditText)findViewById(R.id.title);
+    editText.requestFocus();
+    getWindow().setSoftInputMode(WindowManager.LayoutParams.SOFT_INPUT_STATE_VISIBLE);
+  }
+
   private void onClickCreate() {
-    TextView title = (TextView) findViewById(R.id.title);
+    EditText title = (EditText) findViewById(R.id.title);
     String titleText = title.getText().toString();
     Intent intent = new Intent(this, ImportantInformationActivity.class);
     intent.putExtra("title", titleText);
diff --git a/app/src/main/res/layout/activity_about.xml b/app/src/main/res/layout/activity_about.xml
index 20f0c90..1a2e4a5 100644
--- a/app/src/main/res/layout/activity_about.xml
+++ b/app/src/main/res/layout/activity_about.xml
@@ -29,42 +29,53 @@
       android:id="@+id/main_content"
       android:layout_width="match_parent"
       android:layout_height="match_parent"
+      android:gravity="center"
       tools:context="com.g11x.checklistapp.AboutActivity">
 
-    <TextView
+    <LinearLayout
+        android:layout_width="match_parent"
         android:layout_height="wrap_content"
-        android:layout_width="wrap_content"
-        android:text="@string/app_name"
-        android:textStyle="bold"
-        android:textSize="24sp"
-        android:id="@+id/textView"
-        app:layout_constraintLeft_toLeftOf="@+id/main_content"
+        android:gravity="center_horizontal"
+        android:orientation="vertical"
+        app:layout_constraintBottom_toBottomOf="@+id/main_content"
         app:layout_constraintTop_toTopOf="@+id/main_content"
-        app:layout_constraintRight_toRightOf="@+id/main_content"
-        app:layout_constraintBottom_toBottomOf="@+id/main_content"/>
+        app:layout_constraintVertical_bias="0.5">
 
-    <TextView
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:ems="10"
-        android:id="@+id/all_code_copy_google"
-        android:text="@string/about_text"
-        android:background="@android:color/transparent"
-        android:gravity="center"
-        app:layout_constraintLeft_toLeftOf="@+id/main_content"
-        app:layout_constraintTop_toBottomOf="@+id/textView"
-        android:layout_marginTop="16dp"
-        app:layout_constraintRight_toRightOf="@+id/main_content"/>
+      <TextView
+          android:id="@+id/textView"
+          android:layout_width="wrap_content"
+          android:layout_height="wrap_content"
+          android:text="@string/app_name"
+          android:textSize="24sp"
+          android:textStyle="bold"
+          app:layout_constraintBottom_toBottomOf="@+id/main_content"
+          app:layout_constraintLeft_toLeftOf="@+id/main_content"
+          app:layout_constraintRight_toRightOf="@+id/main_content"
+          app:layout_constraintTop_toTopOf="@+id/main_content"/>
 
-    <TextView
-        android:layout_width="wrap_content"
-        android:layout_height="wrap_content"
-        android:id="@+id/apache_license_link"
-        android:text="@string/apache_license_link"
-        app:layout_constraintLeft_toLeftOf="@+id/main_content"
-        app:layout_constraintTop_toBottomOf="@+id/all_code_copy_google"
-        android:layout_marginTop="16dp"
-        app:layout_constraintRight_toRightOf="@+id/main_content"/>
+      <TextView
+          android:id="@+id/all_code_copy_google"
+          android:layout_width="wrap_content"
+          android:layout_height="wrap_content"
+          android:layout_marginTop="16dp"
+          android:background="@android:color/transparent"
+          android:ems="10"
+          android:gravity="center"
+          android:text="@string/about_text"
+          app:layout_constraintLeft_toLeftOf="@+id/main_content"
+          app:layout_constraintRight_toRightOf="@+id/main_content"
+          app:layout_constraintTop_toBottomOf="@+id/textView"/>
+
+      <TextView
+          android:id="@+id/apache_license_link"
+          android:layout_width="wrap_content"
+          android:layout_height="wrap_content"
+          android:layout_marginTop="16dp"
+          android:text="@string/apache_license_link"
+          app:layout_constraintLeft_toLeftOf="@+id/main_content"
+          app:layout_constraintRight_toRightOf="@+id/main_content"
+          app:layout_constraintTop_toBottomOf="@+id/all_code_copy_google"/>
+    </LinearLayout>
 
 
   </android.support.constraint.ConstraintLayout>
diff --git a/app/src/main/res/layout/activity_important_information.xml b/app/src/main/res/layout/activity_important_information.xml
index 94dd90f..44f6ea5 100644
--- a/app/src/main/res/layout/activity_important_information.xml
+++ b/app/src/main/res/layout/activity_important_information.xml
@@ -28,10 +28,6 @@
       android:id="@+id/activity_important_information"
       android:layout_width="match_parent"
       android:layout_height="match_parent"
-      android:paddingBottom="@dimen/activity_vertical_margin"
-      android:paddingLeft="@dimen/activity_horizontal_margin"
-      android:paddingRight="@dimen/activity_horizontal_margin"
-      android:paddingTop="@dimen/activity_vertical_margin"
       tools:context="com.g11x.checklistapp.ImportantInformationActivity"
       android:orientation="horizontal">
 
diff --git a/app/src/main/res/layout/fragment_dummy.xml b/app/src/main/res/layout/fragment_dummy.xml
deleted file mode 100644
index 0ddf2c3..0000000
--- a/app/src/main/res/layout/fragment_dummy.xml
+++ /dev/null
@@ -1,34 +0,0 @@
-<?xml version="1.0" encoding="utf-8"?>
-<!--
-  ~ Copyright © 2016 Google Inc.
-  ~
-  ~ Licensed under the Apache License, Version 2.0 (the "License");
-  ~ you may not use this file except in compliance with the License.
-  ~ You may obtain a copy of the License at
-  ~
-  ~     http://www.apache.org/licenses/LICENSE-2.0
-  ~
-  ~ Unless required by applicable law or agreed to in writing, software
-  ~ distributed under the License is distributed on an "AS IS" BASIS,
-  ~ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-  ~ See the License for the specific language governing permissions and
-  ~ limitations under the License.
-  ~
-  -->
-
-<LinearLayout
-    xmlns:android="http://schemas.android.com/apk/res/android"
-    android:layout_width="match_parent"
-    android:layout_height="match_parent"
-    android:background="?android:attr/selectableItemBackground"
-    android:clickable="true"
-    android:orientation="vertical">
-
-  <TextView
-      android:id="@+id/dummy_text"
-      android:layout_width="match_parent"
-      android:layout_height="match_parent"
-      android:textSize="30dp"
-      android:gravity="center"
-      android:padding="20dp"/>
-</LinearLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/view_checklist_item.xml b/app/src/main/res/layout/view_checklist_item.xml
index 716347f..e8f98a1 100644
--- a/app/src/main/res/layout/view_checklist_item.xml
+++ b/app/src/main/res/layout/view_checklist_item.xml
@@ -26,7 +26,8 @@
 
   <TextView
       android:id="@+id/info_text"
-      android:padding="20dp"
+      android:padding="14dp"
+      android:textSize="18sp"
       android:layout_width="match_parent"
       android:layout_height="wrap_content" />
 </LinearLayout>
\ No newline at end of file
diff --git a/app/src/main/res/layout/view_important_information_item.xml b/app/src/main/res/layout/view_important_information_item.xml
index 2eebf40..b8dc914 100644
--- a/app/src/main/res/layout/view_important_information_item.xml
+++ b/app/src/main/res/layout/view_important_information_item.xml
@@ -23,5 +23,7 @@
   <TextView
       android:id="@+id/data"
       android:layout_width="match_parent"
-      android:layout_height="wrap_content"/>
+      android:layout_height="wrap_content"
+      android:padding="14dp"
+      android:textSize="18sp"/>
 </LinearLayout>
\ No newline at end of file