Commit 70c6350f authored by Thomas Lübking's avatar Thomas Lübking
Browse files

Fix regression induced (but likely not caused) by commit e91e5fed

Commiting on behalf of Jaime Torres.

Ratio:
On k-c-d I suggested this would be another ouotcome of bug #213068 and Jaime could confirm this and the
scrollbar policy workaround to work.
He suggested to fix it by "setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff);" instead (since presumingly the
pluginselector is always top-down oriented)
I tested that variant as well and could not reproduce the issue.

Since Jaime has no acces to his development environment for today he asked me to either revert his commit or
apply the hotfix working around bug #213068 as mentioned above.

Because i could not spot any bug in his commit i'll go with the hotfix to prevent a regression.
Resolving the actual recursion bug is an outstanding issue.

CCBUG: 213068
CCBUG: 287847
parent df90e1f8
......@@ -258,6 +258,7 @@ KPluginSelector::KPluginSelector(QWidget *parent)
d->listView = new KCategorizedView(this);
d->listView->setVerticalScrollMode(QListView::ScrollPerPixel);
d->listView->setVerticalScrollBarPolicy(Qt::ScrollBarAlwaysOn); // bug 213068
d->listView->setHorizontalScrollBarPolicy(Qt::ScrollBarAlwaysOff); // bug 213068
d->listView->setAlternatingRowColors(true);
d->listView->setCategoryDrawer(d->categoryDrawer);
d->dependenciesWidget = new Private::DependenciesWidget(this);
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment