diff --git a/app/src/main/java/com/philliphsu/clock2/alarms/ExpandedAlarmViewHolder.java b/app/src/main/java/com/philliphsu/clock2/alarms/ExpandedAlarmViewHolder.java index 027330e..a36214e 100644 --- a/app/src/main/java/com/philliphsu/clock2/alarms/ExpandedAlarmViewHolder.java +++ b/app/src/main/java/com/philliphsu/clock2/alarms/ExpandedAlarmViewHolder.java @@ -64,7 +64,7 @@ public class ExpandedAlarmViewHolder extends BaseAlarmViewHolder { super.bindLabel(true, label); } - @OnClick(R.id.save) + @OnClick(R.id.ok) void save() { // TODO } 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 3382b2f..fc1ccd9 100644 --- a/app/src/main/java/com/philliphsu/clock2/editalarm/NumpadTimePickerDialog.java +++ b/app/src/main/java/com/philliphsu/clock2/editalarm/NumpadTimePickerDialog.java @@ -4,13 +4,12 @@ import android.os.Bundle; import android.view.LayoutInflater; import android.view.View; import android.view.ViewGroup; -import android.widget.EditText; +import android.widget.TextView; import com.philliphsu.clock2.R; import butterknife.Bind; import butterknife.OnClick; -import butterknife.OnTouch; /** * Created by Phillip Hsu on 7/12/2016. @@ -39,9 +38,9 @@ public class NumpadTimePickerDialog extends BaseTimePickerDialog private int[] mInputtedDigits; // Don't need to keep a reference to the dismiss ImageButton - @Bind(R.id.input_time) EditText mInputField; + @Bind(R.id.input_time) TextView mInputField; @Bind(R.id.number_grid) NumpadTimePicker mNumpad; - @Bind(R.id.focus_grabber) View mFocusGrabber; +// @Bind(R.id.focus_grabber) View mFocusGrabber; // TODO: We don't need to pass in an initial hour and minute for a new instance. @Deprecated @@ -84,7 +83,7 @@ public class NumpadTimePickerDialog extends BaseTimePickerDialog mNumpad.setOnInputChangeListener(this); mNumpad.insertDigits(mInputtedDigits); // TOneverDO: before mNumpad.setOnInputChangeListener(this); // Show the cursor immediately - mInputField.requestFocus(); +// mInputField.requestFocus(); // TODO: Disabled color //updateInputText(""); // Primarily to disable 'OK' return view; @@ -122,14 +121,14 @@ public class NumpadTimePickerDialog extends BaseTimePickerDialog @Override public void onInputDisabled() { // Steals the focus from the EditText - mFocusGrabber.requestFocus(); +// mFocusGrabber.requestFocus(); } - @OnTouch(R.id.input_time) - boolean captureTouchOnEditText() { - // Capture touch events on the EditText field, because we want it to do nothing. - return true; - } +// @OnTouch(R.id.input_time) +// boolean captureTouchOnEditText() { +// // Capture touch events on the EditText field, because we want it to do nothing. +// return true; +// } // The FAB is not defined directly in this dialog's layout, but rather in the layout // of the NumpadTimePicker. We can always reference a child of a ViewGroup that is @@ -144,11 +143,11 @@ public class NumpadTimePickerDialog extends BaseTimePickerDialog private void updateInputText(String inputText) { TimeTextUtils.setText(inputText, mInputField); - // Move the cursor - mInputField.setSelection(mInputField.length()); - if (mFocusGrabber.isFocused()) { - // Return focus to the EditText - mInputField.requestFocus(); - } +// // Move the cursor +// mInputField.setSelection(mInputField.length()); +// if (mFocusGrabber.isFocused()) { +// // Return focus to the EditText +// mInputField.requestFocus(); +// } } } 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 03aa65c..130ef10 100644 --- a/app/src/main/res/layout/content_numpad_time_picker.xml +++ b/app/src/main/res/layout/content_numpad_time_picker.xml @@ -20,11 +20,13 @@ android:layout_height="wrap_content" app:layout_gravity="center" app:layout_column="1" - android:src="@drawable/ic_done_24dp"/> + android:src="@drawable/ic_done_24dp" + android:layout_marginBottom="@dimen/numeric_keypad_fab_bottom_margin"/> 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 46c64fa..a9798ce 100644 --- a/app/src/main/res/layout/dialog_time_picker_numpad.xml +++ b/app/src/main/res/layout/dialog_time_picker_numpad.xml @@ -7,30 +7,29 @@ doesn't work. --> android:layout_width="match_parent" android:layout_height="match_parent"> - + android:textSize="@dimen/text_size_display_3" + android:fontFamily="sans-serif-light" + style="@style/TextAppearance.AppCompat"/> - + - + android:layout_marginEnd="@dimen/bottom_sheet_edge_margin"/> \ No newline at end of file diff --git a/app/src/main/res/values-v21/dimens.xml b/app/src/main/res/values-v21/dimens.xml new file mode 100644 index 0000000..ee89ac1 --- /dev/null +++ b/app/src/main/res/values-v21/dimens.xml @@ -0,0 +1,7 @@ + + + + 328dp + 56dp + 16dp + \ 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 f865cc4..5b0c81e 100644 --- a/app/src/main/res/values/dimens.xml +++ b/app/src/main/res/values/dimens.xml @@ -17,10 +17,12 @@ 17sp - 300dp - 45sp - 88dp - + 300dp + 64dp + 80dp + 344dp + 88dp + 0dp 24dp diff --git a/app/src/main/res/values/styles.xml b/app/src/main/res/values/styles.xml index 72c8bcb..6f9c550 100644 --- a/app/src/main/res/values/styles.xml +++ b/app/src/main/res/values/styles.xml @@ -35,14 +35,16 @@ Leave off the namespace to reference a custom attribute. Here, we are referencing the "grid:" namespace --> 1 - 1 + + @dimen/numeric_keypad_cell_height ?android:attr/selectableItemBackground center