From 5c97a3abb5f74cc0b20abf57ed3644bc72ffc53c Mon Sep 17 00:00:00 2001 From: Anmol Gautam Date: Fri, 9 Mar 2018 22:28:52 +0100 Subject: [PATCH] Fix losing menubar visibility settings after quit from fullscreen BUG: 391415 Differential Revision: https://phabricator.kde.org/D11062 --- src/lib/app/browserwindow.cpp | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/src/lib/app/browserwindow.cpp b/src/lib/app/browserwindow.cpp index ce3ff88a1..bcc90ea95 100644 --- a/src/lib/app/browserwindow.cpp +++ b/src/lib/app/browserwindow.cpp @@ -610,15 +610,17 @@ void BrowserWindow::loadSettings() } settings.endGroup(); + m_statusBarVisible = showStatusBar; statusBar()->setVisible(!isFullScreen() && showStatusBar); m_bookmarksToolbar->setVisible(showBookmarksToolbar); m_navigationToolbar->setVisible(showNavigationToolbar); #ifndef Q_OS_MACOS + m_menuBarVisible = showMenuBar; menuBar()->setVisible(!isFullScreen() && showMenuBar); #endif - m_navigationToolbar->setSuperMenuVisible(!showMenuBar); + m_navigationToolbar->setSuperMenuVisible(isFullScreen() || !showMenuBar); } void BrowserWindow::goForward() @@ -1226,6 +1228,9 @@ bool BrowserWindow::event(QEvent *event) m_navigationContainer->hide(); m_navigationToolbar->enterFullScreen(); + + // Show main menu button since menubar is hidden + m_navigationToolbar->setSuperMenuVisible(true); } else if (e->oldState() & Qt::WindowFullScreen && !(windowState() & Qt::WindowFullScreen)) { // Leave fullscreen