<table><tr><td style="">maxrd2 added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D13808">View Revision</a></tr></table><br /><div><div><p>The change in qt affects when the window close event is handled in queue, it didn't change when it is fired. With qt 5.11.1 child widgets get destroyed a bit sooner, but it's still gets triggered by same close event - i think that stayed like it always was.<br />
Also KMainWindow code is made to autosave settings (if not disabled) every time some relevant widget changes, so am pretty sure that this patch won't ever cause it to save incorrect settings.<br />
Window/widget visibility and size is unlikely to change by user after they click to close window.</p>

<p>However to be 100% sure will submit the test.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R263 KXmlGui</div></div></div><br /><div><strong>BRANCH</strong><div><div>fix-window-state-save</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13808">https://phabricator.kde.org/D13808</a></div></div><br /><div><strong>To: </strong>maxrd2, KDE Applications, dfaure, elvisangelaccio, broulik, cfeck<br /><strong>Cc: </strong>wbauer, aacid, ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>