Shubhendu Tripathi has uploaded a new change for review.

Change subject: webadmin: Disabled selection of table rows
......................................................................

webadmin: Disabled selection of table rows

Earlier the gluster volume sanpshot config listing table
supported selection of rows, and it used to cause issues
as the editable column values used to become invisible.
Disabled the row selection for the table so that user
can only change the values in the text boxes.

Change-Id: I27ab7ee81745d468d0423294cc06b9183406c103
Signed-off-by: Shubhendu Tripathi <[email protected]>
---
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterClusterSnapshotConfigureOptionsPopupView.java
M 
frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterVolumeSnapshotConfigureOptionsPopupView.java
2 files changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/10/40010/1

diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterClusterSnapshotConfigureOptionsPopupView.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterClusterSnapshotConfigureOptionsPopupView.java
index 073c7b5..e7c3c0f 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterClusterSnapshotConfigureOptionsPopupView.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterClusterSnapshotConfigureOptionsPopupView.java
@@ -26,6 +26,7 @@
 import com.google.gwt.uibinder.client.UiField;
 import com.google.gwt.user.cellview.client.Column;
 import com.google.gwt.user.client.ui.Label;
+import com.google.gwt.view.client.NoSelectionModel;
 import com.google.inject.Inject;
 
 public class GlusterClusterSnapshotConfigureOptionsPopupView extends 
AbstractModelBoundPopupView<GlusterClusterSnapshotConfigModel> implements 
GlusterClusterSnapshotConfigureOptionsPopupPresenterWidget.ViewDef {
@@ -82,6 +83,7 @@
 
         configsTable =
                 new 
EntityModelCellTable<ListModel<EntityModel<GlusterVolumeSnapshotConfig>>>(false,
 true);
+        configsTable.setSelectionModel(new NoSelectionModel());
 
         configsTable.addColumn(new 
EntityModelTextColumn<GlusterVolumeSnapshotConfig>() {
             @Override
diff --git 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterVolumeSnapshotConfigureOptionsPopupView.java
 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterVolumeSnapshotConfigureOptionsPopupView.java
index 0a894bb..d82f51f 100644
--- 
a/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterVolumeSnapshotConfigureOptionsPopupView.java
+++ 
b/frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/gluster/GlusterVolumeSnapshotConfigureOptionsPopupView.java
@@ -24,6 +24,7 @@
 import com.google.gwt.uibinder.client.UiField;
 import com.google.gwt.user.cellview.client.Column;
 import com.google.gwt.user.client.ui.Label;
+import com.google.gwt.view.client.NoSelectionModel;
 import com.google.inject.Inject;
 
 public class GlusterVolumeSnapshotConfigureOptionsPopupView extends 
AbstractModelBoundPopupView<GlusterVolumeSnapshotConfigModel> implements 
GlusterVolumeSnapshotConfigureOptionsPopupPresenterWidget.ViewDef {
@@ -78,6 +79,7 @@
     private void initEditors() {
         configsTable =
                 new 
EntityModelCellTable<ListModel<EntityModel<VolumeSnapshotOptionModel>>>(false, 
true);
+        configsTable.setSelectionModel(new NoSelectionModel());
 
         configsTable.addColumn(new 
EntityModelTextColumn<VolumeSnapshotOptionModel>() {
             @Override


-- 
To view, visit https://gerrit.ovirt.org/40010
To unsubscribe, visit https://gerrit.ovirt.org/settings

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

Reply via email to