From 7c41ac2d29130aee4c90c221346c9176a507d86c Mon Sep 17 00:00:00 2001
From: Phillip Hsu <philliphsu@outlook.com>
Date: Fri, 29 Jul 2016 15:03:26 -0700
Subject: [PATCH] Basic implementation of edit timer screen

---
 .../clock2/edittimer/EditTimerActivity.java   | 43 ++++++++++++++++---
 .../main/res/layout/activity_edit_timer.xml   | 24 ++++++++++-
 .../main/res/layout/content_numpad_timer.xml  |  1 +
 3 files changed, 61 insertions(+), 7 deletions(-)

diff --git a/app/src/main/java/com/philliphsu/clock2/edittimer/EditTimerActivity.java b/app/src/main/java/com/philliphsu/clock2/edittimer/EditTimerActivity.java
index 838235c..0562f0e 100644
--- a/app/src/main/java/com/philliphsu/clock2/edittimer/EditTimerActivity.java
+++ b/app/src/main/java/com/philliphsu/clock2/edittimer/EditTimerActivity.java
@@ -18,12 +18,10 @@ import butterknife.OnClick;
 import butterknife.OnLongClick;
 import butterknife.OnTouch;
 
+// TODO: Rename to CreateTimerActivity
 public class EditTimerActivity extends BaseActivity {
     private static final int FIELD_LENGTH = 2;
 
-    private final int[] mInput = new int[6];
-    private int mCursorAt;
-
     @Bind(R.id.appbar) ViewGroup mAppBar;
     @Bind(R.id.label) TextView mLabel;
     @Bind(R.id.hour) EditText mHour;
@@ -41,6 +39,7 @@ public class EditTimerActivity extends BaseActivity {
     @Override
     protected void onCreate(Bundle savedInstanceState) {
         super.onCreate(savedInstanceState);
+//        updateStartButtonVisibility();
     }
 
     @Override
@@ -54,6 +53,11 @@ public class EditTimerActivity extends BaseActivity {
         return 0;
     }
 
+    @Override
+    public void finish() {
+        super.finish();
+    }
+
     @OnClick({ R.id.zero, R.id.one, R.id.two, R.id.three, R.id.four,
             R.id.five, R.id.six, R.id.seven, R.id.eight, R.id.nine })
     void onClick(TextView view) {
@@ -63,6 +67,7 @@ public class EditTimerActivity extends BaseActivity {
         int at = field.getSelectionStart();
         field.getText().replace(at, at + 1, view.getText());
         field.setSelection(at + 1);
+//        updateStartButtonVisibility();
         if (field.getSelectionStart() == FIELD_LENGTH) {
             // At the end of the current field, so try to focus to the next field.
             // The search will return null if no view can be focused next.
@@ -78,7 +83,7 @@ public class EditTimerActivity extends BaseActivity {
     }
 
     @OnTouch({ R.id.hour, R.id.minute, R.id.second })
-    boolean captureTouchEvent(EditText field, MotionEvent event) {
+    boolean switchField(EditText field, MotionEvent event) {
         int inType = field.getInputType(); // backup the input type
         field.setInputType(InputType.TYPE_NULL); // disable soft input
         boolean result = field.onTouchEvent(event); // call native handler
@@ -114,6 +119,7 @@ public class EditTimerActivity extends BaseActivity {
         } else {
             field.getText().replace(at - 1, at, "0");
             field.setSelection(at - 1);
+//            updateStartButtonVisibility();
         }
     }
     
@@ -126,6 +132,7 @@ public class EditTimerActivity extends BaseActivity {
         mHour.setSelection(0); // always move the cursor WHILE the field is focused, NEVER focus after!
         mMinute.setSelection(0);
         mSecond.setSelection(0);
+//        mFab.hide(); // instead of updateStartButtonVisibility() because we know everything's zero
         return true;
     }
 
@@ -134,7 +141,33 @@ public class EditTimerActivity extends BaseActivity {
         // TODO: Show the edit label alert dialog.
     }
 
-    public final EditText getFocusedField() {
+    @OnClick(R.id.fab)
+    void startTimer() {
+        int hour = Integer.parseInt(mHour.getText().toString());
+        int minute = Integer.parseInt(mMinute.getText().toString());
+        int second = Integer.parseInt(mSecond.getText().toString());
+        if (hour == 0 && minute == 0 && second == 0)
+            return; // TODO: we could show a toast instead if we cared
+        // TODO: do something with the label
+        mLabel.getText();
+        // TODO: Pass back an intent with the data, or make Timer parcelable
+        // and pass back an instance of Timer. Consider overriding finish()
+        // and doing it there.
+        finish();
+    }
+
+    private EditText getFocusedField() {
         return (EditText) mAppBar.findFocus();
     }
+
+//    private void updateStartButtonVisibility() {
+//        // TODO: parse the field's text to an integer and check > 0 instead?
+//        if (TextUtils.equals(mHour.getText(), "00")
+//                && TextUtils.equals(mMinute.getText(), "00")
+//                && TextUtils.equals(mSecond.getText(), "00")) {
+//            mFab.hide();
+//        } else {
+//            mFab.show();
+//        }
+//    }
 }
diff --git a/app/src/main/res/layout/activity_edit_timer.xml b/app/src/main/res/layout/activity_edit_timer.xml
index 0f5482f..492364a 100644
--- a/app/src/main/res/layout/activity_edit_timer.xml
+++ b/app/src/main/res/layout/activity_edit_timer.xml
@@ -112,13 +112,33 @@
         
     </RelativeLayout>
 
+    <!-- When fab.hide() is called, it is gone from the layout.
+    The numpad is anchored in between the fab and the appbar,
+    so if the fab is gone, the numpad won't be constrained
+    above the fab anymore. This placeholder takes up the same
+    height as the fab and is always present in the layout,
+    so we can hide() the fab without losing the bottom anchoring
+    on the numpad. -->
+    <!-- TODO: Use 56dp height for v21. Consider writing
+     a dimen resource for this and make different dimen version files,
+      and/or a style resource and make different style version files. -->
+    <!--<Space
+        android:id="@+id/fab_placeholder"
+        android:layout_width="0dp"
+        android:layout_height="88dp"
+        android:layout_alignParentBottom="true"
+        android:layout_centerHorizontal="true"/>-->
+
+    <!-- TODO: Consider defining a style instead. Use the same attrs
+    as below for base API versions, and make a v21 version
+    that includes margin bottom of 16dp. -->
     <android.support.design.widget.FloatingActionButton
         android:id="@+id/fab"
         android:layout_width="wrap_content"
         android:layout_height="wrap_content"
         android:src="@drawable/ic_half_day_1_black_24dp"
-        android:layout_alignParentBottom="true"
-        android:layout_centerHorizontal="true"/>
+        android:layout_centerHorizontal="true"
+        android:layout_alignParentBottom="true"/>
 
     <!-- We don't really need the overhead of the GridLayoutNumpad,
      because it would be a major hassle to adapt it for timers. -->
diff --git a/app/src/main/res/layout/content_numpad_timer.xml b/app/src/main/res/layout/content_numpad_timer.xml
index 0bfb1b8..1a12ca2 100644
--- a/app/src/main/res/layout/content_numpad_timer.xml
+++ b/app/src/main/res/layout/content_numpad_timer.xml
@@ -8,4 +8,5 @@
         android:src="@drawable/ic_backspace_24dp"
         style="@style/GridLayoutNumpadElement"
         grid:layout_column="2"/>
+
 </merge>
\ No newline at end of file