diff --git a/app/src/main/java/com/philliphsu/clock2/editalarm/NumpadTimePickerDialog.java b/app/src/main/java/com/philliphsu/clock2/editalarm/NumpadTimePickerDialog.java
index d53e27b..3abc66b 100644
--- a/app/src/main/java/com/philliphsu/clock2/editalarm/NumpadTimePickerDialog.java
+++ b/app/src/main/java/com/philliphsu/clock2/editalarm/NumpadTimePickerDialog.java
@@ -2,7 +2,6 @@ package com.philliphsu.clock2.editalarm;
import android.os.Bundle;
import android.support.v4.app.DialogFragment;
-import android.support.v7.widget.Toolbar;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
@@ -13,6 +12,7 @@ import com.philliphsu.clock2.R;
import butterknife.Bind;
import butterknife.ButterKnife;
+import butterknife.OnClick;
import butterknife.OnTouch;
/**
@@ -41,6 +41,7 @@ public class NumpadTimePickerDialog extends DialogFragment implements NumpadTime
*/
private int[] mInputtedDigits;
+ // Don't need to keep a reference to the dismiss ImageButton
@Bind(R.id.input_time) EditText mInputField;
@Bind(R.id.number_grid) NumpadTimePicker mNumpad;
@@ -94,16 +95,6 @@ public class NumpadTimePickerDialog extends DialogFragment implements NumpadTime
View view = inflater.inflate(R.layout.dialog_time_picker_numpad, container, false);
ButterKnife.bind(this, view);
- Toolbar toolbar = ButterKnife.findById(view, R.id.toolbar);
- toolbar.setNavigationIcon(android.R.drawable.ic_menu_close_clear_cancel);
- // Can't do a method bind on the navigation icon because we don't own it
- toolbar.setNavigationOnClickListener(new View.OnClickListener() {
- @Override
- public void onClick(View v) {
- dismiss();
- }
- });
-
// Can't do a method bind because the FAB is not part of this dialog's layout
// Also can't do the bind in the Numpad's class, because it doesn't have access to
// the OnTimeSetListener callback contained here or the dialog's dismiss()
@@ -156,6 +147,11 @@ public class NumpadTimePickerDialog extends DialogFragment implements NumpadTime
return true;
}
+ @OnClick(R.id.cancel)
+ void cancel() {
+ dismiss();
+ }
+
private void updateInputText(String inputText) {
mInputField.setText(inputText);
// Move the cursor
diff --git a/app/src/main/res/layout/content_numpad_time_picker.xml b/app/src/main/res/layout/content_numpad_time_picker.xml
index 85c7847..092ec91 100644
--- a/app/src/main/res/layout/content_numpad_time_picker.xml
+++ b/app/src/main/res/layout/content_numpad_time_picker.xml
@@ -1,30 +1,30 @@
+ xmlns:app="http://schemas.android.com/apk/res-auto">
+ app:layout_rowWeight="1"
+ app:layout_columnWeight="1"
+ app:layout_column="0"/>
+ app:layout_rowWeight="1"
+ app:layout_columnWeight="1"
+ app:layout_column="2"/>
+ android:layout_height="wrap_content"
+ android:minHeight="@dimen/fab_cell_height"
+ app:layout_columnWeight="1"
+ app:layout_column="1">
+ app:layout_rowWeight="1"
+ app:layout_columnWeight="1"
+ app:layout_column="2"/>
\ No newline at end of file
diff --git a/app/src/main/res/layout/dialog_time_picker_numpad.xml b/app/src/main/res/layout/dialog_time_picker_numpad.xml
index 76f6515..7d21558 100644
--- a/app/src/main/res/layout/dialog_time_picker_numpad.xml
+++ b/app/src/main/res/layout/dialog_time_picker_numpad.xml
@@ -1,13 +1,16 @@
+
-
+
@@ -16,21 +19,40 @@
opens, the user sees the header view as just an appbar that has
a single navigation icon, with no indication that the dialog
actually displays input. -->
+
-
+
+
+
+
+ android:layout_below="@id/header"/>
\ No newline at end of file
diff --git a/app/src/main/res/values/dimens.xml b/app/src/main/res/values/dimens.xml
index 54184f2..01c9d31 100644
--- a/app/src/main/res/values/dimens.xml
+++ b/app/src/main/res/values/dimens.xml
@@ -14,7 +14,8 @@
72dp
- 270dp
+ 300dp45sp56dp
+ 88dp