<table><tr><td style="">dfaure 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>I don't understand your question.<br />
By "this", do you mean the KMainWindow fix that was posted in a previous iteration here and that was pushed? That one is very much needed, yes, to make setAutoSaveSettings work.<br />
The change that you link to, is about porting KMail to setAutoSaveSettings, so that it benefits from that fix.</p>

<p>If by "this" you mean what's left in this review request, which is to write a proper unittest for the KMainWindow fix, yes, that would still be very nice to have. But apparently it's pretty hard to write (if emulating native events is the only way then indeed it seems really difficult)... so maybe we have to give up on the idea.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R263 KXmlGui</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>anthonyfieroni, marten, asturmlechner, wbauer, aacid, ngraham, kde-frameworks-devel, michaelh, bruns<br /></div>