Fix bug 374921: Tool view grouping no longer works in 5.1 branch

Authored by antonanikin on Jan 20 2017, 5:21 AM.



This commit fixes view grouping, broken in D3170 revision.

BUG: 373213

Test Plan

Tested on 5.1 and master branches.

Diff Detail

R33 KDevPlatform
Automatic diff as part of commit; lint not applicable.
Automatic diff as part of commit; unit tests not applicable.
antonanikin retitled this revision from to Fix bug 374921: Tool view grouping no longer works in 5.1 branch.
antonanikin updated this object.
antonanikin edited the test plan for this revision. (Show Details)
antonanikin added reviewers: KDevelop, apol.
antonanikin added a subscriber: kdevelop-devel.
mwolff accepted this revision.Jan 22 2017, 7:40 PM
mwolff added a reviewer: mwolff.
mwolff added a subscriber: mwolff.

assuming this actually fixes it, I'm all for it - less code :)

This still works b/c you pushed the grouping functionality into the showWidget code path, right? Nice work!

This revision is now accepted and ready to land.Jan 22 2017, 7:40 PM
This revision was automatically updated to reflect the committed changes.

@mwolff, Aleix reorganize code in the D3170 revision and now we can drop buttonPressed() method - all checks can be made inside showWidget(). And you are right - less code is good :)

Also sorry for mistake with bug number inside revision description.