Change in ovirt-engine[master]: frontend: mouse selection of ListModelTypeAheadListBox

tjelinek at redhat.com tjelinek at redhat.com
Thu Aug 8 08:40:13 UTC 2013


Tomas Jelinek has submitted this change and it was merged.

Change subject: frontend: mouse selection of ListModelTypeAheadListBox
......................................................................


frontend: mouse selection of ListModelTypeAheadListBox

If you are not using the ui binder and select the value
on the ListModelTypeAheadListBox using mouse, the
ListModelTypeAheadListBox.asEditor().getSubEditor().getValue();
does not return the selected value.

The problem was that the mouse selection fired the ValueChangeEvent
but no one was listening on it.

It worked with the editor driver because UiCommonEditorVisitor
listens to this event.

Fixed by listening to this event also in the ListModelTypeAheadListBox.

Change-Id: I5edee9d0b0e64dc4f7e8609aa5fde4c69e133924
Signed-off-by: Tomas Jelinek <tjelinek at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/ListModelTypeAheadListBox.java
1 file changed, 8 insertions(+), 0 deletions(-)

Approvals:
  Tomas Jelinek: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I5edee9d0b0e64dc4f7e8609aa5fde4c69e133924
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>



More information about the Engine-commits mailing list