anmolbabu has posted comments on this change.

Change subject: engine : Rebalance status to show Started and Stopped At fields
......................................................................


Patch Set 8:

(6 comments)

....................................................
File 
frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/UIConstants.java
Line 3: 
Line 4: import org.ovirt.engine.core.common.businessentities.VmPool;
Line 5: 
Line 6: import com.google.gwt.i18n.client.Constants.DefaultStringValue;
Line 7: 
Done
Line 8: public interface UIConstants extends 
com.google.gwt.i18n.client.Constants {
Line 9: 
Line 10:     @DefaultStringValue("OK")
Line 11:     String ok();


....................................................
File 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/ApplicationConstants.java
Line 1: package org.ovirt.engine.ui.webadmin;
Line 2: 
Line 3: import org.ovirt.engine.ui.common.CommonApplicationConstants;
Line 4: 
Line 5: import com.google.gwt.i18n.client.Constants.DefaultStringValue;
Done
Line 6: 
Line 7: public interface ApplicationConstants extends 
CommonApplicationConstants {
Line 8: 
Line 9:     @DefaultStringValue("oVirt Engine Web Administration")


....................................................
File 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/VolumeRebalanceStatusPopupView.java
Line 79: 
Line 80:     @UiField
Line 81:     @Ignore
Line 82:     @WithElementId
Line 83:     VerticalPanel vPanel2;
Done
Line 84: 
Line 85:     @UiField
Line 86:     @Ignore
Line 87:     @WithElementId


Line 94: 
Line 95:     @UiField
Line 96:     @Ignore
Line 97:     @WithElementId
Line 98:     SimpleDialogPanel sdPanel;
Done
Line 99: 
Line 100:     @UiField
Line 101:     @Ignore
Line 102:     Label messageLabel;


Line 101:     @Ignore
Line 102:     Label messageLabel;
Line 103: 
Line 104:     @Path("stopTime.entity")
Line 105:     EntityModelLabelEditor stopTime;
No,it is a dynamic field added only when rebalance is aborted
Line 106: 
Line 107:     ApplicationMessages messages;
Line 108: 
Line 109:     ApplicationConstants constants;


Line 223:                     
completedStatus.setVisible(object.isStatusAvailable());
Line 224:                     status.add(completedStatus);
Line 225:                 } else 
if(e.PropertyName.equals("IS_STOP_TIME_APPLICABLE")) {//$NON-NLS-1$
Line 226:                     sdPanel.setSize("1000px", 
"550px");//$NON-NLS-1$//$NON-NLS-2$
Line 227:                     sPanel.setSize("981px", 
"415px");//$NON-NLS-1$//$NON-NLS-2$
Now,there will 3 columns instead of just 2.And if its not resized, the fields 
allignment goes off.Instead of coming in the same line,the field heading comes 
in one line and its value comes in the next..
Line 228: 
Line 229:                     vPanel2.remove(clusterEditor);
Line 230:                     vPanel2.remove(statusTimeEditor);
Line 231: 


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2957ba2c4bd1f6deb1ef444e28e8afdd8896f0b4
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: anmolbabu <anb...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Kanagaraj M <kmayi...@redhat.com>
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Ramesh N <rnach...@redhat.com>
Gerrit-Reviewer: Sahina Bose <sab...@redhat.com>
Gerrit-Reviewer: Shubhendu Tripathi <shtri...@redhat.com>
Gerrit-Reviewer: anmolbabu <anb...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to