<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/101974/">http://git.reviewboard.kde.org/r/101974/</a>
</td>
</tr>
</table>
<br />
<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 Amarok.</div>
<div>By Thomas Karpiniec.</div>
<p style="color: grey;"><i>Updated July 17, 2011, 6:06 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;">When "show only matches" is enabled the listview tries to select the track and emit a signal about whether or not it matched before the proxy timeout has triggered the update. This happens to work anyway when additional keystrokes are narrowing the results but causes problems when I typo. For example, in my play list, "laz" returns 15 results and "lazz" returns 0. If I then backspace the second "z", it doesn't select the top track and the lineedit widget still appears in red.
This is a fairly direct solution: moving the relevant selection code to the timeout handler with some member vars to store the required arguments. If there's a better way to go about it please let me know and I'll try to bring it up to scratch.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing (updated)</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;">Applied against master. Tried it with a bunch of different patterns and left it for a while.
The only issue I've noticed is that if the list is empty and you press backspace then enter in quick succession, it will apply the "enter" to the empty list rather than the one that appears after the timeout. It's an existing problem though. Might look at that separately.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/playlist/view/listview/PrettyListView.h <span style="color: grey">(f22a7c8)</span></li>
<li>src/playlist/view/listview/PrettyListView.cpp <span style="color: grey">(9ce6573)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101974/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>