<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="https://git.reviewboard.kde.org/r/117984/">https://git.reviewboard.kde.org/r/117984/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#e0e0e0" width="100%" cellpadding="8" style="border: 1px gray solid;">
<tr>
<td>
<h1 style="margin-right: 0.2em; padding: 0; font-size: 10pt;">This change has been marked as submitted.</h1>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma.</div>
<div>By Shantanu Tushar.</div>
<p style="color: grey;"><i>Updated May 12, 2014, 2:14 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
plasma-mediacenter
</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;">The list of images that we get from Baloo has lot of noise because it usually contains small images from data files of software, games and so on.
We do two things-
* Ignore images unless they are wider than 500 pixels
* Make images with EXIF data appear first by giving them preference in sorting by date/time</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;">Photos are fetched, the ones from my recent trip to Kashmir show up first ;)</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>plugins/baloosearch/audiosearchresulthandler.h <span style="color: grey">(11b7219)</span></li>
<li>plugins/baloosearch/audiosearchresulthandler.cpp <span style="color: grey">(de98bb7)</span></li>
<li>plugins/baloosearch/baloosearchmediasource.h <span style="color: grey">(93e3d80)</span></li>
<li>plugins/baloosearch/baloosearchmediasource.cpp <span style="color: grey">(4180b1e)</span></li>
<li>plugins/baloosearch/imagesearchresulthandler.h <span style="color: grey">(200451e)</span></li>
<li>plugins/baloosearch/imagesearchresulthandler.cpp <span style="color: grey">(272b476)</span></li>
<li>plugins/baloosearch/searchresulthandler.h <span style="color: grey">(0df0ba2)</span></li>
<li>plugins/baloosearch/searchresulthandler.cpp <span style="color: grey">(78e41ea)</span></li>
<li>plugins/baloosearch/videosearchresulthandler.h <span style="color: grey">(146efca)</span></li>
<li>plugins/baloosearch/videosearchresulthandler.cpp <span style="color: grey">(1295b30)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/117984/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>