Change in ovirt-engine[ovirt-engine-4.0]: webadmin: Fix dynamic tab reveal logic

oourfali at redhat.com oourfali at redhat.com
Thu Jun 16 05:34:46 UTC 2016


Oved Ourfali has submitted this change and it was merged.

Change subject: webadmin: Fix dynamic tab reveal logic
......................................................................


webadmin: Fix dynamic tab reveal logic

This patch fixes the issue where revealing custom sub tab
(that was added by UI plugin) makes UI jump back to "VMs"
main tab, instead of letting the sub tab reveal as expected.

Change-Id: I4d92f3688cf1662347e74c56e50935c2a531ac33
Bug-Url: https://bugzilla.redhat.com/1347017
Signed-off-by: Vojtech Szocs <vszocs at redhat.com>
---
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/presenter/DynamicUrlContentTabPresenter.java
1 file changed, 4 insertions(+), 1 deletion(-)

Approvals:
  Alexander Wels: Verified
  Jenkins CI: Passed CI tests
  Oved Ourfali: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I4d92f3688cf1662347e74c56e50935c2a531ac33
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list