Arik Hadas has uploaded a new change for review.

Change subject: webadmin: fix layout of multiple lines conf msgs
......................................................................

webadmin: fix layout of multiple lines conf msgs

Fix the layout of messages in some confirmations dialogs that should be
splitted over multiple lines (such as in the "import vm" confirmation
dialog).

The problem was that the widget which is used to contain those messages
is Label, thus the '\n' characters in the message are ignored, and the
message is presented in a single long line.

This patch solves the problem by replacing the '\n' characters in the
message with "<br/>" tags and setting the message as HTML, as done in
RemoveConfirmationPopupView#setNote.

Change-Id: I1576b7576cf93ba9e525f20261d423fd4c0721b0
Signed-off-by: Arik Hadas <aha...@redhat.com>
---
M 
frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/AbstractConfirmationPopupView.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/89/9989/1

diff --git 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/AbstractConfirmationPopupView.java
 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/AbstractConfirmationPopupView.java
index 6faec2e..458ba94 100644
--- 
a/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/AbstractConfirmationPopupView.java
+++ 
b/frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/view/popup/AbstractConfirmationPopupView.java
@@ -19,7 +19,7 @@
 
     @Override
     public void setMessage(String message) {
-        messageLabel.setText(message);
+        messageLabel.getElement().setInnerHTML(message != null ? 
message.replace("\n", "<br/>") : "");//$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
     }
 
 }


--
To view, visit http://gerrit.ovirt.org/9989
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I1576b7576cf93ba9e525f20261d423fd4c0721b0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Arik Hadas <aha...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to