Alexander Wels has posted comments on this change.

Change subject: webadmin: double click invokes default command
......................................................................


Patch Set 2: (2 inline comments)

....................................................
File 
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SearchableListModel.java
Line 908:     /**
Line 909:      * Subclasses with an actual edit command will override this.
Line 910:      * @return An edit {@code UICommand}.
Line 911:      */
Line 912:     public UICommand getEditCommand() {
I think these are leftovers from working through the problem, removed.
Line 913:         return null;
Line 914:     }
Line 915: 
Line 916:     @Override


Line 913:         return null;
Line 914:     }
Line 915: 
Line 916:     @Override
Line 917:     public UICommand getDefaultCommand() {
I think these are leftovers from working through the problem, removed.
Line 918:         return getEditCommand();
Line 919:     }


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I701036539547130a0c1581d179ad2f6de6519427
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Alexander Wels <aw...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Einav Cohen <eco...@redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchap...@redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to