<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>
<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;">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>