Change in ovirt-engine[master]: webadmin: removed incorrect value change handler

tjelinek at redhat.com tjelinek at redhat.com
Thu Feb 5 11:19:43 UTC 2015


Tomas Jelinek has submitted this change and it was merged.

Change subject: webadmin: removed incorrect value change handler
......................................................................


webadmin: removed incorrect value change handler

The same handler has been registerd both on the ListModelTypeAheadListBox and
on BaseListModelSuggestBox (using the addValueChangeHandler).

The deleted one had no meaning - it's logic is handled by the
SuggestBoxFocusHandler but in some cases kicked in sooner than the one in
BaseListModelSuggestBox hiding the suggestions sooner than the correct handler
was able to use them.

Change-Id: I203023384e0d29671220d9ebd2ed7c4fd4b67da3
Bug-Url: https://bugzilla.redhat.com/1166022
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, 0 insertions(+), 11 deletions(-)

Approvals:
  Tomas Jelinek: Verified
  Vojtech Szocs: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I203023384e0d29671220d9ebd2ed7c4fd4b67da3
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list