diff --git a/src/kmainwindow.cpp b/src/kmainwindow.cpp
index 2531aa9..d74832f 100644
--- a/src/kmainwindow.cpp
+++ b/src/kmainwindow.cpp
@@ -1,942 +1,938 @@
 /* This file is part of the KDE libraries
     Copyright
     (C) 2000 Reginald Stadlbauer (reggie@kde.org)
     (C) 1997 Stephan Kulow (coolo@kde.org)
     (C) 1997-2000 Sven Radej (radej@kde.org)
     (C) 1997-2000 Matthias Ettrich (ettrich@kde.org)
     (C) 1999 Chris Schlaeger (cs@kde.org)
     (C) 2002 Joseph Wenninger (jowenn@kde.org)
     (C) 2005-2006 Hamish Rodda (rodda@kde.org)
     (C) 2000-2008 David Faure (faure@kde.org)
 
     This library is free software; you can redistribute it and/or
     modify it under the terms of the GNU Library General Public
     License version 2 as published by the Free Software Foundation.
 
     This library is distributed in the hope that it will be useful,
     but WITHOUT ANY WARRANTY; without even the implied warranty of
     MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
     Library General Public License for more details.
 
     You should have received a copy of the GNU Library General Public License
     along with this library; see the file COPYING.LIB.  If not, write to
     the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
     Boston, MA 02110-1301, USA.
 */
 
 #include "kmainwindow.h"
 
 #include "kmainwindow_p.h"
 #ifdef QT_DBUS_LIB
 #include "kmainwindowiface_p.h"
 #endif
 #include "ktoolbarhandler_p.h"
 #include "khelpmenu.h"
 #include "ktoolbar.h"
 
 #include <QApplication>
 #include <QList>
 #include <QObject>
 #include <QTimer>
 #include <QCloseEvent>
 #include <QDockWidget>
 #include <QMenuBar>
 #include <QSessionManager>
 #include <QStatusBar>
 #include <QStyle>
 #include <QWidget>
 #include <QWindow>
 #ifdef QT_DBUS_LIB
 #include <QDBusConnection>
 #endif
 
 #include <ktoggleaction.h>
 #include <kaboutdata.h>
 #include <kconfig.h>
 #include <ksharedconfig.h>
 #include <klocalizedstring.h>
 #include <kwindowsystem.h>
 #include <kconfiggroup.h>
 #include <kwindowconfig.h>
 #include <kconfiggui.h>
 
 //#include <ctype.h>
 
 static const char WINDOW_PROPERTIES[]="WindowProperties";
 
 static QMenuBar *internalMenuBar(KMainWindow *mw)
 {
     return mw->findChild<QMenuBar *>(QString(), Qt::FindDirectChildrenOnly);
 }
 
 static QStatusBar *internalStatusBar(KMainWindow *mw)
 {
     return mw->findChild<QStatusBar *>(QString(), Qt::FindDirectChildrenOnly);
 }
 
 /**
 
  * Listens to resize events from QDockWidgets. The KMainWindow
  * settings are set as dirty, as soon as at least one resize
  * event occurred. The listener is attached to the dock widgets
  * by dock->installEventFilter(dockResizeListener) inside
  * KMainWindow::event().
  */
 class DockResizeListener : public QObject
 {
     Q_OBJECT
 public:
     DockResizeListener(KMainWindow *win);
     ~DockResizeListener() override;
     bool eventFilter(QObject *watched, QEvent *event) override;
 
 private:
     KMainWindow *m_win;
 };
 
 DockResizeListener::DockResizeListener(KMainWindow *win) :
     QObject(win),
     m_win(win)
 {
 }
 
 DockResizeListener::~DockResizeListener()
 {
 }
 
 bool DockResizeListener::eventFilter(QObject *watched, QEvent *event)
 {
     switch (event->type()) {
     case QEvent::Resize:
     case QEvent::Move:
     case QEvent::Hide:
         m_win->k_ptr->setSettingsDirty(KMainWindowPrivate::CompressCalls);
         break;
 
     default:
         break;
     }
 
     return QObject::eventFilter(watched, event);
 }
 
 KMWSessionManager::KMWSessionManager()
 {
     connect(qApp, &QGuiApplication::saveStateRequest,
             this, &KMWSessionManager::saveState);
     connect(qApp, &QGuiApplication::commitDataRequest,
             this, &KMWSessionManager::commitData);
 }
 
 KMWSessionManager::~KMWSessionManager()
 {
 }
 
 void KMWSessionManager::saveState(QSessionManager &sm)
 {
     KConfigGui::setSessionConfig(sm.sessionId(), sm.sessionKey());
 
     KConfig *config = KConfigGui::sessionConfig();
     if (!KMainWindow::memberList().isEmpty()) {
         // According to Jochen Wilhelmy <digisnap@cs.tu-berlin.de>, this
         // hook is useful for better document orientation
         KMainWindow::memberList().at(0)->saveGlobalProperties(config);
     }
 
     int n = 0;
     foreach (KMainWindow *mw, KMainWindow::memberList()) {
         n++;
         mw->savePropertiesInternal(config, n);
     }
 
     KConfigGroup group(config, "Number");
     group.writeEntry("NumberOfWindows", n);
 
     // store new status to disk
     config->sync();
 
     // generate discard command for new file
     QString localFilePath =  QStandardPaths::writableLocation(QStandardPaths::GenericConfigLocation) + QLatin1Char('/') + config->name();
     if (QFile::exists(localFilePath)) {
         QStringList discard;
         discard << QStringLiteral("rm");
         discard << localFilePath;
         sm.setDiscardCommand(discard);
     }
 }
 
 void KMWSessionManager::commitData(QSessionManager &sm)
 {
     if (!sm.allowsInteraction()) {
         return;
     }
 
     /*
        Purpose of this exercise: invoke queryClose() without actually closing the
        windows, because
        - queryClose() may contain session management code, so it must be invoked
        - actually closing windows may quit the application - cf.
          QGuiApplication::quitOnLastWindowClosed()
        - quitting the application and thus closing the session manager connection
          violates the X11 XSMP protocol.
          The exact requirement of XSMP that would be broken is,
          in the description of the client's state machine:
 
            save-yourself-done: (changing state is forbidden)
 
          Closing the session manager connection causes a state change.
          Worst of all, that is a real problem with ksmserver - it will not save
          applications that quit on their own in state save-yourself-done.
      */
     foreach (KMainWindow *window, KMainWindow::memberList()) {
         if (window->testAttribute(Qt::WA_WState_Hidden)) {
             continue;
         }
         QCloseEvent e;
         QApplication::sendEvent(window, &e);
         if (!e.isAccepted()) {
             sm.cancel();
             return;
         }
     }
 }
 
 Q_GLOBAL_STATIC(KMWSessionManager, ksm)
 Q_GLOBAL_STATIC(QList<KMainWindow *>, sMemberList)
 
 KMainWindow::KMainWindow(QWidget *parent, Qt::WindowFlags f)
     : QMainWindow(parent, f), k_ptr(new KMainWindowPrivate)
 {
     k_ptr->init(this);
 }
 
 KMainWindow::KMainWindow(KMainWindowPrivate &dd, QWidget *parent, Qt::WindowFlags f)
     : QMainWindow(parent, f), k_ptr(&dd)
 {
     k_ptr->init(this);
 }
 
 void KMainWindowPrivate::init(KMainWindow *_q)
 {
     q = _q;
     QGuiApplication::setFallbackSessionManagementEnabled(false);
     q->setAnimated(q->style()->styleHint(QStyle::SH_Widget_Animate, nullptr, q));
 
     q->setAttribute(Qt::WA_DeleteOnClose);
 
     helpMenu = nullptr;
 
     //actionCollection()->setWidget( this );
 #if 0
     QObject::connect(KGlobalSettings::self(), SIGNAL(settingsChanged(int)),
                      q, SLOT(_k_slotSettingsChanged(int)));
 #endif
 
     // force KMWSessionManager creation
     ksm();
 
     sMemberList()->append(q);
 
     // If application is translated, load translator information for use in
     // KAboutApplicationDialog or other getters. The context and messages below
     // both must be exactly as listed, and are forced to be loaded from the
     // application's own message catalog instead of kxmlgui's.
     KAboutData aboutData(KAboutData::applicationData());
     if (aboutData.translators().isEmpty()) {
         aboutData.setTranslator(
                 i18ndc(nullptr, "NAME OF TRANSLATORS", "Your names"),
                 i18ndc(nullptr, "EMAIL OF TRANSLATORS", "Your emails"));
 
         KAboutData::setApplicationData(aboutData);
     }
 
     settingsDirty = false;
     autoSaveSettings = false;
     autoSaveWindowSize = true; // for compatibility
     //d->kaccel = actionCollection()->kaccel();
     settingsTimer = nullptr;
     sizeTimer = nullptr;
 
     dockResizeListener = new DockResizeListener(_q);
     letDirtySettings = true;
 
     sizeApplied = false;
 }
 
 static bool endsWithHashNumber(const QString &s)
 {
     for (int i = s.length() - 1;
             i > 0;
             --i) {
         if (s[ i ] == QLatin1Char('#') && i != s.length() - 1) {
             return true;    // ok
         }
         if (!s[ i ].isDigit()) {
             break;
         }
     }
     return false;
 }
 
 static inline bool isValidDBusObjectPathCharacter(const QChar &c)
 {
     ushort u = c.unicode();
     return (u >= QLatin1Char('a') && u <= QLatin1Char('z'))
            || (u >= QLatin1Char('A') && u <= QLatin1Char('Z'))
            || (u >= QLatin1Char('0') && u <= QLatin1Char('9'))
            || (u == QLatin1Char('_')) || (u == QLatin1Char('/'));
 }
 
 void KMainWindowPrivate::polish(KMainWindow *q)
 {
     // Set a unique object name. Required by session management, window management, and for the dbus interface.
     QString objname;
     QString s;
     int unusedNumber = 1;
     const QString name = q->objectName();
     bool startNumberingImmediately = true;
     bool tryReuse = false;
     if (name.isEmpty()) {
         // no name given
         objname = QStringLiteral("MainWindow#");
     } else if (name.endsWith(QLatin1Char('#'))) {
         // trailing # - always add a number  - KWin uses this for better grouping
         objname = name;
     } else if (endsWithHashNumber(name)) {
         // trailing # with a number - like above, try to use the given number first
         objname = name;
         tryReuse = true;
         startNumberingImmediately = false;
     } else {
         objname = name;
         startNumberingImmediately = false;
     }
 
     s = objname;
     if (startNumberingImmediately) {
         s += QLatin1Char('1');
     }
 
     for (;;) {
         const QList<QWidget *> list = qApp->topLevelWidgets();
         bool found = false;
         foreach (QWidget *w, list) {
             if (w != q && w->objectName() == s) {
                 found = true;
                 break;
             }
         }
         if (!found) {
             break;
         }
         if (tryReuse) {
             objname = name.left(name.length() - 1);   // lose the hash
             unusedNumber = 0; // start from 1 below
             tryReuse = false;
         }
         s.setNum(++unusedNumber);
         s = objname + s;
     }
     q->setObjectName(s);
     q->winId(); // workaround for setWindowRole() crashing, and set also window role, just in case TT
     q->setWindowRole(s);   // will keep insisting that object name suddenly should not be used for window role
 
     dbusName = QLatin1Char('/') + QCoreApplication::applicationName() + QLatin1Char('/');
     dbusName += q->objectName().replace(QLatin1Char('/'), QLatin1Char('_'));
     // Clean up for dbus usage: any non-alphanumeric char should be turned into '_'
     const int len = dbusName.length();
     for (int i = 0; i < len; ++i) {
         if (!isValidDBusObjectPathCharacter(dbusName[i])) {
             dbusName[i] = QLatin1Char('_');
         }
     }
 
 #ifdef QT_DBUS_LIB
     QDBusConnection::sessionBus().registerObject(dbusName, q, QDBusConnection::ExportScriptableSlots |
             QDBusConnection::ExportScriptableProperties |
             QDBusConnection::ExportNonScriptableSlots |
             QDBusConnection::ExportNonScriptableProperties |
             QDBusConnection::ExportAdaptors);
 #endif
 }
 
 void KMainWindowPrivate::setSettingsDirty(CallCompression callCompression)
 {
     if (!letDirtySettings) {
         return;
     }
 
     settingsDirty = true;
     if (autoSaveSettings) {
         if (callCompression == CompressCalls) {
             if (!settingsTimer) {
                 settingsTimer = new QTimer(q);
                 settingsTimer->setInterval(500);
                 settingsTimer->setSingleShot(true);
                 QObject::connect(settingsTimer, &QTimer::timeout,
                                  q, &KMainWindow::saveAutoSaveSettings);
             }
             settingsTimer->start();
         } else {
             q->saveAutoSaveSettings();
         }
     }
 }
 
 void KMainWindowPrivate::setSizeDirty()
 {
     if (autoSaveWindowSize) {
         if (!sizeTimer) {
             sizeTimer = new QTimer(q);
             sizeTimer->setInterval(500);
             sizeTimer->setSingleShot(true);
             QObject::connect(sizeTimer, SIGNAL(timeout()), q, SLOT(_k_slotSaveAutoSaveSize()));
         }
         sizeTimer->start();
     }
 }
 
 KMainWindow::~KMainWindow()
 {
     sMemberList()->removeAll(this);
     delete static_cast<QObject *>(k_ptr->dockResizeListener);  //so we don't get anymore events after k_ptr is destroyed
     delete k_ptr;
 }
 
 #ifndef KXMLGUI_NO_DEPRECATED
 QMenu *KMainWindow::helpMenu(const QString &aboutAppText, bool showWhatsThis)
 {
     K_D(KMainWindow);
     if (!d->helpMenu) {
         if (aboutAppText.isEmpty()) {
             d->helpMenu = new KHelpMenu(this, KAboutData::applicationData(), showWhatsThis);
         } else {
             d->helpMenu = new KHelpMenu(this, aboutAppText, showWhatsThis);
         }
 
         if (!d->helpMenu) {
             return nullptr;
         }
     }
 
     return d->helpMenu->menu();
 }
 
 QMenu *KMainWindow::customHelpMenu(bool showWhatsThis)
 {
     K_D(KMainWindow);
     if (!d->helpMenu) {
         d->helpMenu = new KHelpMenu(this, QString(), showWhatsThis);
         connect(d->helpMenu, &KHelpMenu::showAboutApplication,
                 this, &KMainWindow::showAboutApplication);
     }
 
     return d->helpMenu->menu();
 }
 #endif
 
 bool KMainWindow::canBeRestored(int number)
 {
     if (!qApp->isSessionRestored()) {
         return false;
     }
     KConfig *config = KConfigGui::sessionConfig();
     if (!config) {
         return false;
     }
 
     KConfigGroup group(config, "Number");
     const int n = group.readEntry("NumberOfWindows", 1);
     return number >= 1 && number <= n;
 }
 
 const QString KMainWindow::classNameOfToplevel(int number)
 {
     if (!qApp->isSessionRestored()) {
         return QString();
     }
     KConfig *config = KConfigGui::sessionConfig();
     if (!config) {
         return QString();
     }
 
     KConfigGroup group(config, QByteArray(WINDOW_PROPERTIES).append(QByteArray::number(number)).constData());
     if (!group.hasKey("ClassName")) {
         return QString();
     } else {
         return group.readEntry("ClassName");
     }
 }
 
 bool KMainWindow::restore(int number, bool show)
 {
     if (!canBeRestored(number)) {
         return false;
     }
     KConfig *config = KConfigGui::sessionConfig();
     if (readPropertiesInternal(config, number)) {
         if (show) {
             KMainWindow::show();
         }
         return false;
     }
     return false;
 }
 
 void KMainWindow::setCaption(const QString &caption)
 {
     setPlainCaption(caption);
 }
 
 void KMainWindow::setCaption(const QString &caption, bool modified)
 {
     QString title = caption;
     if (!title.contains(QStringLiteral("[*]")) && !title.isEmpty()) { // append the placeholder so that the modified mechanism works
         title.append(QStringLiteral(" [*]"));
     }
     setPlainCaption(title);
     setWindowModified(modified);
 }
 
 void KMainWindow::setPlainCaption(const QString &caption)
 {
     setWindowTitle(caption);
 }
 
 void KMainWindow::appHelpActivated()
 {
     K_D(KMainWindow);
     if (!d->helpMenu) {
         d->helpMenu = new KHelpMenu(this);
         if (!d->helpMenu) {
             return;
         }
     }
     d->helpMenu->appHelpActivated();
 }
 
 void KMainWindow::closeEvent(QCloseEvent *e)
 {
     K_D(KMainWindow);
 
     // Save settings if auto-save is enabled, and settings have changed
     if (d->settingsTimer && d->settingsTimer->isActive()) {
         d->settingsTimer->stop();
         saveAutoSaveSettings();
     }
     if (d->sizeTimer && d->sizeTimer->isActive()) {
         d->sizeTimer->stop();
         d->_k_slotSaveAutoSaveSize();
     }
 
     if (queryClose()) {
         // widgets will start destroying themselves at this point and we don't
         // want to save state anymore after this as it might be incorrect
         d->autoSaveSettings = false;
         d->letDirtySettings = false;
         e->accept();
     } else {
         e->ignore();    //if the window should not be closed, don't close it
     }
 }
 
 bool KMainWindow::queryClose()
 {
     return true;
 }
 
 void KMainWindow::saveGlobalProperties(KConfig *)
 {
 }
 
 void KMainWindow::readGlobalProperties(KConfig *)
 {
 }
 
 void KMainWindow::savePropertiesInternal(KConfig *config, int number)
 {
     K_D(KMainWindow);
     const bool oldASWS = d->autoSaveWindowSize;
     d->autoSaveWindowSize = true; // make saveMainWindowSettings save the window size
 
     KConfigGroup cg(config, QByteArray(WINDOW_PROPERTIES).append(QByteArray::number(number)).constData());
 
     // store objectName, className, Width and Height  for later restoring
     // (Only useful for session management)
     cg.writeEntry("ObjectName", objectName());
     cg.writeEntry("ClassName", metaObject()->className());
 
     saveMainWindowSettings(cg); // Menubar, statusbar and Toolbar settings.
 
     cg = KConfigGroup(config, QByteArray::number(number).constData());
     saveProperties(cg);
 
     d->autoSaveWindowSize = oldASWS;
 }
 
 void KMainWindow::saveMainWindowSettings(KConfigGroup &cg)
 {
     K_D(KMainWindow);
     //qDebug(200) << "KMainWindow::saveMainWindowSettings " << cg.name();
 
     // Called by session management - or if we want to save the window size anyway
     if (d->autoSaveWindowSize) {
         KWindowConfig::saveWindowSize(windowHandle(), cg);
     }
 
     // One day will need to save the version number, but for now, assume 0
     // Utilise the QMainWindow::saveState() functionality.
     const QByteArray state = saveState();
     cg.writeEntry("State", state.toBase64());
 
     QStatusBar *sb = internalStatusBar(this);
     if (sb) {
         if (!cg.hasDefault("StatusBar") && !sb->isHidden()) {
             cg.revertToDefault("StatusBar");
         } else {
             cg.writeEntry("StatusBar", sb->isHidden() ? "Disabled" : "Enabled");
         }
     }
 
     QMenuBar *mb = internalMenuBar(this);
     if (mb) {
         if (!cg.hasDefault("MenuBar") && !mb->isHidden()) {
             cg.revertToDefault("MenuBar");
         } else {
             cg.writeEntry("MenuBar", mb->isHidden() ? "Disabled" : "Enabled");
         }
     }
 
     if (!autoSaveSettings() || cg.name() == autoSaveGroup()) {
         // TODO should be cg == d->autoSaveGroup, to compare both kconfig and group name
         if (!cg.hasDefault("ToolBarsMovable") && !KToolBar::toolBarsLocked()) {
             cg.revertToDefault("ToolBarsMovable");
         } else {
             cg.writeEntry("ToolBarsMovable", KToolBar::toolBarsLocked() ? "Disabled" : "Enabled");
         }
     }
 
     int n = 1; // Toolbar counter. toolbars are counted from 1,
     foreach (KToolBar *toolbar, toolBars()) {
         QByteArray groupName("Toolbar");
         // Give a number to the toolbar, but prefer a name if there is one,
         // because there's no real guarantee on the ordering of toolbars
         groupName += (toolbar->objectName().isEmpty() ? QByteArray::number(n) : QByteArray(" ").append(toolbar->objectName().toUtf8()));
 
         KConfigGroup toolbarGroup(&cg, groupName.constData());
         toolbar->saveSettings(toolbarGroup);
         n++;
     }
 }
 
 bool KMainWindow::readPropertiesInternal(KConfig *config, int number)
 {
     K_D(KMainWindow);
 
     const bool oldLetDirtySettings = d->letDirtySettings;
     d->letDirtySettings = false;
 
     if (number == 1) {
         readGlobalProperties(config);
     }
 
     // in order they are in toolbar list
     KConfigGroup cg(config, QByteArray(WINDOW_PROPERTIES).append(QByteArray::number(number)).constData());
 
     // restore the object name (window role)
     if (cg.hasKey("ObjectName")) {
         setObjectName(cg.readEntry("ObjectName"));
     }
 
     d->sizeApplied = false; // since we are changing config file, reload the size of the window
     // if necessary. Do it before the call to applyMainWindowSettings.
     applyMainWindowSettings(cg); // Menubar, statusbar and toolbar settings.
 
     KConfigGroup grp(config, QByteArray::number(number).constData());
     readProperties(grp);
 
     d->letDirtySettings = oldLetDirtySettings;
 
     return true;
 }
 
 void KMainWindow::applyMainWindowSettings(const KConfigGroup &cg)
 {
     K_D(KMainWindow);
     //qDebug(200) << "KMainWindow::applyMainWindowSettings " << cg.name();
 
     QWidget *focusedWidget = QApplication::focusWidget();
 
     const bool oldLetDirtySettings = d->letDirtySettings;
     d->letDirtySettings = false;
 
     if (!d->sizeApplied) {
         winId(); // ensure there's a window created
         KWindowConfig::restoreWindowSize(windowHandle(), cg);
         // NOTICE: QWindow::setGeometry() does NOT impact the backing QWidget geometry even if the platform
         // window was created -> QTBUG-40584. We therefore copy the size here.
         // TODO: remove once this was resolved in QWidget QPA
         resize(windowHandle()->size());
         d->sizeApplied = true;
     }
 
     QStatusBar *sb = internalStatusBar(this);
     if (sb) {
         QString entry = cg.readEntry("StatusBar", "Enabled");
         sb->setVisible( entry != QLatin1String("Disabled") );
     }
 
     QMenuBar *mb = internalMenuBar(this);
     if (mb) {
         QString entry = cg.readEntry("MenuBar", "Enabled");
         mb->setVisible( entry != QLatin1String("Disabled") );
     }
 
     if (!autoSaveSettings() || cg.name() == autoSaveGroup()) {   // TODO should be cg == d->autoSaveGroup, to compare both kconfig and group name
         QString entry = cg.readEntry("ToolBarsMovable", "Disabled");
         KToolBar::setToolBarsLocked(entry == QLatin1String("Disabled"));
     }
 
     int n = 1; // Toolbar counter. toolbars are counted from 1,
     foreach (KToolBar *toolbar, toolBars()) {
         QByteArray groupName("Toolbar");
         // Give a number to the toolbar, but prefer a name if there is one,
         // because there's no real guarantee on the ordering of toolbars
         groupName += (toolbar->objectName().isEmpty() ? QByteArray::number(n) : QByteArray(" ").append(toolbar->objectName().toUtf8()));
 
         KConfigGroup toolbarGroup(&cg, groupName.constData());
         toolbar->applySettings(toolbarGroup);
         n++;
     }
 
     QByteArray state;
     if (cg.hasKey("State")) {
         state = cg.readEntry("State", state);
         state = QByteArray::fromBase64(state);
         // One day will need to load the version number, but for now, assume 0
         restoreState(state);
     }
 
     if (focusedWidget) {
         focusedWidget->setFocus();
     }
 
     d->settingsDirty = false;
     d->letDirtySettings = oldLetDirtySettings;
 }
 
 #ifndef KXMLGUI_NO_DEPRECATED
 void KMainWindow::restoreWindowSize(const KConfigGroup &cg)
 {
     KWindowConfig::restoreWindowSize(windowHandle(), cg);
 }
 #endif
 
 #ifndef KXMLGUI_NO_DEPRECATED
 void KMainWindow::saveWindowSize(KConfigGroup &cg) const
 {
     KWindowConfig::saveWindowSize(windowHandle(), cg);
 }
 #endif
 
 void KMainWindow::setSettingsDirty()
 {
     K_D(KMainWindow);
     d->setSettingsDirty();
 }
 
 bool KMainWindow::settingsDirty() const
 {
     K_D(const KMainWindow);
     return d->settingsDirty;
 }
 
 void KMainWindow::setAutoSaveSettings(const QString &groupName, bool saveWindowSize)
 {
     setAutoSaveSettings(KConfigGroup(KSharedConfig::openConfig(), groupName), saveWindowSize);
 }
 
 void KMainWindow::setAutoSaveSettings(const KConfigGroup &group,
                                       bool saveWindowSize)
 {
     K_D(KMainWindow);
     d->autoSaveSettings = true;
     d->autoSaveGroup = group;
     d->autoSaveWindowSize = saveWindowSize;
 
     if (!saveWindowSize && d->sizeTimer) {
         d->sizeTimer->stop();
     }
 
     // Now read the previously saved settings
     applyMainWindowSettings(d->autoSaveGroup);
 }
 
 void KMainWindow::resetAutoSaveSettings()
 {
     K_D(KMainWindow);
     d->autoSaveSettings = false;
     if (d->settingsTimer) {
         d->settingsTimer->stop();
     }
 }
 
 bool KMainWindow::autoSaveSettings() const
 {
     K_D(const KMainWindow);
     return d->autoSaveSettings;
 }
 
 QString KMainWindow::autoSaveGroup() const
 {
     K_D(const KMainWindow);
     return d->autoSaveSettings ? d->autoSaveGroup.name() : QString();
 }
 
 KConfigGroup KMainWindow::autoSaveConfigGroup() const
 {
     K_D(const KMainWindow);
     return d->autoSaveSettings ? d->autoSaveGroup : KConfigGroup();
 }
 
 void KMainWindow::saveAutoSaveSettings()
 {
     K_D(KMainWindow);
     Q_ASSERT(d->autoSaveSettings);
     //qDebug(200) << "KMainWindow::saveAutoSaveSettings -> saving settings";
     saveMainWindowSettings(d->autoSaveGroup);
     d->autoSaveGroup.sync();
     d->settingsDirty = false;
 }
 
 bool KMainWindow::event(QEvent *ev)
 {
     K_D(KMainWindow);
     switch (ev->type()) {
 #if defined(Q_OS_WIN) || defined(Q_OS_OSX)
     case QEvent::Move:
 #endif
     case QEvent::Resize:
         d->setSizeDirty();
         break;
     case QEvent::Polish:
         d->polish(this);
         break;
     case QEvent::ChildPolished: {
         QChildEvent *event = static_cast<QChildEvent *>(ev);
         QDockWidget *dock = qobject_cast<QDockWidget *>(event->child());
         KToolBar *toolbar = qobject_cast<KToolBar *>(event->child());
         QMenuBar *menubar = qobject_cast<QMenuBar *>(event->child());
         if (dock) {
             connect(dock, &QDockWidget::dockLocationChanged,
                     this, &KMainWindow::setSettingsDirty);
-            connect(dock, &QDockWidget::visibilityChanged,
-                    this, &KMainWindow::setSettingsDirty, Qt::QueuedConnection);
             connect(dock, &QDockWidget::topLevelChanged,
                     this, &KMainWindow::setSettingsDirty);
 
             // there is no signal emitted if the size of the dock changes,
             // hence install an event filter instead
             dock->installEventFilter(k_ptr->dockResizeListener);
         } else if (toolbar) {
             // there is no signal emitted if the size of the toolbar changes,
             // hence install an event filter instead
             toolbar->installEventFilter(k_ptr->dockResizeListener);
         } else if (menubar) {
             // there is no signal emitted if the size of the menubar changes,
             // hence install an event filter instead
             menubar->installEventFilter(k_ptr->dockResizeListener);
         }
     }
     break;
     case QEvent::ChildRemoved: {
         QChildEvent *event = static_cast<QChildEvent *>(ev);
         QDockWidget *dock = qobject_cast<QDockWidget *>(event->child());
         KToolBar *toolbar = qobject_cast<KToolBar *>(event->child());
         QMenuBar *menubar = qobject_cast<QMenuBar *>(event->child());
         if (dock) {
             disconnect(dock, &QDockWidget::dockLocationChanged,
                        this, &KMainWindow::setSettingsDirty);
-            disconnect(dock, &QDockWidget::visibilityChanged,
-                       this, &KMainWindow::setSettingsDirty);
             disconnect(dock, &QDockWidget::topLevelChanged,
                        this, &KMainWindow::setSettingsDirty);
             dock->removeEventFilter(k_ptr->dockResizeListener);
         } else if (toolbar) {
             toolbar->removeEventFilter(k_ptr->dockResizeListener);
         } else if (menubar) {
             menubar->removeEventFilter(k_ptr->dockResizeListener);
         }
     }
     break;
     default:
         break;
     }
     return QMainWindow::event(ev);
 }
 
 bool KMainWindow::hasMenuBar()
 {
     return internalMenuBar(this);
 }
 
 void KMainWindowPrivate::_k_slotSettingsChanged(int category)
 {
     Q_UNUSED(category);
 
     // This slot will be called when the style KCM changes settings that need
     // to be set on the already running applications.
 
     // At this level (KMainWindow) the only thing we need to restore is the
     // animations setting (whether the user wants builtin animations or not).
 
     q->setAnimated(q->style()->styleHint(QStyle::SH_Widget_Animate, nullptr, q));
 }
 
 void KMainWindowPrivate::_k_slotSaveAutoSaveSize()
 {
     if (autoSaveGroup.isValid()) {
         KWindowConfig::saveWindowSize(q->windowHandle(), autoSaveGroup);
     }
 }
 
 KToolBar *KMainWindow::toolBar(const QString &name)
 {
     QString childName = name;
     if (childName.isEmpty()) {
         childName = QStringLiteral("mainToolBar");
     }
 
     KToolBar *tb = findChild<KToolBar *>(childName);
     if (tb) {
         return tb;
     }
 
     KToolBar *toolbar = new KToolBar(childName, this); // non-XMLGUI toolbar
     return toolbar;
 }
 
 QList<KToolBar *> KMainWindow::toolBars() const
 {
     QList<KToolBar *> ret;
 
     foreach (QObject *child, children())
         if (KToolBar *toolBar = qobject_cast<KToolBar *>(child)) {
             ret.append(toolBar);
         }
 
     return ret;
 }
 
 QList<KMainWindow *> KMainWindow::memberList()
 {
     return *sMemberList();
 }
 
 QString KMainWindow::dbusName() const
 {
     return k_func()->dbusName;
 }
 
 #include "moc_kmainwindow.cpp"
 #include "kmainwindow.moc"