<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/100837/">http://git.reviewboard.kde.org/r/100837/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Any news on this? Please update the patch or remove it.</pre>
<br />
<p>- Myriam</p>
<br />
<p>On March 10th, 2011, 10 p.m., Nicholas Wilson 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 Nicholas Wilson.</div>
<p style="color: grey;"><i>Updated March 10, 2011, 10 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;">When playing albums, I want add a delay between albums. Each album should ideally be played seamlessly, but jumping with not even a second's delay to a random album creates a very jarring effect. This is a preliminary patch to add a few seconds' pause in random album mode.
Questions:
1) UI. If this makes the cut, I guess there should be a config for it. The play modes menu is getting a bit cluttered, so perhaps it could be slightly re-engineered. I'll think about it if there is a chance of the feature being shipped.
2) Do we want this option to be available in normal album traversal? Adding a few seconds between albums, possibly with config, seems very sensible to me.
Also, the RandomAlbums mode needs some love. It currently ignores the favouring code which the UI suggests it should follow, and is less clever than the favoured track mode at avoiding playing the same thing again based on history. Would it be OK to fix that?
So, if I write it, will you take it, or is this feature not wanted?</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 on local files only. Using it already to improve random albums experience.</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/EngineController.h <span style="color: grey">(4117a89)</span></li>
<li>src/EngineController.cpp <span style="color: grey">(ad85533)</span></li>
<li>src/playlist/navigators/RandomAlbumNavigator.cpp <span style="color: grey">(6f0fe37)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100837/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>