<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/100083/">http://git.reviewboard.kde.org/r/100083/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/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 Ralf Engels.</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;">Use the default Qt tooltips for playlist tooltips.
The playlist tooltip text was invisible on black backgounds.
In addition the custom tooltip implementation did not follow the platform style, was complicate and unneeded.
Just using a normal tooltip solves all problems.
</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;">Testing on two different machines with different styles and colors.</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/CMakeLists.txt <span style="color: grey">(c55bb83)</span></li>
<li>src/playlist/PlaylistModel.h <span style="color: grey">(f213d65)</span></li>
<li>src/playlist/PlaylistModel.cpp <span style="color: grey">(4008cf9)</span></li>
<li>src/playlist/view/listview/PrettyListView.h <span style="color: grey">(e4893e3)</span></li>
<li>src/playlist/view/listview/PrettyListView.cpp <span style="color: grey">(ca3d580)</span></li>
<li>src/playlist/view/tooltips/AmarokToolTip.cpp <span style="color: grey">(831f08d)</span></li>
<li>src/playlist/view/tooltips/KFormattedBalloonTipDelegate.cpp <span style="color: grey">(e8a1479)</span></li>
<li>src/playlist/view/tooltips/KToolTip.cpp <span style="color: grey">(d219cf9)</span></li>
<li>src/playlist/view/tooltips/KToolTipDelegate.cpp <span style="color: grey">(2724031)</span></li>
<li>src/playlist/view/tooltips/KToolTipItem.cpp <span style="color: grey">(cbc09b3)</span></li>
<li>src/playlist/view/tooltips/ToolTipManager.cpp <span style="color: grey">(ffe91b5)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100083/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>