Change in ovirt-engine[master]: webadmin: Enable tooltip on greyed out LUNs

derez at redhat.com derez at redhat.com
Mon Apr 25 13:58:24 UTC 2016


Daniel Erez has submitted this change and it was merged.

Change subject: webadmin: Enable tooltip on greyed out LUNs
......................................................................


webadmin: Enable tooltip on greyed out LUNs

When editing an ISCSI/FC storage domain, if a LUN is
already in use it will be greyed out.
If the user hovers over the LUN a tooltip appears
with the reason why it is disabled.

In this patch, the tooltip is added to the whole row,
not only the checkbox cell.
It will always appear on the left of the row.

Bug-Url: https://bugzilla.redhat.com/1329906

Change-Id: I4ea714768c4e427cf82a8b062fd49dcaf127335b
Signed-off-by: Fred Rolland <frolland at redhat.com>
Signed-off-by: Daniel Erez <derez at redhat.com>
---
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/uicommon/storage/AbstractSanStorageList.java
1 file changed, 4 insertions(+), 4 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, but someone else must approve
  Daniel Erez: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Freddy Rolland: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4ea714768c4e427cf82a8b062fd49dcaf127335b
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Freddy Rolland <frolland at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list