<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/100181/">http://git.reviewboard.kde.org/r/100181/</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 15th, 2011, 11:15 a.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;">Latest state of this review request please.
Did you come up with a new solution?
Is this request discarded or still worked on?</pre>
</blockquote>
<p>On February 28th, 2012, 9:07 a.m., <b>Myriam Schweingruber</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;">Any news on this? Please update the patch if needed, else remove it.
The 3 bug reports have the target set to 2.6 now. Bart?</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;">I'll integrate the code for saving selected tracks as a new playlist but not for the updating. It just make the Playlist even more like an editor, which is the source of so many usability problems. Patches are on the way that make the playlist browser do that job a lot better.</pre>
<br />
<p>- Bart</p>
<br />
<p>On December 6th, 2010, 2:07 p.m., Dennis Francis 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 Dennis Francis.</div>
<p style="color: grey;"><i>Updated Dec. 6, 2010, 2:07 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;">Added functionality in the context menu to save the selected tracks ( on the right side )
- to a new playlist ( currently only sqlplaylist and fileplaylist providers are only supported )
- into an already existing playlist
( Duplication is avoided hence feels like a playlist update )</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;">It works fine foe me.</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="https://bugs.kde.org/show_bug.cgi?id=186545">186545</a>,
<a href="https://bugs.kde.org/show_bug.cgi?id=211811">211811</a>,
<a href="https://bugs.kde.org/show_bug.cgi?id=239950">239950</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>src/browsers/playlistbrowser/PlaylistBrowserModel.cpp <span style="color: grey">(34d84d9)</span></li>
<li>src/browsers/playlistbrowser/UserPlaylistModel.cpp <span style="color: grey">(2178d27)</span></li>
<li>src/core-impl/collections/mediadevicecollection/playlist/MediaDeviceUserPlaylistProvider.h <span style="color: grey">(e1ae132)</span></li>
<li>src/core-impl/collections/mediadevicecollection/playlist/MediaDeviceUserPlaylistProvider.cpp <span style="color: grey">(12d3b46)</span></li>
<li>src/core-impl/playlists/providers/user/UserPlaylistProvider.h <span style="color: grey">(609e1a8)</span></li>
<li>src/core/playlists/PlaylistFormat.h <span style="color: grey">(b93180d)</span></li>
<li>src/core/playlists/PlaylistFormat.cpp <span style="color: grey">(6b3cb6b)</span></li>
<li>src/core/playlists/PlaylistProvider.h <span style="color: grey">(4cc3417)</span></li>
<li>src/core/playlists/PlaylistProvider.cpp <span style="color: grey">(2d37e8b)</span></li>
<li>src/playlist/PlaylistDock.cpp <span style="color: grey">(d512b28)</span></li>
<li>src/playlist/PlaylistModel.h <span style="color: grey">(3ad8030)</span></li>
<li>src/playlist/PlaylistModel.cpp <span style="color: grey">(1acbc9f)</span></li>
<li>src/playlistmanager/PlaylistManager.h <span style="color: grey">(943fcf1)</span></li>
<li>src/playlistmanager/PlaylistManager.cpp <span style="color: grey">(fb13ea7)</span></li>
<li>src/playlistmanager/file/PlaylistFileProvider.h <span style="color: grey">(bd19f79)</span></li>
<li>src/playlistmanager/file/PlaylistFileProvider.cpp <span style="color: grey">(b0ec53a)</span></li>
<li>src/playlistmanager/sql/SqlUserPlaylistProvider.h <span style="color: grey">(3a5a62e)</span></li>
<li>src/playlistmanager/sql/SqlUserPlaylistProvider.cpp <span style="color: grey">(a899843)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100181/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>