Sidebar: Prevent items turning to top-level windows

Sometimes, this behavior occurred especially on OS X, and to lesser
extent on Linux and windows too.

Reviewed-by: Thomas Hartmann

diff --git a/src/plugins/coreplugin/sidebar.cpp b/src/plugins/coreplugin/sidebar.cpp
index 44cb0fd..e7cf5f1 100644 (file)
--- a/src/plugins/coreplugin/sidebar.cpp
+++ b/src/plugins/coreplugin/sidebar.cpp
@@ -401,6 +401,7 @@ void SideBarWidget::setCurrentItem(const QString &id)
     m_currentItem = item;
 
     layout()->addWidget(m_currentItem->widget());
+    m_currentItem->widget()->show();
 
     // Add buttons and remember their actions for later removal
     foreach (QToolButton *b, m_currentItem->createToolBarWidgets())
@@ -436,6 +437,7 @@ void SideBarWidget::removeCurrentItem()
         return;
 
     QWidget *w = m_currentItem->widget();
+    w->hide();
     layout()->removeWidget(w);
     w->setParent(0);
     m_sideBar->makeItemAvailable(m_currentItem);