diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogFactory.java b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogFactory.java index 8117b4f3ee6..2e1590a23bb 100644 --- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogFactory.java +++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogFactory.java @@ -3,6 +3,7 @@ package com.sap.sailing.gwt.home.shared.partials.dialog; import com.google.gwt.event.dom.client.ClickEvent; import com.google.gwt.user.client.Event; import com.google.gwt.user.client.ui.PopupPanel; +import com.sap.sailing.gwt.ui.client.StringMessages; import com.sap.sse.gwt.client.dialog.DataEntryDialog.DialogCallback; /** factory class for confirm dialog with yes/no button */ @@ -19,6 +20,8 @@ public final class ConfirmDialogFactory { ConfirmDialogPanel confirmDialogPanel = new ConfirmDialogPanel(message, title, callback, dialog); dialog.setWidget(confirmDialogPanel); confirmDialogPanel.addStyleName(DialogResources.INSTANCE.css().dialog()); + confirmDialogPanel.setButtonLabels(StringMessages.INSTANCE.yes(), StringMessages.INSTANCE.no()); + confirmDialogPanel.setFirstButtonDestructive(); dialog.addStyleName(DialogResources.INSTANCE.css().backgroundPanel()); dialog.show(); diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogPanel.java b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogPanel.java index 3c87e67a7eb..a4d67c67f04 100644 --- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogPanel.java +++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogPanel.java @@ -24,10 +24,10 @@ public class ConfirmDialogPanel extends Composite { } @UiField - Button yesButton; + Button firstButton; @UiField - Button noButton; + Button secondButton; @UiField DivElement messageField; @@ -51,7 +51,16 @@ public class ConfirmDialogPanel extends Composite { titleField.setInnerText(title); } - @UiHandler("yesButton") + public void setButtonLabels(String firstButtonText, String secondButtonText) { + firstButton.setText(firstButtonText); + secondButton.setText(secondButtonText); + } + + public void setFirstButtonDestructive() { + firstButton.addStyleName(DialogResources.INSTANCE.css().yesButton()); + } + + @UiHandler("firstButton") void onYesClick(ClickEvent e) { if (parent != null) { parent.hide(); @@ -59,7 +68,7 @@ public class ConfirmDialogPanel extends Composite { } } - @UiHandler("noButton") + @UiHandler("secondButton") void onNoClick(ClickEvent e) { if (parent != null) { parent.hide(); diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogPanel.ui.xml b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogPanel.ui.xml index 1889cf72145..1dc9627fa2b 100644 --- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogPanel.ui.xml +++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/ConfirmDialogPanel.ui.xml @@ -1,13 +1,19 @@ +
- Yes - No + + +
\ No newline at end of file diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/Dialog.gss b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/Dialog.gss index 4bd99cc2597..bc0956c205b 100644 --- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/Dialog.gss +++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/Dialog.gss @@ -14,22 +14,28 @@ box-shadow: 0.1em 0.1em 1em #bbb; } -@media (max-width: 400px) { - .dialog { - width: 90%; - } +.buttonAdjustments { + margin-left: .5em; + margin-top: .5em; + min-width: 11em; } -@media (min-width: 400px) { - .dialog { - width: 50%; - } +@media ( max-width: 400px) { + .dialog { + width: 90%; + } } -@media (min-width: 600px) { - .dialog { - width: 40%; - } +@media ( min-width: 400px) { + .dialog { + width: 50%; + } +} + +@media ( min-width: 600px) { + .dialog { + width: 40%; + } } .message { diff --git a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/DialogResources.java b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/DialogResources.java index a6909bfcbc1..9b4ddee2e10 100644 --- a/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/DialogResources.java +++ b/java/com.sap.sailing.gwt.ui/src/main/java/com/sap/sailing/gwt/home/shared/partials/dialog/DialogResources.java @@ -28,5 +28,7 @@ public interface DialogResources extends ClientBundle { String backgroundPanel(); String title(); + + String buttonAdjustments(); } }