Bug 494359

Summary:	KConfig.WindowStateSaver does not store window maximization state
Product:	[Frameworks and Libraries] frameworks-kconfig	Reporter:	EpicTux123 <EpicTux123>
Component:	general	Assignee:	Matthew Dawson <matthew>
Status:	RESOLVED FIXED
Severity:	normal	CC:	aleixpol, kdelibs-bugs, nate, report.bug
Priority:	NOR	Keywords:	regression
Version:	6.7.0
Target Milestone:	---
Platform:	Fedora RPMs
OS:	Linux
Latest Commit:	https://invent.kde.org/frameworks/kconfig/-/commit/43de355b2cb1bceda6884729c9a3f10eaf982ede	Version Fixed In:	6.9
Sentry Crash Report:

Description EpicTux123 2024-10-09 17:04:43 UTC

After upgrading to Plasma 6.2.0, Discover window state is not stored — i.e. maximized state.
Size and position are stored, but state — maximized or minimized — is not.

On X11 with proprietary NVIDIA drivers.

Operating System: Fedora Linux 40
KDE Plasma Version: 6.2.0
KDE Frameworks Version: 6.6.0
Qt Version: 6.7.2
Kernel Version: 6.10.12-200.fc40.x86_64 (64-bit)
Graphics Platform: X11

Comment 1 Nate Graham 2024-10-10 21:27:20 UTC

This will have been caused by the port to KConfig.WindowStateSaver, which gained the feature to remember maximization …but then later lost it due to the feature being reverted because it broke something else. Moving to KConfig to track where it needs to be fixed, since we have since ported, like, almost everything to KConfig.WindowStateSaver.

Comment 2 Bug Janitor Service 2024-11-05 22:24:14 UTC

A possibly relevant merge request was started @ https://invent.kde.org/frameworks/kconfig/-/merge_requests/336

Comment 3 Nate Graham 2024-11-06 05:49:36 UTC

*** Bug 494843 has been marked as a duplicate of this bug. ***

Comment 4 Joshua Goins 2024-11-06 15:16:51 UTC

Git commit 43de355b2cb1bceda6884729c9a3f10eaf982ede by Joshua Goins.
Committed on 05/11/2024 at 22:26.
Pushed by cullmann into branch 'master'.

Fix restoration of maximization state for QtQuick windows (for real)

The previous fix in 93b63d128022ebf670717d3330226a0949ffedc0 caused a
behavior change for QtWidgets applications since we ended up called
show().

To be safe let's only use setVisibility when dealing with QtQuick
windows. The fix was written to work around oddities only present in
QtQuick windows anyway.

M  +9    -1    src/qml/kwindowstatesaverquick.cpp

https://invent.kde.org/frameworks/kconfig/-/commit/43de355b2cb1bceda6884729c9a3f10eaf982ede