Change in ovirt-engine[master]: webadmin: Synched Cpu Pinning label and textbox

[tnisan at redhat.com]

Tal Nisan has submitted this change and it was merged.

Change subject: webadmin: Synched Cpu Pinning label and textbox
......................................................................


webadmin: Synched Cpu Pinning label and textbox

"Cpu Pinning Topology" label wasn't in sync with the availability of
it's textbox. It was constantly greyed out(disabled) even when the
textbox was enabled.

This patch introduced synching with between the textbox and the label.

Due to the blue tooltip between the label and the textbox it's impossible
to use the combined label and textbox object. So, This patch is using
listeners inorder to overcome this problem.

Change-Id: I007814512506170699050a4cb82abfea2278c4d4
Bug-Url: https://bugzilla.redhat.com/1226767
Signed-off-by: Tomer Saban <tsaban at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.java
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/popup/AbstractVmPopupWidget.ui.xml
2 files changed, 30 insertions(+), 3 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved; Passed CI tests
  Tomer Saban: Verified

Objections:
  Jenkins CI: Failed CI tests



-- 
To view, visit https://gerrit.ovirt.org/56919
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I007814512506170699050a4cb82abfea2278c4d4
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>