Add mute button to error dialog.
So that users needn't hastily close the dialog to stop the sound
and then not being able to read the actual message.
(cherry picked from commit 43755f0
)
This commit is contained in:
parent
7fa51a9248
commit
1985ed379d
3 changed files with 29 additions and 6 deletions
|
@ -19,6 +19,7 @@ import info.nightscout.androidaps.Services.AlarmSoundService;
|
|||
|
||||
public class ErrorDialog extends DialogFragment implements View.OnClickListener {
|
||||
private static Logger log = LoggerFactory.getLogger(ErrorDialog.class);
|
||||
Button muteButton;
|
||||
Button okButton;
|
||||
TextView statusView;
|
||||
ErrorHelperActivity helperActivity;
|
||||
|
@ -52,14 +53,14 @@ public class ErrorDialog extends DialogFragment implements View.OnClickListener
|
|||
Bundle savedInstanceState) {
|
||||
getDialog().setTitle(title);
|
||||
View view = inflater.inflate(R.layout.overview_error_dialog, container, false);
|
||||
muteButton = (Button) view.findViewById(R.id.overview_error_mute);
|
||||
okButton = (Button) view.findViewById(R.id.overview_error_ok);
|
||||
statusView = (TextView) view.findViewById(R.id.overview_error_status);
|
||||
muteButton.setOnClickListener(this);
|
||||
okButton.setOnClickListener(this);
|
||||
setCancelable(false);
|
||||
|
||||
Intent alarm = new Intent(MainApp.instance().getApplicationContext(), AlarmSoundService.class);
|
||||
alarm.putExtra("soundid", soundId);
|
||||
MainApp.instance().startService(alarm);
|
||||
startAlarm();
|
||||
return view;
|
||||
}
|
||||
|
||||
|
@ -77,13 +78,16 @@ public class ErrorDialog extends DialogFragment implements View.OnClickListener
|
|||
if (helperActivity != null) {
|
||||
helperActivity.finish();
|
||||
}
|
||||
Intent alarm = new Intent(MainApp.instance().getApplicationContext(), AlarmSoundService.class);
|
||||
MainApp.instance().stopService(alarm);
|
||||
stopAlarm();
|
||||
}
|
||||
|
||||
@Override
|
||||
public void onClick(View view) {
|
||||
switch (view.getId()) {
|
||||
case R.id.overview_error_mute:
|
||||
log.debug("Error dialog mute button pressed");
|
||||
stopAlarm();
|
||||
break;
|
||||
case R.id.overview_error_ok:
|
||||
log.debug("Error dialog ok button pressed");
|
||||
dismiss();
|
||||
|
@ -91,4 +95,14 @@ public class ErrorDialog extends DialogFragment implements View.OnClickListener
|
|||
}
|
||||
}
|
||||
|
||||
private void startAlarm() {
|
||||
Intent alarm = new Intent(MainApp.instance().getApplicationContext(), AlarmSoundService.class);
|
||||
alarm.putExtra("soundid", soundId);
|
||||
MainApp.instance().startService(alarm);
|
||||
}
|
||||
|
||||
private void stopAlarm() {
|
||||
Intent alarm = new Intent(MainApp.instance().getApplicationContext(), AlarmSoundService.class);
|
||||
MainApp.instance().stopService(alarm);
|
||||
}
|
||||
}
|
||||
|
|
|
@ -16,11 +16,19 @@
|
|||
android:layout_gravity="center_horizontal" />
|
||||
|
||||
<Button
|
||||
android:id="@+id/overview_error_ok"
|
||||
android:id="@+id/overview_error_mute"
|
||||
android:layout_width="wrap_content"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_gravity="center_horizontal"
|
||||
android:layout_marginTop="15dp"
|
||||
android:text="@string/mute" />
|
||||
|
||||
<Button
|
||||
android:id="@+id/overview_error_ok"
|
||||
android:layout_width="wrap_content"
|
||||
android:layout_height="wrap_content"
|
||||
android:layout_gravity="center_horizontal"
|
||||
android:layout_marginTop="5dp"
|
||||
android:text="@string/ok" />
|
||||
|
||||
</LinearLayout>
|
||||
|
|
|
@ -875,5 +875,6 @@
|
|||
<string name="combo_error_failure_reading_changed_basal_rate">Basal rate changed on pump, but reading it failed</string>
|
||||
<string name="combo_activity_checking_for_history_changes">Checking for history changes</string>
|
||||
<string name="combo_error_multiple_boluses_with_identical_timestamp">Multiple boluses with the same amount within the same minute were just imported. Only one record could be added to treatments. Please check the pump and manually add a bolus record using the Careportal tab. Make sure to create a bolus with a time no other bolus uses.</string>
|
||||
<string name="mute">Mute</string>
|
||||
</resources>
|
||||
|
||||
|
|
Loading…
Reference in a new issue