<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/101015/">http://git.reviewboard.kde.org/r/101015/</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 14th, 2011, 12:15 p.m., <b>Ralf Engels</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;">When the track is stopped you should get the trackChanged signal (with the new track being empty). So that should update the lastPlayed time.
Actually you might want to wait a little bit after the "trackChanged" signal in case that the currentTrack Applet slot get's called before the collection had time to update the change.
Also if you stop the track to early it's not counted as being played.
So having a timer every minute seems to be unneeded.
Can you change the timer to a one shot timer being started after "trackChanged"?
Also is there a bug report about this problem?</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;">Without a timer every minute the times of the tracks are not updated when amarok is in stopped state. I think it is possible to update the tracks when the mouse is over the widget. But with the current architecture of the widget the list will be recreated every time it is updated, which does not look good.
Maybe every item of this can be updated on its own, but I don't know if I can manage it during lecture time.
I found no bug report.</pre>
<br />
<p>- Conrad</p>
<br />
<p>On April 4th, 2011, 5:23 p.m., Conrad Hübler 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 Amarok.</div>
<div>By Conrad Hübler.</div>
<p style="color: grey;"><i>Updated April 4, 2011, 5:23 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;">The recentlyplayedtracks-widget from the current track applet does not update the last played time of recently played tracks. It now happens, every 60000 msec and after amarok stops.
It is just a hack for the next release making this applet working as it should, but it is not a proper solution for this problem.</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/context/widgets/RecentlyPlayedListWidget.h <span style="color: grey">(b9b3d3b)</span></li>
<li>src/context/widgets/RecentlyPlayedListWidget.cpp <span style="color: grey">(2959c1c)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/101015/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>