Bug 358146

Summary:	Window size and position are not saved
Product:	[Applications] krita	Reporter:	zottg <zottgrammes>
Component:	General	Assignee:	Krita Bugs <krita-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	halla, herrkjeldsen
Priority:	NOR	Keywords:	release_blocker
Version First Reported In:	3.0 Alpha
Target Milestone:	---
Platform:	Microsoft Windows
OS:	Microsoft Windows
Latest Commit:	http://commits.kde.org/krita/c3dc97c005ec0ad8408dce078ca159385be4a4ae	Version Fixed/Implemented In:
Sentry Crash Report:

Description zottg 2016-01-18 00:02:22 UTC

1. Launch Krita
2. Resize and reposition window
3. Close Krita
4. Open Krita
5. Previous size and position settings are not saved

Windows 10 64bit

Comment 1 zottg 2016-01-18 00:09:27 UTC

May be related to
https://bugs.kde.org/show_bug.cgi?id=335923
https://bugs.kde.org/show_bug.cgi?id=331819

Both were fixed but I don't know if they've been included in the 3.0 pre alpha.

Comment 2 zottg 2016-01-29 06:54:41 UTC

After some more inspection, it appears this is a duplicate of https://bugs.kde.org/show_bug.cgi?id=349165

*** This bug has been marked as a duplicate of bug 349165 ***

Comment 3 zottg 2016-01-29 06:57:44 UTC

Sorry, I tested this a little more and this is still an issue, despite the information in
https://bugs.kde.org/show_bug.cgi?id=349165

Comment 4 Marcus Kjeldsen 2016-04-10 09:33:23 UTC

Confirmed for 3.0 Alpha1 on win10

Comment 5 Halla Rempt 2016-04-16 13:46:02 UTC

https://phabricator.kde.org/T2300

Comment 6 Scott Petrovic 2016-04-27 03:14:04 UTC

Git commit 83b454224b67c14019d162ecf3a188f069f022e0 by Scott Petrovic.
Committed on 27/04/2016 at 03:13.
Pushed by scottpetrovic into branch 'master'.

T2300 : Window size and position is not saved

Save the x and y positions as well as the height and width of each docker .

M  +5    -0    libs/ui/KisMainWindow.cpp

http://commits.kde.org/krita/83b454224b67c14019d162ecf3a188f069f022e0

Comment 7 Halla Rempt 2016-04-27 07:03:20 UTC

Actually, we still don't correctly restore a maximized window on Windows, this only fixes the docker positions. (Which is weird, that should be saved correctly by the saving the workspace state).

Comment 8 Scott Petrovic 2016-04-28 19:03:43 UTC

Git commit c3dc97c005ec0ad8408dce078ca159385be4a4ae by Scott Petrovic.
Committed on 28/04/2016 at 18:39.
Pushed by scottpetrovic into branch 'master'.

Load position of the main window correctly
Fixes T2300

M  +1    -0    libs/ui/KisApplication.cpp
M  +0    -2    libs/ui/KisMainWindow.cpp

http://commits.kde.org/krita/c3dc97c005ec0ad8408dce078ca159385be4a4ae