Change in ovirt-engine[master]: webadmin, userportal: change suggest box event handling

tjelinek at redhat.com tjelinek at redhat.com
Wed Jul 31 11:20:41 UTC 2013


Tomas Jelinek has submitted this change and it was merged.

Change subject: webadmin,userportal: change suggest box event handling
......................................................................


webadmin,userportal: change suggest box event handling

The problem was that the ListModelTypeAheadListBox could
be closed only by either selecting one of the values or clicking
somewhere outside of it, but not by clicking inside it again
or clicking to the drop down triangle.

Fixed by disabling the auto-hide feature of the suggest box
and handling this events by hand.

The SuggestBoxFocusHandler now contains a field called "enabled"
which enables-disables the handling by this handler. It is set
from FocusHandlerEnablingMouseHandlers. The order of the events is:
onMouseDown
onBlur
onMouseUp
makeing the onBlur bordered between the mouse events. The reason is
that the blur handler is supposed to hide the suggestions only if
the other ways are not used (e.g.: clicking on the triangle or selecting a
suggestion by clicking on some suggestion).

One other fix is that the EnterIgnoringNativePreviewHandler has been
fixed to process the enter event only once.

Change-Id: Ib73fa0160ea1a80246bb8e9cc9a6052fad63f1fe
Bug-Url: https://bugzilla.redhat.com/987845
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/BaseListModelSuggestBox.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/ListModelTypeAheadListBox.java
2 files changed, 132 insertions(+), 36 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib73fa0160ea1a80246bb8e9cc9a6052fad63f1fe
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Gilad Chaplik <gchaplik at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list