Rekonq stores visibility information of panels and toolbars if it enters 
fullscreen mode. This information is used when leaving fullscreen mode to 
decide which elements need to be made visible again.

However, rekonq overwrites this visibility information if the mouse is moved 
during fullscreen mode. The result is that some gui elements are not made 
visible again after fullscreen mode is left, e.g. the bookmarks toolbar.

The attached patch fixes this issue. It ensures that visibility of elements is 
only updated in windowed mode.
diff --git a/src/mainwindow.cpp b/src/mainwindow.cpp
index 1608014..ebadda2 100644
--- a/src/mainwindow.cpp
+++ b/src/mainwindow.cpp
@@ -813,16 +813,17 @@ void MainWindow::setWidgetsVisible(bool makeVisible)
 
     if (!makeVisible)
     {
-        // save current state
-        bookmarksToolBarFlag = m_bmBar->isHidden();
-        m_bmBar->hide();
-
-        m_view->setTabBarHidden(true);
+        // save current state, if in windowed mode
+        if (!isFullScreen())
+        {
+            bookmarksToolBarFlag = m_bmBar->isHidden();
+            sidePanelFlag = sidePanel()->isHidden();
+            bookmarksPanelFlag = bookmarksPanel()->isHidden();
+        }
         
-        sidePanelFlag = sidePanel()->isHidden();
-        sidePanel()->hide();
-
-        bookmarksPanelFlag = bookmarksPanel()->isHidden();
+        m_bmBar->hide();        
+        m_view->setTabBarHidden(true);
+        sidePanel()->hide();       
         bookmarksPanel()->hide();
 
         // hide main toolbar
@@ -834,7 +835,7 @@ void MainWindow::setWidgetsVisible(bool makeVisible)
         m_mainBar->show();
         m_view->setTabBarHidden(false);
 
-        // restore previous state
+        // restore state of windowed mode
         if (!bookmarksToolBarFlag)
             m_bmBar->show();
         if (!sidePanelFlag)
_______________________________________________
rekonq mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/rekonq

Reply via email to