Daniel Erez has uploaded a new change for review.

Change subject: webadmin: error label in reinit datacenter dialog
......................................................................

webadmin: error label in reinit datacenter dialog

Added an error label in 'Re-Initialize Data Center' dialog.
The following message will be displayed when relevant:
"There are no compatible Storage Domains to attach to this
Data Center. Please add new Storage from the Storage tab."

Change-Id: Iaea0b1542ba4e9fdd60c8d3997e7be06a39d4181
Signed-off-by: Daniel Erez <de...@redhat.com>
---
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/datacenter/RecoveryStorageConfirmationPopupView.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/datacenter/RecoveryStorageConfirmationPopupView.ui.xml
2 files changed, 21 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/05/15505/1

diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/datacenter/RecoveryStorageConfirmationPopupView.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/datacenter/RecoveryStorageConfirmationPopupView.java
index 39e4cbd..477a377 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/datacenter/RecoveryStorageConfirmationPopupView.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/datacenter/RecoveryStorageConfirmationPopupView.java
@@ -56,6 +56,10 @@
 
     @UiField
     @Ignore
+    Label errorLabel;
+
+    @UiField
+    @Ignore
     Label selectNewDSDLabel;
 
     @UiField(provided = true)
@@ -120,6 +124,15 @@
     }
 
     @Override
+    public void setMessage(String message) {
+        if (message != null) {
+            errorLabel.setText(message);
+            errorLabel.setVisible(true);
+            messageLabel.setVisible(false);
+        }
+    }
+
+    @Override
     public ConfirmationModel flush() {
         storageDomainItems.flush();
         return driver.flush();
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/datacenter/RecoveryStorageConfirmationPopupView.ui.xml
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/datacenter/RecoveryStorageConfirmationPopupView.ui.xml
index a2cfdbb..a0f7e98 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/datacenter/RecoveryStorageConfirmationPopupView.ui.xml
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/datacenter/RecoveryStorageConfirmationPopupView.ui.xml
@@ -15,6 +15,13 @@
                        padding-bottom: 20px;
                }
 
+               .errorLabel {
+                       padding-bottom: 20px;
+                       width: 480px;
+                       color: red;
+                       font-weight: bold;
+               }
+
                .selectNewDSDLabel {
                        padding-top: 10px;
                }
@@ -32,6 +39,7 @@
                <d:content>
                        <g:FlowPanel addStyleNames="{style.content}">
                                <g:FlowPanel addStyleNames="{style.messages}">
+                    <g:Label ui:field="errorLabel" 
addStyleNames="{style.errorLabel}" visible="false"/>
                                        <g:HTML ui:field="messageLabel" 
addStyleNames="{style.messageLabel}"/>
                                </g:FlowPanel>
                                <e:EntityModelCheckBoxEditor ui:field="latch" 
visible="false"/>


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

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

Reply via email to