Fix #24678: Remember which tab was last open within a session

musescore · Oct 1, 2024 · 8560e29 · 8560e29
1 parent 0512822
commit 8560e29
Show file tree

Hide file tree

Showing 4 changed files with 24 additions and 1 deletion.
diff --git a/src/appshell/iappshellconfiguration.h b/src/appshell/iappshellconfiguration.h
@@ -66,6 +66,9 @@ class IAppShellConfiguration : MODULE_EXPORT_INTERFACE
     virtual bool needShowSplashScreen() const = 0;
     virtual void setNeedShowSplashScreen(bool show) = 0;
 
+    virtual QString preferencesDialogLastOpenedPageId() const = 0;
+    virtual void setPreferencesDialogLastOpenedPageId(QString lastOpenedPageId) = 0;
+
     virtual void startEditSettings() = 0;
     virtual void applySettings() = 0;
     virtual void rollbackSettings() = 0;

diff --git a/src/appshell/internal/appshellconfiguration.cpp b/src/appshell/internal/appshellconfiguration.cpp
@@ -188,6 +188,16 @@ void AppShellConfiguration::setNeedShowSplashScreen(bool show)
     settings()->setSharedValue(SPLASH_SCREEN_VISIBLE_KEY, Val(show));
 }
 
+QString AppShellConfiguration::preferencesDialogLastOpenedPageId() const
+{
+    return m_preferencesDialogCurrentPageIdQString;
+}
+
+void AppShellConfiguration::setPreferencesDialogLastOpenedPageId(QString lastOpenedPageId)
+{
+    m_preferencesDialogCurrentPageIdQString = lastOpenedPageId;
+}
+
 void AppShellConfiguration::startEditSettings()
 {
     settings()->beginTransaction();

diff --git a/src/appshell/internal/appshellconfiguration.h b/src/appshell/internal/appshellconfiguration.h
@@ -85,6 +85,9 @@ class AppShellConfiguration : public IAppShellConfiguration, public muse::Inject
     bool needShowSplashScreen() const override;
     void setNeedShowSplashScreen(bool show) override;
 
+    QString preferencesDialogLastOpenedPageId() const override;
+    void setPreferencesDialogLastOpenedPageId(QString lastOpenedPageId) override;
+
     void startEditSettings() override;
     void applySettings() override;
     void rollbackSettings() override;
@@ -106,6 +109,8 @@ class AppShellConfiguration : public IAppShellConfiguration, public muse::Inject
     muse::Ret writeSessionState(const QByteArray& data);
 
     muse::io::paths_t parseSessionProjectsPaths(const QByteArray& json) const;
+
+    QString m_preferencesDialogCurrentPageIdQString = "";
 };
 }
 

diff --git a/src/appshell/view/preferences/preferencesmodel.cpp b/src/appshell/view/preferences/preferencesmodel.cpp
@@ -143,7 +143,11 @@ void PreferencesModel::load(const QString& currentPageId)
     if (!currentPageId.isEmpty()) {
         setCurrentPageId(currentPageId);
     } else {
-        setCurrentPageId("general");
+        if (configuration()->preferencesDialogLastOpenedPageId().isEmpty()) {
+            setCurrentPageId("general");
+        } else {
+            setCurrentPageId(configuration()->preferencesDialogLastOpenedPageId());
+        }
     }
 
     m_rootItem = new PreferencePageItem();
@@ -275,6 +279,7 @@ void PreferencesModel::setCurrentPageId(QString currentPageId)
     }
 
     m_currentPageId = currentPageId;
+    configuration()->setPreferencesDialogLastOpenedPageId(currentPageId);
     emit currentPageIdChanged(m_currentPageId);
 }