<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://svn.reviewboard.kde.org/r/4256/">http://svn.reviewboard.kde.org/r/4256/</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;">What's the use case for such an addition?
The case that does not work in your patch is when you add 1 picture which is the same as a static picture but then you get the slideshow arrows.
I am not very keen of adding code for new features without a good use case as it generates bugs and clutters the code (which happened a lot with this applet).
It is understood that people will display all files from a folder in a slideshow (or from different folders) and if they don't want this picture they will remove it from the folder. If you want separate pictures from several folders, just copy them in a unique folder and point the slideshow to this folder. For consistency, the Desktop slideshow has no 'Add file' for example.
</pre>
<br />
<p>- Anne-Marie</p>
<br />
<p>On October 6th, 2010, 12:19 p.m., michal seben wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://svn.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 Plasma.</div>
<div>By michal seben.</div>
<p style="color: grey;"><i>Updated 2010-10-06 12:19:36</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 enhance slideshow in frame plasmoid, after applying user could add separate pictures to slideshow. Currently slideshow could contain only (all) files in chosen directories.
</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>/trunk/KDE/kdeplasma-addons/applets/frame/configdialog.cpp <span style="color: grey">(1135431)</span></li>
<li>/trunk/KDE/kdeplasma-addons/applets/frame/frame.h <span style="color: grey">(1135431)</span></li>
<li>/trunk/KDE/kdeplasma-addons/applets/frame/frame.cpp <span style="color: grey">(1135431)</span></li>
<li>/trunk/KDE/kdeplasma-addons/applets/frame/imageSettings.ui <span style="color: grey">(1135431)</span></li>
<li>/trunk/KDE/kdeplasma-addons/applets/frame/slideshow.cpp <span style="color: grey">(1135431)</span></li>
</ul>
<p><a href="http://svn.reviewboard.kde.org/r/4256/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>