low hanging fruits:

- add break after case
- center vertically the smart wakeup checkbox
This commit is contained in:
Daniele Gobbetti 2015-06-25 12:42:47 +02:00 committed by Daniele Gobbetti
parent 1caca1439a
commit 764dd70e45
2 changed files with 26 additions and 20 deletions

View File

@ -268,7 +268,7 @@ public class BluetoothCommunicationService extends Service {
break; break;
case ACTION_SET_ALARMS: case ACTION_SET_ALARMS:
mDeviceSupport.onSetAlarms(); mDeviceSupport.onSetAlarms();
break;
} }
return START_STICKY; return START_STICKY;

View File

@ -1,5 +1,6 @@
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android" <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:tools="http://schemas.android.com/tools" android:layout_width="match_parent" xmlns:tools="http://schemas.android.com/tools" android:layout_width="match_parent"
android:orientation="vertical"
android:layout_height="match_parent" android:paddingLeft="@dimen/activity_horizontal_margin" android:layout_height="match_parent" android:paddingLeft="@dimen/activity_horizontal_margin"
android:paddingRight="@dimen/activity_horizontal_margin" android:paddingRight="@dimen/activity_horizontal_margin"
android:paddingTop="@dimen/activity_vertical_margin" android:paddingTop="@dimen/activity_vertical_margin"
@ -7,23 +8,29 @@
tools:context="nodomain.freeyourgadget.gadgetbridge.activities.AlarmDetails"> tools:context="nodomain.freeyourgadget.gadgetbridge.activities.AlarmDetails">
<TimePicker <LinearLayout
android:layout_width="wrap_content" android:orientation="horizontal"
android:layout_height="wrap_content" android:layout_width="fill_parent"
android:id="@+id/alarm_time_picker" android:layout_height="wrap_content">
android:layout_alignParentTop="true"
android:layout_alignParentStart="true" />
<CheckedTextView <TimePicker
android:layout_width="wrap_content" android:layout_width="wrap_content"
android:layout_height="fill_parent" android:layout_height="wrap_content"
android:text="@string/alarm_smart_wakeup" android:id="@+id/alarm_time_picker"
android:orientation="vertical" android:layout_alignParentTop="true"
android:id="@+id/alarm_ctv_smart_wakeup" android:layout_alignParentStart="true" />
android:checked="false"
android:drawableTop="@drawable/abc_btn_check_material" <CheckedTextView
android:layout_above="@+id/dowSelector" android:layout_width="wrap_content"
android:layout_toEndOf="@+id/alarm_time_picker" /> android:layout_height="wrap_content"
android:text="@string/alarm_smart_wakeup"
android:orientation="vertical"
android:id="@+id/alarm_ctv_smart_wakeup"
android:checked="false"
android:drawableTop="@drawable/abc_btn_check_material"
android:layout_toEndOf="@+id/alarm_time_picker"
android:layout_gravity="center_vertical" />
</LinearLayout>
<LinearLayout <LinearLayout
android:orientation="horizontal" android:orientation="horizontal"
@ -119,5 +126,4 @@
</LinearLayout> </LinearLayout>
</LinearLayout>
</RelativeLayout>