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(a)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(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Roy Golan <rgolan(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: Tomer Saban <tsaban(a)redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation(a)ovirt.org>