<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/104229/">http://git.reviewboard.kde.org/r/104229/</a>
</td>
</tr>
</table>
<br />
<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
<p style="margin-top: 0;">On March 11th, 2012, 10:36 p.m., <b>Marco Martin</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;">looks fine, but i would prefer the thumbnail plugins to be configurable if more than images are going to be used</pre>
</blockquote>
<p>On March 12th, 2012, 1:07 p.m., <b>Shantanu Tushar Jha</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;">I didn't understand the "more than images are going to be used" part :(
(KIO includes three plugins by default "directorythumbnail", "imagethumbnail", "jpegthumbnail")</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 mean, support for different types of thumbnail plugins configurable.
but since they are generated in a quite lazy way (no preview is generated until asked for) you can go for it, then we'll see if it impacts performancein any way
</pre>
<br />
<p>- Marco</p>
<br />
<p>On March 11th, 2012, 4:51 p.m., Shantanu Tushar Jha 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 Plasma and Marco Martin.</div>
<div>By Shantanu Tushar Jha.</div>
<p style="color: grey;"><i>Updated March 11, 2012, 4:51 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;">This patch makes two additions-
1. Thumbnail size is hardcoded into metadatamodel, this patch adds a property to make it configurable
2. KIO Thumbnailer uses a default set of plugins (from global config, which can't be set from PA) which doesn't include video thumbnailers for example. This makes metadatamodel fail when its asked for a thumbnail for a video file. This patch explicitly tells KIO to use all plugins.</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;">Both features work when tested from QML and don't seem to have any side-effects.</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>components/metadatamodel/metadatamodel.h <span style="color: grey">(85750fe)</span></li>
<li>components/metadatamodel/metadatamodel.cpp <span style="color: grey">(74ad09c)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104229/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>