Lior Vernia has uploaded a new change for review.

Change subject: webadmin: Sort checkbox columns in Network/Clusters subtab
......................................................................

webadmin: Sort checkbox columns in Network/Clusters subtab

Implemented sorting using the recently added default sorting behavior
in CheckboxColumn.

Change-Id: Ic1ea118ded65ce1ebe426a7bfbf2a7cc993380b0
Bug-Url: https://bugzilla.redhat.com/1120054
Signed-off-by: Lior Vernia <[email protected]>
---
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/network/SubTabNetworkClusterView.java
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/55/30155/1

diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/network/SubTabNetworkClusterView.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/network/SubTabNetworkClusterView.java
index e42d672..2e58a06 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/network/SubTabNetworkClusterView.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/tab/network/SubTabNetworkClusterView.java
@@ -88,7 +88,7 @@
                 return false;
             }
         };
-
+        attachedColumn.makeSortable();
         getTable().addColumn(attachedColumn, 
constants.attachedNetworkCluster(), "120px"); //$NON-NLS-1$
 
         getTable().addColumn(new NetworkClusterStatusColumn(), 
constants.networkStatus(), "120px"); //$NON-NLS-1$
@@ -108,6 +108,7 @@
                 return false;
             }
         };
+        netRequiredColumn.makeSortable();
         getTable().addColumn(netRequiredColumn, 
constants.requiredNetCluster(), "120px"); //$NON-NLS-1$
 
         SafeHtmlWithSafeHtmlTooltipColumn<PairQueryable<VDSGroup, 
NetworkCluster>> netRoleColumn =


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic1ea118ded65ce1ebe426a7bfbf2a7cc993380b0
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Lior Vernia <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to