[kmail2] [Bug 484327] Bad initial window sizes

[Jonathan Marten]

https://bugs.kde.org/show_bug.cgi?id=484327

--- Comment #14 from Jonathan Marten <jjm at keelhaul.me.uk> ---
Git commit 2b7c3723f92f283c71b73fdf23b20a4c9ccf8cd1 by Jonathan Marten.
Committed on 24/09/2024 at 12:08.
Pushed by marten into branch 'master'.

Fix saving and restoring of window position as well as size

Commit 28480aeb implemented the use of the state configuration to store
window size and position for the composer and reader windows.  However,
to also correctly restore the window position, as noted in the referenced
bug, it is necessary to clear the flag used by KMainWindow to ensure that
multiple windows do not open in exactly the same position.  This is normally
done by KMainWindow::closeEvent() but has to be done by SecondaryWindow
itself for these windows.

The main window is also changed to use the state configuration, so that
its size will be correctly restored.  Restoring its position works when
KMail is run without a system tray icon, but with the icon the main window
always seems to open at the top left of the screen.  Possibly this is a
result of KWin overriding the application specified window placement
only in this case.

M  +3    -5    src/editor/kmcomposerwin.cpp
M  +3    -5    src/kmmainwin.cpp
M  +2    -4    src/kmreadermainwin.cpp
M  +23   -2    src/secondarywindow.cpp

https://invent.kde.org/pim/kmail/-/commit/2b7c3723f92f283c71b73fdf23b20a4c9ccf8cd1

-- 
You are receiving this mail because:
You are the assignee for the bug.