Improve initialization order.
authorSoren Stoutner <soren@stoutner.com>
Mon, 21 Feb 2022 22:33:37 +0000 (15:33 -0700)
committerSoren Stoutner <soren@stoutner.com>
Mon, 21 Feb 2022 22:33:37 +0000 (15:33 -0700)
src/BrowserWindow.cpp
src/BrowserWindow.h
src/MainView.cpp
src/MainView.h

index a2eda98e36b5a23aefcc2ca5bcb8c9ef329bf02a..7e908a34dff6a876eba5e340c8f82c69ba2191c0 100644 (file)
@@ -132,18 +132,14 @@ BrowserWindow::BrowserWindow() : KXmlGuiWindow()
     // Display dialogs.
     connect(zoomFactorActionPointer, SIGNAL(triggered()), this, SLOT(getZoomFactorFromUser()));
 
-    // Set the current zoom factor.
-    currentZoomFactor = Settings::zoomFactor();
-
-    // Initialize the on-the-fly menus, as the slot connections had not been created when MainView was constructed.
-    updateOnTheFlyZoomFactor(currentZoomFactor);
-    updateOnTheFlySearchEngine(Settings::searchEngine());
-
     // Update the status bar with the URL when a link is hovered.
     connect(mainViewPointer, SIGNAL(linkHovered(QString)), this, SLOT(updateStatusBar(QString)));
 
     // Setup the GUI based on the privacybrowserui.rc file.
     setupGUI();
+
+    // Load the initial webstie.
+    mainViewPointer->loadInitialWebsite();
 }
 
 void BrowserWindow::fileNew() const
@@ -349,8 +345,11 @@ void BrowserWindow::updateOnTheFlyUserAgent(const QString &userAgent) const
     }
 }
 
-void BrowserWindow::updateOnTheFlyZoomFactor(const double &zoomFactor) const
+void BrowserWindow::updateOnTheFlyZoomFactor(const double &zoomFactor)
 {
+    // Set the current zoom factor.
+    currentZoomFactor = Settings::zoomFactor();
+
     // Update the zoom factor action text, formatting the double with 2 decimal places.
     zoomFactorActionPointer->setText(ki18nc("@action", "Zoom Factor - %1").subs(zoomFactor, 0, '0', 2).toString());
 }
index 357909e3dc43bcf5bf2b2998f8b0cb5c62525f81..14ef8cc2d074d4ce312a0bcdb366ecd52eb3937e 100644 (file)
@@ -45,7 +45,7 @@ private Q_SLOTS:
     void settingsConfigure();
     void updateOnTheFlySearchEngine(const QString &searchEngine) const;
     void updateOnTheFlyUserAgent(const QString &userAgent) const;
-    void updateOnTheFlyZoomFactor(const double &zoomFactor) const;
+    void updateOnTheFlyZoomFactor(const double &zoomFactor);
     void updateSearchEngineLabel(const QString &searchEngineString) const;
     void updateStatusBar(const QString &statusBarMessage) const;
     void updateUserAgentLabel(const QString &userAgentName) const;
index 04cc1ee87d86082db817e3023d110000904e52e7..3893f2b96c683b445d5c1eb92e49f475060e554b 100644 (file)
@@ -93,26 +93,8 @@ MainView::MainView(QWidget *parent) : QWidget(parent)
     // Don't allow JavaScript to open windows.
     webEngineSettingsPointer->setAttribute(QWebEngineSettings::JavascriptCanOpenWindows, false);
 
-    // Apply the application settings.
-    applyApplicationSettings();
-
     // Set the focus on the WebEngine view.
     webEngineViewPointer->setFocus();
-
-    // Get the arguments.
-    QStringList argumentsStringList = qApp->arguments();
-
-    // Check to see if the arguments lists contains a URL.
-    if (argumentsStringList.size() > 1)
-    {
-        // Load the URL from the arguments list.
-        webEngineViewPointer->setUrl(QUrl::fromUserInput(argumentsStringList.at(1)));
-    }
-    else
-    {
-        // Load the homepage.
-        goHome();
-    }
 }
 
 void MainView::applyApplicationSettings()
@@ -209,6 +191,27 @@ void MainView::goHome() const
     webEngineViewPointer->setUrl(QUrl::fromUserInput(Settings::homepage()));
 }
 
+void MainView::loadInitialWebsite()
+{
+    // Apply the application settings.
+    applyApplicationSettings();
+
+    // Get the arguments.
+    QStringList argumentsStringList = qApp->arguments();
+
+    // Check to see if the arguments lists contains a URL.
+    if (argumentsStringList.size() > 1)
+    {
+        // Load the URL from the arguments list.
+        webEngineViewPointer->setUrl(QUrl::fromUserInput(argumentsStringList.at(1)));
+    }
+    else
+    {
+        // Load the homepage.
+        goHome();
+    }
+}
+
 void MainView::loadUrlFromTextBox(QString urlFromUser) const
 {
     // Remove the focus from the URL line edit.
index 38b5928dcc328affceffde0e786293c9e28c1402..dba6914551e50ad693f0ce6c9cde75caafcfa8b5 100644 (file)
@@ -40,6 +40,7 @@ public:
 
     // The public functions.
     void applyOnTheFlyZoomFactor(const double &zoomFactor) const;
+    void loadInitialWebsite();
 
 signals:
     // The signals.