<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/105841/">http://git.reviewboard.kde.org/r/105841/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On August 3rd, 2012, 10:43 p.m., <b>David Faure</b> wrote:</p>
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">How does this relate to the call to
mainWindow->resetAutoSaveSettings(); // Don't autosave
in KonqMainWindow::slotCreateNewWindow? It's kind of the same idea. Well, in fact no, it's better ;)
You're only setting "save window size" to false, while "resetAutoSaveSettings" means "don't autosave anything, not even toolbar placement or anything".
I guess what this means is: if the new bool is only set in slotCreateNewWindow then the existing resetAutoSaveSettings is fine, otherwise it needs to be done elsewhere too.
The code in setProfileConfig is about loading the default settings (from the profile), but I guess there isn't much to load for "popup windows without toolbars" indeed. (e.g. if someone had the sidebar enabled in the profile, it should probably not show up in such windows). So maybe rather a if() in that method.</pre>
</blockquote>
</blockquote>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I saw the mainWindow->resetAutoSaveSettings() call too and thought that should prevent any information about the new window from being saved. Unfortunately that does not happen for some reason and the bug reported in 200907 can be readily reproduced. Perhaps something else overrides that setting later on. I guess I will have to further investigate this and solve it properly just like the location bar focus issue.</pre>
<br />
<p>- Dawit</p>
<br />
<p>On August 3rd, 2012, 9:02 p.m., Dawit Alemayehu wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for KDE Base Apps and David Faure.</div>
<div>By Dawit Alemayehu.</div>
<p style="color: grey;"><i>Updated Aug. 3, 2012, 9:02 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Now that the special window popup scenario has been dealt with, the attached one liner attempts to resolve the issue of the size of such poup windows being saved whenver "setAutoSaveSettings" is invoked.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=200907">200907</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>konqueror/src/konqmainwindow.cpp <span style="color: grey">(6faba58)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105841/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>