Bug 358146 - Window size and position are not saved
Summary: Window size and position are not saved
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: General (show other bugs)
Version: 3.0 Alpha
Platform: Microsoft Windows Microsoft Windows
: NOR normal
Target Milestone: ---
Assignee: Krita Bugs
URL:
Keywords: release_blocker
Depends on:
Blocks:
 
Reported: 2016-01-18 00:02 UTC by zottg
Modified: 2016-04-28 19:03 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
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