Created attachment 1068639 [details]
UI with tabs
Description of problem:
The tabs are listed thrice once the gluster-monitoring-uiplugin is installed. Attached is the screenshot
Version-Release number of selected component (if applicable):
Target release should be placed once a package build is known to fix a issue. Since this bug is not modified, the target version has been reset. Please use target milestone to plan a fix for a oVirt release.
this is an automated message. oVirt 3.6.0 RC3 has been released and GA is targeted to next week, Nov 4th 2015.
Please review this bug and if not a blocker, please postpone to a later release.
All bugs not postponed on GA release will be automatically re-targeted to
- 3.6.1 if severity >= high
- 4.0 if severity < high
oVirt 3.6.0 has been released on November 4th, re-targeting to 4.0 since this bug has been marked with severity < high
This bug is flagged for 3.6, yet the milestone is for 4.0 version, therefore the milestone has been reset.
Please set the correct milestone or add the flag.
Seems like some thing wrong with the way how we add new tabs through UI Plugin. When ever a new tab is added using UI Plugin, all the entries in the drop down is repeated once. So if we add 3 tabs then same list will be repeated 3 times ( excluding the default one time).
RedrawDynamicTabContainerEvent is fired whenever a new tab is added. We are removing all the existing tabs as part of this event and redrawing the same. But it fails to remove the entries from dropdown list.
ScrollableTabBarView.removeTabWidget() tries to remove the widget from dropdownPandel by calling dropdownPanel.remove(listWidget). But it fails to remove and returns false.
May be Vojtech can help he to fine the root cause here.
This bug is not marked for z-stream, yet the milestone is for a z-stream version, therefore the milestone has been reset.
Please set the correct milestone or add the z-stream flag.
Seems like UI infra issue.
I'll write sample UI plugin that adds 3 main tabs to reproduce and try to fix this.
(In reply to Ramesh N from comment #8)
> ScrollableTabBarView.removeTabWidget() tries to remove the widget from
> dropdownPandel by calling dropdownPanel.remove(listWidget). But it fails to
> remove and returns false.
Exactly! We're trying to remove wrong "listWidget" :-) I'm going to fix that.
Created attachment 1106138 [details]
Test UI plugin that adds 3 new main tabs
Attached test UI plugin (see comment #12) to ease verification.
oVirt 3.6.2 RC1 has been released for testing, moving to ON_QA
Tested with RHEV 184.108.40.206 with RHGS 3.1.2 RC. After adding UI plugin, tabs are listed only once