Change in ovirt-engine[master]: webadmin: Changed editor prohibition reason mechanism

alkaplan at redhat.com alkaplan at redhat.com
Tue Jul 9 09:40:26 UTC 2013


Alona Kaplan has submitted this change and it was merged.

Change subject: webadmin: Changed editor prohibition reason mechanism
......................................................................


webadmin: Changed editor prohibition reason mechanism

The tooltip displayed over disabled editor widgets (change prohibition
reason) is now updated upon call to setChangeProhibitionReason() as
well as to setIsChangeable(). This means that the correct tooltip will
be shown even if setChangeProhibitionReason() is called after
setIsChangeable(false), which hasn't been the case up 'til now.

Change-Id: Ifb76b2348922bfebf62c54da4ee2a8ca45ebef31
Signed-off-by: Lior Vernia <lvernia at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/editor/UiCommonEditorVisitor.java
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/Model.java
2 files changed, 17 insertions(+), 2 deletions(-)

Approvals:
  Alona Kaplan: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ifb76b2348922bfebf62c54da4ee2a8ca45ebef31
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: Alona Kaplan <alkaplan at redhat.com>
Gerrit-Reviewer: Lior Vernia <lvernia at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list