<table><tr><td style="">ahmadsamir 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/D21876">View Revision</a></tr></table><br /><div><div><p>OK. So "window configuration" here means, the window size, state, and the view properties e.g. splitters... etc. The name threw me off (I confused it with KMainWindow window configuration), and the fact that saveSessionTo() always executes KateMainWindow::saveProperties() regardless of the config setting.</p>

<p>So saveSesssionTo() should only execute saveProperties() if "saveWindowConfig" is true; if you agree with that reasoning, I'll open a separate review request to keep the commits atomic.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R40 Kate</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D21876">https://phabricator.kde.org/D21876</a></div></div><br /><div><strong>To: </strong>ahmadsamir, Kate, cullmann, dhaumann<br /><strong>Cc: </strong>cfeck, kwrite-devel, domson, michaelh, ngraham, demsking, cullmann, sars, dhaumann<br /></div>