Arik Hadas has uploaded a new change for review. Change subject: webadmin: change layout of table of vms to be imported ......................................................................
webadmin: change layout of table of vms to be imported Put the VM name as the first column since putting the collapse snapshots as the first column is confusing because it contains checkboxes so it looks it indicates the selected VMs. Change-Id: I0dea6b6b525f4f66697c9bfb76c90437dc06b160 Signed-off-by: Arik Hadas <[email protected]> --- M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/backup/ImportVmFromExportDomainPopupView.java 1 file changed, 8 insertions(+), 8 deletions(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/80/37880/1 diff --git a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/backup/ImportVmFromExportDomainPopupView.java b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/backup/ImportVmFromExportDomainPopupView.java index ca0d1f9..e22bbb3 100644 --- a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/backup/ImportVmFromExportDomainPopupView.java +++ b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/storage/backup/ImportVmFromExportDomainPopupView.java @@ -238,6 +238,14 @@ protected void initMainTable() { this.table = new ListModelObjectCellTable<Object, ImportVmFromExportDomainModel>(); + AbstractTextColumnWithTooltip<Object> nameColumn = new AbstractTextColumnWithTooltip<Object>() { + @Override + public String getValue(Object object) { + return ((ImportVmData) object).getVm().getName(); + } + }; + table.addColumn(nameColumn, constants.nameVm(), "150px"); //$NON-NLS-1$ + AbstractCheckboxColumn<Object> collapseSnapshotsColumn = new AbstractCheckboxColumn<Object>(new FieldUpdater<Object, Boolean>() { @Override @@ -287,14 +295,6 @@ } }; table.addColumn(cloneVMColumn, constants.cloneVM(), "50px"); //$NON-NLS-1$ - - AbstractTextColumnWithTooltip<Object> nameColumn = new AbstractTextColumnWithTooltip<Object>() { - @Override - public String getValue(Object object) { - return ((ImportVmData) object).getVm().getName(); - } - }; - table.addColumn(nameColumn, constants.nameVm(), "150px"); //$NON-NLS-1$ AbstractTextColumnWithTooltip<Object> originColumn = new AbstractEnumColumn<Object, OriginType>() { @Override -- To view, visit http://gerrit.ovirt.org/37880 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I0dea6b6b525f4f66697c9bfb76c90437dc06b160 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
