SUMMARY System Settings is no longer remembering window size and position STEPS TO REPRODUCE 1. open System Settings 2. ... 3. ... OBSERVED RESULT It used to open with the same geometry as when I closed it (at least the size) but now is opening the same size, and definitely not the same position. EXPECTED RESULT Opening the same size as when I closed it. SOFTWARE/OS VERSIONS Linux/KDE Plasma: 6.2 ADDITIONAL INFORMATION This is quite irritating as many of the settings pages are hard to use without making the window considerably larger.
Yeah, we tried to improve this in Plasma 6.2, but I think unfortunately we might have made it worse instead. I can intermittently reproduce that window sizes don't get saved consistently now. Sometimes they do, but sometimes they don't. Akseli, can you take a look at this?
Looks like the system settings, when it is resized to the absolute minimum size, ignores saving the state completely.
Found the reason: When window is set to its absolute minimum size, its seen as the window default size and kwindowconfig ignores saving the default size.
A possibly relevant merge request was started @ https://invent.kde.org/plasma/systemsettings/-/merge_requests/335
A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kconfig/-/merge_requests/335
Git commit 65db9ccc4bc8e3fbb10f1694391141287bd253ef by Akseli Lahtinen. Committed on 21/10/2024 at 18:08. Pushed by akselmo into branch 'master'. kwindowconfig: If sizes are same as default, revert them to default when saving This is most noticeable with system settings: If system settings size is saved and its the same as the default size for it, the code does nothing and leaves the old size state. This makes sure that when we have same size as the default, we also revert them to default. See also for more context: https://invent.kde.org/plasma/systemsettings/-/merge_requests/335#note_1055385 M +6 -1 src/gui/kwindowconfig.cpp https://invent.kde.org/frameworks/kconfig/-/commit/65db9ccc4bc8e3fbb10f1694391141287bd253ef
Git commit f3b32ad08b1e5685fcd2d70dba04c5a621872e1e by Akseli Lahtinen. Committed on 11/10/2024 at 11:31. Pushed by akselmo into branch 'master'. SettingsBase: Make sure we save window state on quit or geometryChange This change makes sure we save the window state settings when: - Screen geometry changes, such as moving between two different screens - When application is closed M +2 -0 app/SettingsBase.cpp https://invent.kde.org/plasma/systemsettings/-/commit/f3b32ad08b1e5685fcd2d70dba04c5a621872e1e
Git commit 4bd4bce4b834840772d91e643c4ddd3d633c636c by Akseli Lahtinen. Committed on 21/10/2024 at 18:14. Pushed by akselmo into branch 'Plasma/6.2'. SettingsBase: Make sure we save window state on quit or geometryChange This change makes sure we save the window state settings when: - Screen geometry changes, such as moving between two different screens - When application is closed (cherry picked from commit f3b32ad08b1e5685fcd2d70dba04c5a621872e1e) f3b32ad0 SettingsBase: Make sure we save window state on quit or geometryChange Co-authored-by: Akseli Lahtinen <akselmo@akselmo.dev> M +2 -0 app/SettingsBase.cpp https://invent.kde.org/plasma/systemsettings/-/commit/4bd4bce4b834840772d91e643c4ddd3d633c636c