anmolbabu has posted comments on this change.

Change subject: webadmin: show storage devices in host
......................................................................


Patch Set 7:

(1 comment)

http://gerrit.ovirt.org/#/c/35315/7/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/HostStorageDevicesListModel.java
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/HostStorageDevicesListModel.java:

Line 13: import org.ovirt.engine.ui.uicommonweb.models.SearchableListModel;
Line 14: import org.ovirt.engine.ui.uicompat.ConstantsManager;
Line 15: import org.ovirt.engine.ui.uicompat.PropertyChangedEventArgs;
Line 16: 
Line 17: public class HostStorageDevicesListModel extends SearchableListModel {
Please override onSelectedItemChanged() and selectedItemsChanged() to overcome 
the bug which makes a selection in the sub tab appear as if multiple rows are 
selected.
Line 18: 
Line 19:     @Override
Line 20:     public VDS getEntity() {
Line 21:         return (VDS) super.getEntity();


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I10da2f35012795de32367f295260ce9151c44909
Gerrit-PatchSet: 7
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ramesh N <rnach...@redhat.com>
Gerrit-Reviewer: anmolbabu <anb...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
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