From 6ba03dec35959de93f7a076bb952f23397d60d83 Mon Sep 17 00:00:00 2001 From: Sam Atkins Date: Fri, 29 Apr 2022 19:49:52 +0100 Subject: [PATCH] WindowServer: Save config file when mouse cursor theme is changed When user code requests the current cursor theme name with `GUI::ConnectionToWindowServer::the().get_cursor_theme()`, that reads the name from the config file. If we don't write that out when it's changed, then users get given an outdated cursor theme instead of the current one. In particular, changing the theme in MouseSettings, saving and closing it, then reopening MouseSettings, would show the wrong theme selected. So, that's fixed now. :^) --- Userland/Services/WindowServer/WindowManager.cpp | 3 +++ 1 file changed, 3 insertions(+) diff --git a/Userland/Services/WindowServer/WindowManager.cpp b/Userland/Services/WindowServer/WindowManager.cpp index 41015015d2d..912cd4dbf5c 100644 --- a/Userland/Services/WindowServer/WindowManager.cpp +++ b/Userland/Services/WindowServer/WindowManager.cpp @@ -2260,6 +2260,9 @@ void WindowManager::apply_cursor_theme(String const& theme_name) Compositor::the().invalidate_cursor(); m_config->write_entry("Mouse", "CursorTheme", theme_name); + if (auto result = m_config->sync(); result.is_error()) { + dbgln("Failed to save config file: {}", result.error()); + } } }