<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/114152/">http://git.reviewboard.kde.org/r/114152/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<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 good to me.
Please push this patch to master if you have KDE developer account, if not then I will do that.</pre>
<br />
<p>- Sinny Kumari</p>
<br />
<p>On November 27th, 2013, 4:02 p.m. UTC, Egor Matirov wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://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 Egor Matirov.</div>
<p style="color: grey;"><i>Updated Nov. 27, 2013, 4:02 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;">Currently, when you have selected first image from picture strip and clicks previous image button then nothing happen. Same way, when user reaches to the last image and clicks next image button, then nothing happens.
This patch fix it. GCI task: http://www.google-melange.com/gci/task/view/google/gci2013/5783943471169536</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>mediaelements/imageviewer/PictureStrip.qml <span style="color: grey">(4825f2e)</span></li>
<li>mediaelements/imageviewer/PictureStripDelegate.qml <span style="color: grey">(9202a52)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/114152/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>