Unverified Commit e737554c authored by Yichao Yu's avatar Yichao Yu
Browse files

Set visibility after setting parent

This avoids recreating native window, which should be generally good,
and works around QTBUG-87345.

BUG: 427569
parent d014f692
......@@ -148,7 +148,6 @@ struct ViewMainPagePrivate
mThumbnailBar = new ThumbnailBarView;
ThumbnailBarItemDelegate* delegate = new ThumbnailBarItemDelegate(mThumbnailBar);
mThumbnailBar->setItemDelegate(delegate);
mThumbnailBar->setVisible(GwenviewConfig::thumbnailBarIsVisible());
mThumbnailBar->setSelectionMode(QAbstractItemView::ExtendedSelection);
}
......@@ -330,6 +329,9 @@ struct ViewMainPagePrivate
mThumbnailSplitter->addWidget(mAdapterContainer);
mThumbnailSplitter->addWidget(mThumbnailBar);
mThumbnailSplitter->setSizes(GwenviewConfig::thumbnailSplitterSizes());
// Show the thumbnail bar after setting the parent to avoid recreating
// the native window and to avoid QTBUG-87345.
mThumbnailBar->setVisible(GwenviewConfig::thumbnailBarIsVisible());
QVBoxLayout* layout = new QVBoxLayout(q);
layout->setContentsMargins(0, 0, 0, 0);
......
Markdown is supported
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