Change in ovirt-engine[ovirt-engine-3.3]: webadmin: TypeAheadListBox to not override acceptable null

iheim at redhat.com iheim at redhat.com
Wed Dec 18 07:24:40 UTC 2013


Itamar Heim has submitted this change and it was merged.

Change subject: webadmin: TypeAheadListBox to not override acceptable null
......................................................................


webadmin: TypeAheadListBox to not override acceptable null

Altered ListModelTypeAheadListBox to only visually override null
values if they're not allowed, so that when clicking outside of the
list box it won't seem like another choice was chosen (when in reality
the backing value is still null).

Change-Id: I35b34097c9068f2baaa0573c74ec6c930b915f71
Bug-Url: https://bugzilla.redhat.com/1042872
Signed-off-by: Lior Vernia <lvernia at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/editor/ListModelTypeAheadListBox.java
1 file changed, 6 insertions(+), 2 deletions(-)

Approvals:
  Itamar Heim: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I35b34097c9068f2baaa0573c74ec6c930b915f71
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.3
Gerrit-Owner: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Itamar Heim <iheim at redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list