From 0cd8d4b8421700ee6e0f0cfc7da03e71185d2792 Mon Sep 17 00:00:00 2001
From: Andrei Kortunov <andrei.kortunov@yandex.ru>
Date: Thu, 2 May 2019 19:00:47 +0400
Subject: [PATCH] Do not track resolution settings changes in the InputManager
 (bug #4902)

---
 CHANGELOG.md                            |  1 +
 apps/openmw/mwgui/settingswindow.cpp    |  1 +
 apps/openmw/mwinput/inputmanagerimp.cpp | 10 +++++++---
 components/settings/settings.cpp        |  6 ++++++
 components/settings/settings.hpp        |  2 ++
 5 files changed, 17 insertions(+), 3 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 96a5566be..b2fa5985e 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -58,6 +58,7 @@
     Bug #4877: Startup script executes only on a new game start
     Bug #4888: Global variable stray explicit reference calls break script compilation
     Bug #4896: Title screen music doesn't loop
+    Bug #4902: Using scrollbars in settings causes resolution to change
     Bug #4911: Editor: QOpenGLContext::swapBuffers() warning with Qt5
     Bug #4916: Specular power (shininess) material parameter is ignored when shaders are used.
     Bug #4918: Abilities don't play looping VFX when they're initially applied
diff --git a/apps/openmw/mwgui/settingswindow.cpp b/apps/openmw/mwgui/settingswindow.cpp
index deb1116d0..661c8c428 100644
--- a/apps/openmw/mwgui/settingswindow.cpp
+++ b/apps/openmw/mwgui/settingswindow.cpp
@@ -584,6 +584,7 @@ namespace MWGui
 
     void SettingsWindow::onOpen()
     {
+        highlightCurrentResolution();
         updateControlsBox();
         resetScrollbars();
         MWBase::Environment::get().getWindowManager()->setKeyFocusWidget(mOkButton);
diff --git a/apps/openmw/mwinput/inputmanagerimp.cpp b/apps/openmw/mwinput/inputmanagerimp.cpp
index 151d4add2..b45b9138c 100644
--- a/apps/openmw/mwinput/inputmanagerimp.cpp
+++ b/apps/openmw/mwinput/inputmanagerimp.cpp
@@ -826,9 +826,6 @@ namespace MWInput
                                         Settings::Manager::getInt("resolution y", "Video"),
                                         Settings::Manager::getBool("fullscreen", "Video"),
                                         Settings::Manager::getBool("window border", "Video"));
-
-            // We should reload TrueType fonts to fit new resolution
-            MWBase::Environment::get().getWindowManager()->loadUserFonts();
         }
     }
 
@@ -1129,10 +1126,17 @@ namespace MWInput
 
     void InputManager::windowResized(int x, int y)
     {
+        // Note: this is a side effect of resolution change or window resize.
+        // There is no need to track these changes.
         Settings::Manager::setInt("resolution x", "Video", x);
         Settings::Manager::setInt("resolution y", "Video", y);
+        Settings::Manager::apply("resolution x", "Video");
+        Settings::Manager::apply("resolution y", "Video");
 
         MWBase::Environment::get().getWindowManager()->windowResized(x, y);
+
+        // We should reload TrueType fonts to fit new resolution
+        MWBase::Environment::get().getWindowManager()->loadUserFonts();
     }
 
     void InputManager::windowClosed()
diff --git a/components/settings/settings.cpp b/components/settings/settings.cpp
index 27463ea88..81544aedc 100644
--- a/components/settings/settings.cpp
+++ b/components/settings/settings.cpp
@@ -414,6 +414,12 @@ void Manager::setBool(const std::string &setting, const std::string &category, c
     setString(setting, category, value ? "true" : "false");
 }
 
+void Manager::apply(const std::string &setting, const std::string &category)
+{
+    CategorySettingValueMap::key_type key = std::make_pair(category, setting);
+    mChangedSettings.erase(key);
+}
+
 const CategorySettingVector Manager::apply()
 {
     CategorySettingVector vec = mChangedSettings;
diff --git a/components/settings/settings.hpp b/components/settings/settings.hpp
index 7adcb9b39..f11f7ae6d 100644
--- a/components/settings/settings.hpp
+++ b/components/settings/settings.hpp
@@ -35,6 +35,8 @@ namespace Settings
         void saveUser (const std::string& file);
         ///< save user settings to file
 
+        static void apply(const std::string &setting, const std::string &category);
+
         static const CategorySettingVector apply();
         ///< returns the list of changed settings and then clears it