<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/104982/">http://git.reviewboard.kde.org/r/104982/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On May 23rd, 2012, 12:36 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;">I'm ok with the change, but it seems this area is still buggy in any case...
"the location bar must be restored to the URL of the content currently displayed" doesn't work here, e.g. when typing ~/foo.exe (yes a windows executable, everything else I tried got embedded) in the location bar in a tab that shows www.kde.org, doesn't result in a "revert" of the location bar url.
Generally speaking, I'm not sure that it's such a problem that the location bar doesn't match the contents. This happens all the time -- e.g. when you start typing in the location bar ;) And this even gets saved when switching between tabs, very much on purpose.
So maybe we just should not revert the location bar contents, just leave what the user typed. That's what the user in 270723 asks for, that's what my testing with foo.exe shows too (i.e. it's already the current behavior in some cases), and that's consistent with "we preserve what the user has typed" (already the case currently, when editing the location bar, without hitting Enter, and switching tabs).</pre>
</blockquote>
<p>On May 24th, 2012, 4:34 p.m., <b>Dawit Alemayehu</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;">I do not have a problem with doing that especially since pressing ESC while in the location bar will automatically restore the URL of the current content. Do you want me to completely remove the setLocationBarURL call then ?</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;">ESC restores the url when typing, indeed. But not in the case of URLs not handled by konqueror.
Anyway, it seems we agree, let's remove the call.</pre>
<br />
<p>- David</p>
<br />
<p>On May 18th, 2012, 7:54 a.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 May 18, 2012, 7:54 a.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;">The attached patch fixes the issue reported in the aforementioned bug report. Specifically, it stops Konqueror from restoring the location bar to the previous URL if
a.) It was empty as in a newly created tab.
b.) The user typed in a new URL that cannot be handled by Konqueror. The URL was launched with another application.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </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;">Tested the following three scenarios:
1.) Open a new tab in Konqueror and type-in a URL it cannot handle.
2.) Open a new tab, browse to any location, and then type-in a URL Konqueror cannot handle.
3.) Open a new tab, browse to any location, clear the location bar, and then type-in a URL Konqueror cannot handle.
Results with the patch:
1.) The typed in URL remains in the location bar.
2.) The previous URL is restored in the location bar.
3.) Same as #2.
</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=270723">270723</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/konqview.cpp <span style="color: grey">(bfeb20b)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104982/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>